Skip to content
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

win,build: forward release_urlbase to configure #8430

Conversation

joaocgreis
Copy link
Member

Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

Windows, Build.

Description of change

The RELEASE_URLBASE environment variable is used in releases as a prefix for links in the process.release object (#2154). The Makefile picks it and forwards it to configure, but vcbuild.bat did not. Hence, in Windows, Node releases have a correct process.release because it uses the default URL, but nightlies, RCs and so on do not, breaking node-gyp. This enables native modules to be built with such versions of Node.

cc @nodejs/platform-windows

The RELEASE_URLBASE environment variable is used in releases as
a prefix for links in the process.release object. The Makefile picks
it and forwards it to configure, but vcbuild.bat did not. Hence, in
Windows, Node releases have a correct process.release because it uses
the default URL, but nightlies, RCs and so on do not, breaking
node-gyp. This enables native modules to be built with such versions
of Node.
@joaocgreis joaocgreis added windows Issues and PRs related to the Windows platform. build Issues and PRs related to build files or the CI. labels Sep 7, 2016
@joaocgreis
Copy link
Member Author

@jbergstroem
Copy link
Member

LGTM

@orangemocha
Copy link
Contributor

Nothing in CI tests, but just to be extra safe: https://ci.nodejs.org/job/node-test-commit/4973/

The arm failure is certainly unrelated.

orangemocha pushed a commit that referenced this pull request Sep 9, 2016
The RELEASE_URLBASE environment variable is used in releases as
a prefix for links in the process.release object. The Makefile picks
it and forwards it to configure, but vcbuild.bat did not. Hence, in
Windows, Node releases have a correct process.release because it uses
the default URL, but nightlies, RCs and so on do not, breaking
node-gyp. This enables native modules to be built with such versions
of Node.

PR-URL: #8430
Reviewed-By: jbergstroem - Johan Bergström <[email protected]>
@orangemocha
Copy link
Contributor

Landed in 137b7cb (@joaocgreis is off today, I am landing this on his behalf so that this can make the v7.x cut on Monday).

@orangemocha orangemocha closed this Sep 9, 2016
Fishrock123 pushed a commit that referenced this pull request Sep 14, 2016
The RELEASE_URLBASE environment variable is used in releases as
a prefix for links in the process.release object. The Makefile picks
it and forwards it to configure, but vcbuild.bat did not. Hence, in
Windows, Node releases have a correct process.release because it uses
the default URL, but nightlies, RCs and so on do not, breaking
node-gyp. This enables native modules to be built with such versions
of Node.

PR-URL: #8430
Reviewed-By: jbergstroem - Johan Bergström <[email protected]>
joaocgreis added a commit to JaneaSystems/node that referenced this pull request Sep 14, 2016
The RELEASE_URLBASE environment variable is used in releases as
a prefix for links in the process.release object. The Makefile picks
it and forwards it to configure, but vcbuild.bat did not. Hence, in
Windows, Node releases have a correct process.release because it uses
the default URL, but nightlies, RCs and so on do not, breaking
node-gyp. This enables native modules to be built with such versions
of Node.

PR-URL: nodejs#8430
Reviewed-By: jbergstroem - Johan Bergström <[email protected]>
@MylesBorins
Copy link
Contributor

@joaocgreis should this be backported?

@joaocgreis
Copy link
Member Author

@thealphanerd yes, thanks!

MylesBorins pushed a commit that referenced this pull request Oct 11, 2016
The RELEASE_URLBASE environment variable is used in releases as
a prefix for links in the process.release object. The Makefile picks
it and forwards it to configure, but vcbuild.bat did not. Hence, in
Windows, Node releases have a correct process.release because it uses
the default URL, but nightlies, RCs and so on do not, breaking
node-gyp. This enables native modules to be built with such versions
of Node.

PR-URL: #8430
Reviewed-By: jbergstroem - Johan Bergström <[email protected]>
rvagg pushed a commit that referenced this pull request Oct 18, 2016
The RELEASE_URLBASE environment variable is used in releases as
a prefix for links in the process.release object. The Makefile picks
it and forwards it to configure, but vcbuild.bat did not. Hence, in
Windows, Node releases have a correct process.release because it uses
the default URL, but nightlies, RCs and so on do not, breaking
node-gyp. This enables native modules to be built with such versions
of Node.

PR-URL: #8430
Reviewed-By: jbergstroem - Johan Bergström <[email protected]>
MylesBorins pushed a commit that referenced this pull request Oct 26, 2016
The RELEASE_URLBASE environment variable is used in releases as
a prefix for links in the process.release object. The Makefile picks
it and forwards it to configure, but vcbuild.bat did not. Hence, in
Windows, Node releases have a correct process.release because it uses
the default URL, but nightlies, RCs and so on do not, breaking
node-gyp. This enables native modules to be built with such versions
of Node.

PR-URL: #8430
Reviewed-By: jbergstroem - Johan Bergström <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Oct 26, 2016
joaocgreis added a commit to joaocgreis/node-chakracore that referenced this pull request Nov 16, 2016
The RELEASE_URLBASE environment variable is used in releases as
a prefix for links in the process.release object. The Makefile picks
it and forwards it to configure, but vcbuild.bat did not. Hence, in
Windows, Node releases have a correct process.release because it uses
the default URL, but nightlies, RCs and so on do not, breaking
node-gyp. This enables native modules to be built with such versions
of Node.

PR-URL: nodejs/node#8430
Reviewed-By: jbergstroem - Johan Bergström <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues and PRs related to build files or the CI. windows Issues and PRs related to the Windows platform.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants