[issue46744] installers on ARM64 suggest wrong folders

2022-03-15 Thread conio
conio added the comment: I opened a PR for this change (https://github.com/python/cpython/pull/31803). I'd appreciate your comments and/or approval. I didn't add anything to the NEWS.d folder since it seems to me like something completely internal to the build process and so minor not worth

[issue46744] installers on ARM64 suggest wrong folders

2022-03-10 Thread Roundup Robot
Change by Roundup Robot : -- nosy: +python-dev nosy_count: 5.0 -> 6.0 pull_requests: +29904 pull_request: https://github.com/python/cpython/pull/31803 ___ Python tracker ___

[issue46744] installers on ARM64 suggest wrong folders

2022-03-07 Thread Steve Dower
Steve Dower added the comment: Good call on the batch file. It should be easy enough to make those options case-insensitive, and to support both forms of x86 flag. I'll leave this issue open for that if someone wants to have a go, otherwise I'll get to them at some point. --

[issue46744] installers on ARM64 suggest wrong folders

2022-03-06 Thread conio
conio added the comment: 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

[issue46744] installers on ARM64 suggest wrong folders

2022-03-03 Thread Steve Dower
Steve Dower added the comment: New changeset 8f31bf46980956c735dd18f9914f3e7144e87c77 by Steve Dower in branch 'main': bpo-46744: Move Windows ARM64 installation directory to correct ProgramFiles (GH-31677) https://github.com/python/cpython/commit/8f31bf46980956c735dd18f9914f3e7144e87c77

[issue46744] installers on ARM64 suggest wrong folders

2022-03-03 Thread Steve Dower
Steve Dower added the comment: Posted half a PR for this, which is worth taking in itself, but doesn't completely address the whole change. It will install to the correct ProgramFiles folder on ARM64 now, but still with the suffix. If we are to take the rest of the change (use "-amd64" for

[issue46744] installers on ARM64 suggest wrong folders

2022-03-03 Thread Steve Dower
Change by Steve Dower : -- keywords: +patch pull_requests: +29797 stage: -> patch review pull_request: https://github.com/python/cpython/pull/31677 ___ Python tracker ___

[issue46744] installers on ARM64 suggest wrong folders

2022-02-15 Thread Steve Dower
Steve Dower added the comment: Yeah, currently the only detection that happens is OS version, and just so we can block (or at least log) users who are trying to install on supported versions. The install directories are unconditional. Any dynamic detection code would have to go into

[issue46744] installers on ARM64 suggest wrong folders

2022-02-14 Thread conio
conio added the comment: I understand. Though I think the naming scheme that includes platform identification is better, if *all* installers include the platform name unconditionally, including both amd64 and arm64, that's also an improvement. (So the names will be `Python311-32`,

[issue46744] installers on ARM64 suggest wrong folders

2022-02-14 Thread Steve Dower
Steve Dower added the comment: Thanks for the additional info! I agree with the design you propose. It would be much easier to make the ARM64 install use the tagged directory though (C:\Program Files\Python 3.11-arm64). I'd rather not get into the business of detecting platforms from the

[issue46744] installers on ARM64 suggest wrong folders

2022-02-13 Thread conio
New submission from conio : Thank you for your work on bringing Python to Windows on ARM64. I recently installed it an noticed some strange behaviours. On ARM64 Windows 11 the recent prerelease (3.11.0a5, post #33125) acts in a way I believe is wrong: Checking the _install for all users_