This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author conio
Recipients conio, paul.moore, steve.dower, tim.golden, zach.ware
Date 2022-03-07.00:01:13
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1646611273.61.0.561959744764.issue46744@roundup.psfhosted.org>
In-reply-to
Content
Thank you very much, Steve.

I took a look but didn't get to making the patch. I'm happy to see it was just removing the
calculation of the PF(ARM) folder.

For the second issue, even though I prefer it the way I said (unqualified = native), since

(i)   the default is using the value of `sys.winver` at the end of the path, and it's "VER"
      for amd64 Python and "VER-32" for x86 Python regardless of the OS (i.e. even on 32-bit
      Windows); 
(ii)  this has been the situation long enough to be a compatibility concern as you said, and
(iii) if someone really wants the names to be the other way around they may choose whatever
      they like during the install, and it's only the defaults that are that way

I think you are right in keeping it out for now. I wouldn't insist on it. If there comes a
time when more arguments in favor of this change come up or the circumstances change (e.g.
the compatibility issue becomes less of a concern) I'm sure you'd revisit this.

I did notice one small issue when I tried to build Python for ARM64 following your message
from 2022-02-15.

The help message for `Tools/msi/build.bat` says to use the `-ARM64` flag to build installers
for ARM64 but it didn't work for me. I took me a few attempts at clean and rebuild before I
finally looked at the script itself and saw that it actually expects `-arm64` in lowercase.

If you could change either the help string or the flag itself so they'd math it would be great.

Uppercase is nice because it is consistent with `PCbuild/build.bat` and lowercase is nice
because if there's come CI that uses this flag it wouldn't have to be modified too. Obviously
you if there are such consideration better than me so I leave the decision to you.

If there's no problem or difficulty in changing the arguments for `Tools/msi/build.bat` it
might be nice if building for ARM64 would be "-ARM64" and building for x86 would be "-Win32"
to match the platform names for `PCbuild/build.bat`.
History
Date User Action Args
2022-03-07 00:01:13coniosetrecipients: + conio, paul.moore, tim.golden, zach.ware, steve.dower
2022-03-07 00:01:13coniosetmessageid: <1646611273.61.0.561959744764.issue46744@roundup.psfhosted.org>
2022-03-07 00:01:13coniolinkissue46744 messages
2022-03-07 00:01:13coniocreate