Skip to content

Upgrade node-pre-gyp to version 0.9.0#952

Merged
springmeyer merged 1 commit intoTryGhost:masterfrom
SebastianSchmidt:master
Mar 14, 2018
Merged

Upgrade node-pre-gyp to version 0.9.0#952
springmeyer merged 1 commit intoTryGhost:masterfrom
SebastianSchmidt:master

Conversation

@SebastianSchmidt
Copy link
Copy Markdown
Contributor

Closes #950

@SebastianSchmidt SebastianSchmidt changed the title Upgrade node-pre-gyp to version 0.9.0 [Work in progress] Upgrade node-pre-gyp to version 0.9.0 Mar 14, 2018
@SebastianSchmidt SebastianSchmidt changed the title [Work in progress] Upgrade node-pre-gyp to version 0.9.0 Upgrade node-pre-gyp to version 0.9.0 Mar 14, 2018
@SebastianSchmidt
Copy link
Copy Markdown
Contributor Author

SebastianSchmidt commented Mar 14, 2018

@springmeyer The OS X NODE_VERSION="5" COVERAGE=true PUBLISHABLE=false job failed. This error has already occurred before this pull request.

build/temp.macosx-10.12-intel-2.7/_openssl.c:493:10: fatal error: 'openssl/opensslv.h' file not found
#include <openssl/opensslv.h>
         ^
1 error generated.
error: Setup script exited with error: command 'cc' failed with exit status 1
The command "if [[ ${COVERAGE} == true ]]; then if [[ $(uname -s) == 'Linux' ]]; then PYTHONUSERBASE=$(pwd)/py-local pip install --user cpp-coveralls; else PYTHONUSERBASE=$(pwd)/py-local easy_install --user cpp-coveralls; fi; fi" failed and exited with 1 during .

@springmeyer
Copy link
Copy Markdown
Contributor

Yes, that is safe to ignore/unrelated to your PR. Thanks for providing this, merging now.

@springmeyer springmeyer merged commit c4659eb into TryGhost:master Mar 14, 2018
@ronkorving
Copy link
Copy Markdown

Awesome fellas :)
Hoping to see a release on NPM soon :)

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