[issue36588] change sys.platform() to just "aix" for AIX

2019-04-13 Thread Michael Felt
Michael Felt added the comment: On 12/04/2019 23:16, Michael Felt wrote: > Agreed, in case of doubt - leave alone (never change a winning team). > > And, to make it a short reply - I'll get started, and we see where it > leads us. I opened issue36624 (https://bugs.python.org/issue36624) - befo

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-12 Thread Michael Felt
Michael Felt added the comment: On 12/04/2019 17:34, STINNER Victor wrote: > STINNER Victor added the comment: > >> But, should I just continue standard practice (sys.platform), or would >> this be a moment to move towards platform.system() (i.e., set the >> example to be to use "run-time" rat

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-12 Thread STINNER Victor
STINNER Victor added the comment: > But, should I just continue standard practice (sys.platform), or would > this be a moment to move towards platform.system() (i.e., set the > example to be to use "run-time" rather than "build-time"). Oh, now I'm confused :-) I checked the Python test suite:

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-12 Thread Michael Felt
Michael Felt added the comment: On 12/04/2019 16:16, STINNER Victor wrote: > STINNER Victor added the comment: > > Do you want to work on a change to replace sys.platform.startswith("aix") to > cleanup the stdlib and tests? Not sure if it's needed :-) It's up to you. > > -- > > __

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-12 Thread STINNER Victor
STINNER Victor added the comment: Do you want to work on a change to replace sys.platform.startswith("aix") to cleanup the stdlib and tests? Not sure if it's needed :-) It's up to you. -- ___ Python tracker ___

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-12 Thread STINNER Victor
Change by STINNER Victor : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed versions: -Python 3.9 ___ Python tracker ___ ___

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-12 Thread STINNER Victor
STINNER Victor added the comment: New changeset 9d949f7796da612f1b588d18c6f041376992a9fc by Victor Stinner (Michael Felt) in branch 'master': bpo-36588: On AIX, remove major version from sys.platform (GH-12787) https://github.com/python/cpython/commit/9d949f7796da612f1b588d18c6f041376992a9fc

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-11 Thread Michael Felt
Change by Michael Felt : -- keywords: +patch pull_requests: +12714 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-li

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-11 Thread Michael Felt
Michael Felt added the comment: Was: root@x064:[/data/prj/python/python3-3.8]./python Python 3.8.0a3+ (heads/bpo-28009-2-dirty:2fb2bc81c3, Apr 11 2019, 07:09:55) [C] on aix6 Type "help", "copyright", "credits" or "license" for more information. >>> import sys >>> sys.platform 'aix6' >>> import

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-10 Thread STINNER Victor
STINNER Victor added the comment: > However, I am less familiar with (where) the appropriate documentation is. A > pointer to the documentation is appreciated. It would be nice to add a note to: * https://docs.python.org/dev/whatsnew/3.8.html#changes-in-the-python-api * https://docs.python.o

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-10 Thread Michael Felt
Michael Felt added the comment: On 10/04/2019 17:05, STINNER Victor wrote: > STINNER Victor added the comment: > > I like the idea. Would you like to propose a patch? I suggest to only make > such change in Python 3.8 and properly document it. > > -- > nosy: +vstinner > > _

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-10 Thread STINNER Victor
STINNER Victor added the comment: I like the idea. Would you like to propose a patch? I suggest to only make such change in Python 3.8 and properly document it. -- nosy: +vstinner ___ Python tracker ___

[issue36588] change sys.platform() to just "aix" for AIX

2019-04-10 Thread Michael Felt
New submission from Michael Felt : This is something that probably shouts - boring - but back in 2012 it was a hot topic for linux2 and linux3. Maybe - as far back as 1996 (when AIX4 was new) "aix3" and "aix4" made sense. Whether that is true, or not - is pointless these days - for Python3.