-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Release proposal: v5.0.4 #1893
Release proposal: v5.0.4 #1893
Conversation
Added "#pragma unmanaged" in win_delay_load_hook.cc to support clr PR-URL: #1819 Reviewed-By: João Reis <reis@janeasystems.com>
Make Python 3 compatiblity changes so the code works in both Python 2 and Python 3. Especially, make changes required because the compiler module was removed in Python 3 in favor of the ast module that exists in both Python 2 and Python 3. PR-URL: #1820 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Rod Vagg <r@va.gg> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
This change supports access to StringIO on both Python 2 and Python 3 PR-URL: #1836 Reviewed-By: Rod Vagg <r@va.gg> Reviewed-By: João Reis <reis@janeasystems.com>
PR-URL: #1843 Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Rod Vagg <r@va.gg> Reviewed-By: Matt Cowley <me@mattcowley.co.uk> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Used scoped disabling of managed code handling to ensure no other files get affected. PR-URL: #1852 Reviewed-By: João Reis <reis@janeasystems.com>
PR-URL: #1860 Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Rod Vagg <r@va.gg>
PR-URL: #1853 Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
PR-URL: #1858 Reviewed-By: Christian Clauss <cclauss@me.com> Reviewed-By: Rod Vagg <r@va.gg>
(accepted with minor modifications by rvagg) PR-URL: #1802 Reviewed-By: Rod Vagg <rod@vagg.org>
PR-URL: #1863 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
PR-URL: #1888 Reviewed-By: Rod Vagg <rod@vagg.org>
PR-URL: #1884 Reviewed-By: Rod Vagg <rod@vagg.org>
PR-URL: #1890 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
This PR isn't meant to be merged into |
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.
A few suggestions... Implemented on #1894
@@ -44,6 +45,12 @@ matrix: | |||
python: 3.7 | |||
env: NODE_GYP_FORCE_PYTHON=python3 EXPERIMENTAL_NODE_GYP_PYTHON3=1 | |||
before_install: nvm install 12 | |||
- name: "Python 3.7 on macOS" | |||
os: osx | |||
#osx_image: xcode11 |
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.
This # should be removed so that we use Py37 instead of Py36.
stdout=subprocess.PIPE, | ||
stderr=subprocess.STDOUT) | ||
stdout,stderr = out.communicate() | ||
return "CYGWIN" in str(stdout) |
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.
Insert the if PY3: stdout = stdout.decode("utf-8") code from #1890.
stdout=subprocess.PIPE, | ||
stderr=subprocess.STDOUT) | ||
stdout,stderr = out.communicate() | ||
return str(stdout) |
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.
Insert the if PY3: stdout = stdout.decode("utf-8") code from #1890.
stdout=subprocess.PIPE, | ||
stderr=subprocess.STDOUT) | ||
stdout,stderr = out.communicate() | ||
return "CYGWIN" in str(stdout) |
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.
Insert the if PY3: stdout = stdout.decode("utf-8") code from #1890.
hm, I'm not sure why this isn't showing properly, I've only removed that one commit which makes this the only meaningful diff when doing a diff --git a/lib/find-python.js b/lib/find-python.js
index 30bb25f..1a4390d 100644
--- a/lib/find-python.js
+++ b/lib/find-python.js
@@ -19,7 +19,7 @@ PythonFinder.prototype = {
argsExecutable: [ '-c', 'import sys; print(sys.executable);' ],
argsVersion: [ '-c', 'import sys; print("%s.%s.%s" % sys.version_info[:3]);' ],
semverRange: process.env.EXPERIMENTAL_NODE_GYP_PYTHON3 ? '2.7.x || >=3.5.0'
- : '>=2.7.0 <3.0.0',
+ : '>=2.6.0 <3.0.0',
// These can be overridden for testing:
execFile: cp.execFile, The "files changed" isn't correct, at least according to my diff. I suppose it's because this PR is adding one HEAD commit (changelog and version bump) but removing a commit way back in history, and for some reason it's getting confused? @cclauss the changes you're commenting on are on master so maybe they need a PR to address them? |
PR-URL: #1894 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
done, I suppose I'll push this out then! |
…1000" Ref: nodejs/node-addon-api#445 (comment) PR-URL: #1895 Reviewed-By: Rod Vagg <rod@vagg.org>
Included #1895 too and published as v5.0.4. Closing this PR but this branch, v5.x, will live on as our branch to maintain the v5.x series while we also move on to 6.0.0. |
This is the same as the #1892 but without the lone apparent semver-major: #1818. See thread in #1791 (comment) for more discussion on this approach.
Will take advice on what else this needs, I don't have any strong opinions other than "let's get this stuff out".
f753c167c5
] - gyp: decode stdout on Python 3 (cclauss) #189060a4083523
] - doc: update xcode install instructions to match Node's BUILDING (Nhan Khong) #188419dbc9ac32
] - deps: update tar to 4.4.12 (Matheus Marchini) #18895f3ed92181
] - bin: fix the usage instructions (Halit Ogunc) #1888aab118edf1
] - lib: adding keep-alive header to download requests (Milad Farazmand) #18631186e89326
] - lib: ignore non-critical os.userInfo() failures (Rod Vagg) #1835785e527c3d
] - doc: fix missing argument for setting python path (lagorsse) #1802a97615196c
] - gyp: rm semicolons (Python != JavaScript) (MattIPv4) #185806019bac24
] - gyp: assorted typo fixes (XhmikosR) #18533f4972c1ca
] - gyp: use "is" when comparing to None (Vladyslav Burzakovskyy) #18601cb4708073
] - src,win: improve unmanaged handling (Peter Sabath) #18525553cd910e
] - gyp: improve Windows+Cygwin compatibility (Jose Quijada) #18178bcb1fbb43
] - gyp: Python 3 Windows fixes (João Reis) #18432e24d0a326
] - test: accept Python 3 in test-find-python.js (João Reis) #18431267b4dc1c
] - build: add test run Python 3.7 on macOS (Christian Clauss) #1843da1b031aa3
] - build: import StringIO on Python 2 and Python 3 (Christian Clauss) #1836fa0ed4aa42
] - build: more Python 3 compat, replace compile with ast (cclauss) #182018d5c7c9d0
] - win,src: update win_delay_load_hook.cc to work with /clr (Ivan Petrovic) #1819