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

Check for zip dependency. #94

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@cirosantilli
Contributor

cirosantilli commented Mar 8, 2014

It is possible to install all other dependencies while missing the zip command.

If that happens, then you are not able to:

softcover build:epub

which gives error:

sh: 1: zip: not found

because EPUB is contained inside a zip.

Therefore, we must also check for zip as a dependency.

@mhartl

This comment has been minimized.

Contributor

mhartl commented May 10, 2014

I independently implemented a version of this a while back; sorry for not getting to the PR sooner.

@mhartl mhartl closed this May 10, 2014

@cirosantilli cirosantilli deleted the cirosantilli:check-zip branch May 10, 2014

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