Skip to content

feat: Add proper support for IBM i - #140

Merged
cclauss merged 1 commit into
nodejs:mainfrom
kadler:ibmi-py39-support
Mar 4, 2022
Merged

feat: Add proper support for IBM i#140
cclauss merged 1 commit into
nodejs:mainfrom
kadler:ibmi-py39-support

Conversation

@kadler

@kadler kadler commented Mar 3, 2022

Copy link
Copy Markdown
Contributor

Python 3.9 on IBM i now properly returns "os400" for sys.platform
instead of claiming to be AIX as it did previously. While the IBM i PASE
environment is compatible with AIX, it is a subset and has numerous
differences which makes it beneficial to distinguish, however this means
that it now needs explicit support here.

Python 3.9 on IBM i now properly returns "os400" for sys.platform
instead of claiming to be AIX as it did previously. While the IBM i PASE
environment is compatible with AIX, it is a subset and has numerous
differences which makes it beneficial to distinguish, however this means
that it now needs explicit support here.
@kadler

kadler commented Mar 3, 2022

Copy link
Copy Markdown
Contributor Author

Related node-gyp PR: nodejs/node-gyp#2626

CC @ThePrez

@cclauss cclauss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Thanks for doing this.

@cclauss
cclauss merged commit fdda4a3 into nodejs:main Mar 4, 2022
@kadler
kadler deleted the ibmi-py39-support branch September 22, 2022 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants