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

Update Gemfile (progit:5a2ed7b7) #139

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update Gemfile (progit:5a2ed7b7) #139

wants to merge 1 commit into from

Conversation

tnir
Copy link
Collaborator

@tnir tnir commented Jan 13, 2022

Signed-off-by: Takuya Noguchi <[email protected]>
@tnir tnir requested a review from jnavila January 13, 2022 11:56
@tnir tnir self-assigned this Jan 13, 2022
@tnir tnir added dependencies Pull requests that update a dependency file ruby Pull requests that update Ruby code labels Jan 13, 2022
@jnavila
Copy link
Member

jnavila commented Jan 13, 2022

TBH, I don't care about this Gemfile. It is overwritten by the compilation scripts from https://github.com/progit/progit2-pub which are downloaded by the CI tasks.

@tnir tnir requested a review from okuryu January 13, 2022 13:02
Copy link
Member

@jnavila jnavila left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is only for local compilation. For CI, another file is used.

Please let me know if it works for you with this set up.


gem 'asciidoctor-epub3', :git => 'https://github.com/asciidoctor/asciidoctor-epub3'
gem 'asciidoctor-pdf', '1.5.0.alpha.16'
gem 'asciidoctor-pdf-cjk', '~> 0.1.3'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unless it has changed lately, you have to keep asciidoctor-pdf-cjk in order to be able to build a PDF with CJK chars. This requirement pulls other requirements on asciidoctor-pdf and asciidoctor

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you try to compile a PDF book with these settings?

@jnavila
Copy link
Member

jnavila commented Jan 14, 2022

Can you check you can push to the repository?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file ruby Pull requests that update Ruby code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants