-
-
Notifications
You must be signed in to change notification settings - Fork 30.7k
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
bpo-40747: Make py_version_nodot 3_10 not 310 #20333
Changes from all commits
fd5a12d
76a596c
5eb6a84
9759ea9
b9fe6c2
7149e08
4f9b869
48ec1e9
11be5f5
079e5e3
5e3760b
411c150
d201030
b549f09
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
The PEP 425 python tag, taken from ``py_version_nodot``, adds a ``_`` so ``cp310`` is now ``cp3_10``. | ||
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1683,10 +1683,10 @@ SelectPythonDlgProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam) | |
&py_major, &py_minor); | ||
if (result == 2) { | ||
#ifdef _DEBUG | ||
wsprintf(pythondll, "python%d%d_d.dll", | ||
wsprintf(pythondll, "python%d_%d_d.dll", | ||
py_major, py_minor); | ||
#else | ||
wsprintf(pythondll, "python%d%d.dll", | ||
wsprintf(pythondll, "python%d_%d.dll", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This will cause us to have to manually rebuild and commit There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Even if PEP 632 to remove distutils is approved, these changes will need to be reflected in pypa/setuptools' copy of distutils, or did they drop bdist_wininst ? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Question for @jaraco, but hopefully they dropped it. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thusfar, it only raises a DeprecationWarning (aka not a user-visible warning), so may need some care to disable it. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think it's fine to merge with or without this change, and we'll just track the distutils breakage as a new bug. Don't hold up this PR on it. |
||
py_major, py_minor); | ||
#endif | ||
} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,7 +10,7 @@ | |
</PropertyGroup> | ||
<Import Project="..\msi.props" /> | ||
<PropertyGroup> | ||
<DocFilename>python$(MajorVersionNumber)$(MinorVersionNumber)$(MicroVersionNumber)$(ReleaseLevelName).chm</DocFilename> | ||
<DocFilename>python$(MajorVersionNumber)_$(MinorVersionNumber)$(MicroVersionNumber)$(ReleaseLevelName).chm</DocFilename> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should probably separate the minor and micro version of the doc filename too - |
||
<IncludeDocFile>false</IncludeDocFile> | ||
<IncludeDocFile Condition="$(BuildForRelease) or Exists('$(PySourcePath)Doc\build\htmlhelp\$(DocFilename)')">true</IncludeDocFile> | ||
</PropertyGroup> | ||
|
@@ -27,4 +27,4 @@ | |
</ItemGroup> | ||
|
||
<Import Project="..\msi.targets" /> | ||
</Project> | ||
</Project> |
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.
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.
None of the other blurbs in that directory have a contributor name.
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.
That doesn't mean the other news entries are wrong. 😉 (And this is coming from the person who created this workflow, so I can say with certainty you can put your name in the news entry.)
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.
I would prefer not to, but won't stop you from commiting the change.