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

platform-dependent name of the node-webkit's executable #36

Merged
merged 3 commits into from Feb 13, 2014
Merged

platform-dependent name of the node-webkit's executable #36

merged 3 commits into from Feb 13, 2014

Conversation

Mithgol
Copy link
Contributor

@Mithgol Mithgol commented Feb 13, 2014

An attempt to run node-webkit on Linux fails inevitably (see, for example, this Travis CI log, line 1700).

This pull request provides a set of platform-dependent names.

@Mithgol
Copy link
Contributor Author

Mithgol commented Feb 13, 2014

Travis CI test of this change fails for some obscure reason.

As an additional test I pushed a commit to the source branch of TryGhost/node-sqlite3#252, using a tarball of the source branch of #36 as a source of node-pre-gyp.

The resulting Travis CI log (line 1700) shows that node-pre-gyp now calls nw on Linux.

springmeyer pushed a commit that referenced this pull request Feb 13, 2014
platform-dependent name of the node-webkit's executable
@springmeyer springmeyer merged commit 0152085 into mapbox:master Feb 13, 2014
springmeyer pushed a commit to TryGhost/node-sqlite3 that referenced this pull request Apr 9, 2014
A pull request (mapbox/node-pre-gyp#36) is opened to land a partial fix for node-pre-gyp's validation process.

This commit uses a source branch of that pull request.

After this commit, node-pre-gyp's validator is expected to launch `nw` instead of `node-webkit` on Travis CI's Linux.

A former ENOENT error is expected to be replaced by a node-webkit's error when the engine realises it's given a node-sqlite3 module's JavaScript instead of some HTML5 application.
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.

None yet

2 participants