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

Bump gyp 19495aa #1183

Closed
wants to merge 12 commits into from
Closed

Bump gyp 19495aa #1183

wants to merge 12 commits into from

Commits on Jun 6, 2017

  1. Upgrade to tar v3

    Tar version 3 performs better and is more well tested than its
    predecessor.  npm will be using this in the near future, so there is no
    benefit in shipping a node-gyp that uses the slower and less reliable
    fstream-based tar.
    
    This drops support for node 0.x, and thus should be considered a
    breaking semver-major change.
    
    PR-URL: nodejs#1212
    Reviewed-By: Refael Ackermann <refack@gmail.com>
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
    isaacs authored and refack committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    5f924ce View commit details
    Browse the repository at this point in the history
  2. bump to v4.0.0

    * dropping support for node < 4
    * signal the CI not to test node < 4
    refack committed Jun 6, 2017
    4 Configuration menu
    Copy the full SHA
    75cfae2 View commit details
    Browse the repository at this point in the history

Commits on Jun 7, 2017

  1. configure: don't set ensure if tarball is set

    If you're providing a path to a header tarball to install, you probably
    want it to always be re-installed.
    
    PR-URL: nodejs#1220
    Fixes: nodejs#1216
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: Refael Ackermann <refack@gmail.com>
    gibfahn authored Jun 7, 2017
    Configuration menu
    Copy the full SHA
    c84a541 View commit details
    Browse the repository at this point in the history

Commits on Jun 8, 2017

  1. gyp: update xml string encoding conversion

    * test: build simple addon in path with non-ascii characters
    * test: add test-charmap.py
    
    PR-URL: nodejs#1203
    Reviewed-By: Refael Ackermann <refack@gmail.com>
    lc-soft authored and refack committed Jun 8, 2017
    Configuration menu
    Copy the full SHA
    d6139b5 View commit details
    Browse the repository at this point in the history

Commits on Jun 11, 2017

  1. doc: headerify the Install instructions

    Enable linking to the platform specific installation instructions
    
    PR-URL: nodejs#1225
    Reviewed-By: Refael Ackermann <refack@gmail.com>
    nschonni authored and refack committed Jun 11, 2017
    Configuration menu
    Copy the full SHA
    35e1dbb View commit details
    Browse the repository at this point in the history

Commits on Jun 14, 2017

  1. doc: update proposed DCO and CoC

    Lifted verbatim from
    https://github.com/nodejs/node/blob/master/CONTRIBUTING.md
    then `s/Node.js/node-gyp/`.
    
    PR-URL: nodejs#1229
    Reviewed-By: Anna Henningsen <anna@addaleax.net>
    Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
    mikeal authored and gibfahn committed Jun 14, 2017
    Configuration menu
    Copy the full SHA
    2e40fad View commit details
    Browse the repository at this point in the history

Commits on Jul 4, 2017

  1. doc: add github PR and Issue templates

    Give users reporting bugs a clearer idea of the info that will be
    helpful when reporting issues.
    
    PR-URL: nodejs#1228
    Refs: https://github.com/nodejs/node/tree/master/.github
    Reviewed-By: Refael Ackermann <refack@gmail.com>
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    gibfahn committed Jul 4, 2017
    Configuration menu
    Copy the full SHA
    7245415 View commit details
    Browse the repository at this point in the history

Commits on Aug 19, 2017

  1. gyp: bump gyp to 324dd166b7

    refack committed Aug 19, 2017
    Configuration menu
    Copy the full SHA
    c6d8d59 View commit details
    Browse the repository at this point in the history
  2. gyp: fix android generator (rebased)

    this is a rebase of
    a981ef8
    After bumping gyp to 19495aa
    
    Actually I just took the file as is, because of a discontinues
    patch floating.
    robertchiras authored and refack committed Aug 19, 2017
    Configuration menu
    Copy the full SHA
    07e0d94 View commit details
    Browse the repository at this point in the history
  3. gyp: inherit CC/CXX for CC/CXX.host

    Gyp defaults to gcc/g++ if CC.host/CXX.host is unset. This is not
    suitable for environments that only uses the clang toolchain.
    
    Since we already assume that the user will provide clang/clang++
    through CC/CXX, lean against it (then drop to gcc/g++).
    
    PR-URL: nodejs#908
    Refs: nodejs/node#6173
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
    jbergstroem authored and refack committed Aug 19, 2017
    Configuration menu
    Copy the full SHA
    742d4f9 View commit details
    Browse the repository at this point in the history
  4. gyp: improvements for android generator

    When generating Android.mk files for HOST binaries, we need to specify
    the LOCAL_CXX_STL to use when linking.
    Also, set the variable GYP_HOST_MULTILIB to 'first'. This is used as
    LOCAL_MULTILIB, and since we don't provide flags for both types of
    builds (32-bit and 64-bit), the build system will try to build both and
    fail.
    More details about LOCAL_MULTILIB can be found here:
    https://source.android.com/source/64-bit-builds.html
    
    Signed-off-by: Robert Chiras <robert.chiras@intel.com>
    PR-URL: nodejs#935
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    robertchiras authored and refack committed Aug 19, 2017
    Configuration menu
    Copy the full SHA
    1dd9ea8 View commit details
    Browse the repository at this point in the history
  5. gyp: Open the build file with universal-newlines mode

    To make sure platform specific newlines ('\r\n' or '\r') are converted
    to '\n' which otherwise will fail eval().
    
    This should handle multiple issues reported on syntax error reading
    binding.gyp (partial list):
    
    nodejs#979
    nodejs#199
    nospaceships/node-net-ping#24
    nospaceships/node-net-ping#21
    mathiask88/node-snap7#11
    node-hid/node-hid#28
    xdenser/node-firebird-libfbclient#24
    
    PR-URL: nodejs#1053
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    guymguym authored and refack committed Aug 19, 2017
    Configuration menu
    Copy the full SHA
    0c98bee View commit details
    Browse the repository at this point in the history