Skip to content

src: introduce process.release object

Rodrigo Muino Tomonari requested to merge github/fork/rvagg/process.release.master into master

Replaces #493 and looking for new feedback.

Requirements: provide URLs to node-gyp for downloading headers and other required files for building native addons. Decoupling it from a hardwired http://nodejs.org/dist/ gives us the ability to provide locations for nightlies, next-nightlies, rcs and any other kind of test builds we want to provide with a working node-gyp. This is also the path to stop patching node-gyp for io.js and into the future gives us the option to continue releasing as iojs (or whatever name we happened to want!) if that's what we decide.

This new PR simplifies from the original approach, requiring only --release-urlbase be passed to configure. It's optional but required for release builds (e.g. make binary RELEASE_URLBASE=https://iojs.org/download/nightly/). We would supply a string such as https://iojs.org/download/nightly/. This gets converted to process.release.sourceUrl, process.release.headersUrl and process.release.libUrl if windows.

A few minor whitespace fixes in Makefile too.

$ ./iojs -pe process.release
{ sourceUrl: 'https://iojs.org/download/nightly/v2.3.5-nightly2015071047e2c5c828/iojs-v2.3.5-nightly2015071047e2c5c828.tar.gz',
  headersUrl: 'https://iojs.org/download/nightly/v2.3.5-nightly2015071047e2c5c828/iojs-v2.3.5-nightly2015071047e2c5c828-headers.tar.gz' }

(on Windows you get a libUrl property that would point to something like 'https://iojs.org/download/nightly/v2.3.5-nightly2015071047e2c5c828/win-x64/iojs.lib')

Merge request reports