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

Epub, Mobi, and PDF formats are not generated anymore #610

Closed
ronaldborman opened this issue Oct 6, 2020 · 3 comments · Fixed by #613
Closed

Epub, Mobi, and PDF formats are not generated anymore #610

ronaldborman opened this issue Oct 6, 2020 · 3 comments · Fixed by #613

Comments

@ronaldborman
Copy link

During the last builds the step 'Run gitbook install' was not successful so the build failed and subsequent steps were skipped. The returned error is TypeError: cb.apply is not a function at FSReqCallback.oncomplete.

As a result the Epub, Mobi, and PDF formats are not generated anymore with Github Actions.

@jtcmedia
Copy link
Contributor

This appears to be an issue with gitbook-cli: GitbookIO/gitbook-cli#110.
It has a dependency on an older version of graceful-fs where this problem hasn't been resolved for Node 12. gitbook-cli is deprecated so this won't get fixed but judging from the comments, it looks like there is an effort to maintain a fork of it. I modified the build script in this repo to downgrade to Node 10 and built the books here.

@wilitp
Copy link

wilitp commented Dec 12, 2020

I made a Dockerfile that builds and image containing the output in /output/.

FROM node:10

RUN apt update
RUN apt install calibre-bin -y
RUN npm install -g gitbook-cli
WORKDIR /app/
COPY . .

RUN gitbook install
RUN mkdir /output

RUN gitbook mobi . /output/typescript-book.mobi`

Just make sure that the context for the image build is the root of the book's repo.

@geekdenz
Copy link

Built the book just now from master:

https://drive.google.com/drive/folders/1kBv2xzKssiQfrR6PED8rWPS2IFcoxGU6?usp=sharing

Hoping to make someone's life easier...

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 a pull request may close this issue.

4 participants