Added Pre built binaries. #319
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…rt using node-pre-gyp Add fallback to buil to node-pre-gyp install command, in case remote binary download fails. Added travis automated binary build, to generate and pubslish the serialport packages. Publish binaries adding `[publish binary]` to a commit message or by adding a release tag. Add travis automated build for x86 32 bit binaries.[publish binary]
1. Created new appveyor.yml configuration file. 2. We are now packaging and publishing windows 32bit version of the binary.
|
@voodootikigod Let us know what you think, the only parts missing to fully automate this is having a osx-node-pre-gyp branch to generate the OSx binaries and setup node-serialport for appveyor CI. Once it is created we can merge our branch with the OSx travis.yml configuration. |
|
This is PR #1 of 3. We still need a |
This was referenced Apr 6, 2014
voodootikigod
pushed a commit
that referenced
this pull request
Apr 8, 2014
Added Pre built binaries.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We are now packaging and publishing binaries for:
3 Win 32bit.
There is an issue with compiling the windows 64bit version of the binary in appveyor, it seems some libraries are missing for serialport to compile, hopefully they will have them sorted out sonner rather than later.
Thanks to everyone for comments and help in:
#290
Specially to @springmeyer (since most of the code is based on his previous contributions) and to @BergWerkGIS since his code for appveyor.yml configuration and suggestions kickstarted the windows binaries packaging and publishing.