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

Update README.md #46

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@mhulse

mhulse commented Mar 27, 2016

Updated installation instructions for gifsicle (motived by https://gist.github.com/SlexAxton/4989674#gistcomment-1476312).

The example from comment above uses wget. I prefer to use curl of wget. I don't have wget installed on my system. Not sure if you want to have instructions for both (isn't curl ubiquitous?).

Update README.md
Updated installation instructions for gifsicle (motived by https://gist.github.com/SlexAxton/4989674#gistcomment-1476312).

The example from comment above uses wget. I prefer to use curl of wget. I don't have wget installed on my system. Not sure if you want to have instructions for both (isn't curl ubiquitous?).
@vvo

This comment has been minimized.

Show comment
Hide comment
@vvo

vvo Mar 31, 2016

Owner

Hi mhulse, those instructions are only for OSX. I would rather link to the INSTALL file of the giflossy repository: https://github.com/pornel/giflossy/blob/master/INSTALL

Those instructions were not sufficient?

Owner

vvo commented Mar 31, 2016

Hi mhulse, those instructions are only for OSX. I would rather link to the INSTALL file of the giflossy repository: https://github.com/pornel/giflossy/blob/master/INSTALL

Those instructions were not sufficient?

@mhulse

This comment has been minimized.

Show comment
Hide comment
@mhulse

mhulse Apr 1, 2016

Those instructions were not sufficient?

Ah, my bad! I just saw this comment over here:

could you please add this to the gifify repo as a PR? very valuable

I should have looked to see if an installation script was already accounted for. I just assumed there was still a need for something as your comment was from Feb.

Since I didn't use that install script I'm not sure I'd be the best person to write a README sentence referring folks to it's location and usage.

mhulse commented Apr 1, 2016

Those instructions were not sufficient?

Ah, my bad! I just saw this comment over here:

could you please add this to the gifify repo as a PR? very valuable

I should have looked to see if an installation script was already accounted for. I just assumed there was still a need for something as your comment was from Feb.

Since I didn't use that install script I'm not sure I'd be the best person to write a README sentence referring folks to it's location and usage.

@mhulse mhulse closed this Apr 1, 2016

@mhulse

This comment has been minimized.

Show comment
Hide comment
@mhulse

mhulse Apr 1, 2016

For prosperity's sake:

curl -J -L https://github.com/pornel/giflossy/releases/download/lossy%2F1.82.1/gifsicle-1.82.1-lossy.zip  | tar -xzv --strip-components 1 --include=mac/gifsicle - -C /usr/local/bin/

mhulse commented Apr 1, 2016

For prosperity's sake:

curl -J -L https://github.com/pornel/giflossy/releases/download/lossy%2F1.82.1/gifsicle-1.82.1-lossy.zip  | tar -xzv --strip-components 1 --include=mac/gifsicle - -C /usr/local/bin/

@mhulse mhulse deleted the mhulse:patch-1 branch Apr 1, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment