Skip to content

[2.7] Docs: fix some wrong words (GH-6987)#7572

Closed
Mariatta wants to merge 1 commit into
python:2.7from
Mariatta:backport-b5c246f-2.7
Closed

[2.7] Docs: fix some wrong words (GH-6987)#7572
Mariatta wants to merge 1 commit into
python:2.7from
Mariatta:backport-b5c246f-2.7

Conversation

@Mariatta

Copy link
Copy Markdown
Member

Fix typos in code comments: bdb.py and configure.ac..
(cherry picked from commit b5c246f)

Co-authored-by: Eitan Adler grimreaper@users.noreply.github.com

Fix typos in code comments: bdb.py and configure.ac..
(cherry picked from commit b5c246f)

Co-authored-by: Eitan Adler <grimreaper@users.noreply.github.com>
@Mariatta

Copy link
Copy Markdown
Member Author

@ned-deily I ran autoconf here. Can you check if this is right? It has more changes than I expect..

@ned-deily ned-deily left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The giveaway is that all the diffs are in configure.ac, not in configure. The latter is generated by the autoconf not the former. Looks like the issue is that the 2.7 configure.ac has a lot of trailing spaces cruft that has been there forever and now is probably not the time to change it. Recommend that you revert the edit to configure.ac and just make the changes manually then rerun autoconf.

@matrixise

Copy link
Copy Markdown
Member

@Mariatta @ned-deily
You will find my backport of this PR here #10315

@ned-deily

Copy link
Copy Markdown
Member

Thanks for the PR. It has been superseded by #10315.

@ned-deily ned-deily closed this Nov 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants