-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
gh-136759: Rename lock.h to pylock.h #137041
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
🤖 New build scheduled with the buildbot fleet by @colesbury for commit f265080 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F137041%2Fmerge If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again. |
I thought it would be good to review OSS code to see if there are any third-party projects that have a file "pylock.h", as we probably don't want to break those projects either. Fortunately I was not able to find anything that obviously conflicts. Pixar's OpenUSD repository does have a file that is name-equivalent on case-insensitive filesystems like NTFS on Windows: https://github.com/PixarAnimationStudios/OpenUSD/blob/d0d5c33d5655f206a02b58311efa841890fb3894/pxr/base/tf/pyLock.h. However, this file is only included as There were no matches on pypi.org data either. I ran the following ClickHouse query against the https://py-code.org data to verify that:
This came back with every row having 0 count. |
Thanks @aisk for the PR, and @kumaraditya303 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13. |
Thanks @aisk for the PR, and @kumaraditya303 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14. |
Sorry, @aisk and @kumaraditya303, I could not cleanly backport this to
|
Rename `lock.h` to `pylock.h` to avoid conflicts with headers of other projects. (cherry picked from commit ec02db5) Co-authored-by: AN Long <aisk@users.noreply.github.com>
GH-137072 is a backport of this pull request to the 3.14 branch. |
I'll try to resolve the backport conflict for 3.13. |
GH-137075 is a backport of this pull request to the 3.13 branch. |
Uh oh!
There was an error while loading. Please reload this page.