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

change giflossy link #16

Merged
merged 1 commit into from
Feb 1, 2015
Merged

change giflossy link #16

merged 1 commit into from
Feb 1, 2015

Conversation

zwhitchcox
Copy link
Contributor

In reference to #15

vvo added a commit that referenced this pull request Feb 1, 2015
@vvo vvo merged commit 15e188b into vvo:master Feb 1, 2015
@vvo
Copy link
Owner

vvo commented Feb 1, 2015

I will link to generic releases page so that we are not out of date

@vvo
Copy link
Owner

vvo commented Feb 1, 2015

Thanks

@zwhitchcox
Copy link
Contributor Author

Glad to help :)

Just curious, but have you thought at all about just including the binaries in your project? That's what I did when I used this module

I've never really seen anyone do that, and I was just wondering if there was a reason.

@vvo
Copy link
Owner

vvo commented Feb 1, 2015

I think I would have to compile the requirements at install time, THought about it but seems error prone.

I cannot put binaries inside the project they would be not compatible between targets (osx, linux). you have to compile them

@zwhitchcox
Copy link
Contributor Author

Ahh ok

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