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

Use of node-pre-gyp (latest) #257

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Use of node-pre-gyp (latest) #257

wants to merge 12 commits into from

Conversation

seanmcilvenna
Copy link

No description provided.

@seanmcilvenna
Copy link
Author

Ok... I pulled joeferner:master into lantanagroup:master
Travis is reporting some issues, not sure what they are though...
I forgot to rebase when I did the pull and I'm not good enough with git to know how to squash after-the-fact... Can I give someone here collaborator access to my fork and have one of you squash it?

@jsdevel
Copy link
Collaborator

jsdevel commented Jan 21, 2016

While this is a novel idea, the build isn't passing and I don't believe we want to reference binaries that are hosted in s3. If there is a way to avoid s3 and have a passing build, then I feel we should merge this. @joeferner thoughts?

@joeferner
Copy link
Owner

I love the idea. Building the project seems to be the biggest hold up in the issues. The problem is updating the binaries and the number of platforms we would support. At a minimum Windows 64bit, OSX, and Linux.

On January 21, 2016 1:45:09 AM EST, Joseph Spencer [email protected] wrote:

While this is a novel idea, the build isn't passing and I don't believe
we want to reference binaries that are hosted in s3. If there is a way
to avoid s3 and have a passing build, then I feel we should merge this.
@joeferner thoughts?


Reply to this email directly or view it on GitHub:
#257 (comment)

Sent from my Android device with K-9 Mail. Please excuse my brevity.

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