Package cygwin #26
Merged
Package cygwin #26
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
|
Does this also help with Windows: Upgrading / downgrading 'esy' always reinstalls cygwin layer? |
Good question. This doesn't directly fix that issue, but it does mean that For CI to get these improvements - we'll need to make sure the npm cache is specified in the travis/appveyor cache sections. |
bryphe
added a commit
that referenced
this pull request
Oct 23, 2018
* Create build-cygwin.js * Update appveyor.yml / package.json * Update build step to remove cache * Continue fixing up files * Temporarily tweak package name * v0.0.100 * Update appveyor / build scripts * Revert package.json * Fix up comments * Remove unused yarn install * Try fixing appveyor * Add space * Add defaults folder Bump version -> 0.2.24 Set version to 0.2.1
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This addresses #12 and #13 - we're taking a snapshot of Cygwin at build time, and including the
.cygwinfolder as part of our NPM package.The NPM package ends up being ~120MB, but low on dependencies - and installs much faster than the equivalent operation via the cygwin installer.
A few changes that were needed:
package.jsonNODE_OPTIONSto use a larger heap size, otherwise the publish step fails with an out-of-memory error./var/cachefolder, as cached packages are kept there, and we don't need those once they are unpacked - they'd just weigh the package down.