Skip to content

bpo-31569: globally fix case for PCbuild where it was PCBuild #3711

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 25, 2017

Conversation

gronke
Copy link
Contributor

@gronke gronke commented Sep 23, 2017

While setting up the development environment for Windows I've noticed PCbuild/ was sometimes written with a different case. This pull-requests replaces all occasions of PCBuild with PCbuild.

https://bugs.python.org/issue31569

@serhiy-storchaka
Copy link
Member

Please open an issue on the bug tracker for discussing your changes.

File systems on Windows are case-insensitive. It is not clear what is the canonical name of PCbuild. Wouldn't be easier to make a replace in the reverse direction?

@gronke gronke changed the title globally fix case for PCbuild where it was PCBuild bpo-31569: globally fix case for PCbuild where it was PCBuild Sep 24, 2017
@gronke
Copy link
Contributor Author

gronke commented Sep 24, 2017

File systems on Windows are case-insensitive. It is not clear what is the canonical name of PCbuild. Wouldn't be easier to make a replace in the reverse direction?

The directory was called PCbuild since 2.7, so I guess PCBuild is a typo that was never noticed because it was never an issue on case-insensitive filesystems.

@gronke gronke force-pushed the correct-PCbuild-typo branch from 1e4d331 to af22f6c Compare September 24, 2017 22:31
@pfmoore pfmoore merged commit f1502d0 into python:master Sep 25, 2017
@gronke gronke deleted the correct-PCbuild-typo branch September 25, 2017 17:00
@josephshen
Copy link
Contributor

this bug happened and I think it's need port back to all affected version.
it happens when try to compile on on a VM's mapped host filesystem ( e.g. host macOS with case-sensetive filesystem and client Windows)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants