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 gitbook version #235

Merged
merged 2 commits into from
Sep 17, 2020
Merged

Conversation

zeusongit
Copy link
Contributor

Updating only for en for staging to test

Node: v10.16.3
Calibre: v4.23.0

FYI
@alfredo-pozo

@alfredo-pozo
Copy link
Contributor

alfredo-pozo commented Sep 11, 2020

@zeusongit @QilongTang On the container tests with these updates:

  https://nodejs.org/dist/v10.16.3/node-v10.16.3-x64.msi
  https://download.calibre-ebook.com/4.23.0/calibre-64bit-4.23.0.msi

I'm getting this error on HTML-PDF generation:

`EbookError: Error during ebook generation: Conversion options changed from defaults:
pdf_default_font_size: 12.0
pdf_page_numbers: True
no_chapters_in_toc: True
level1_toc: u"descendant-or-self::[contains(concat(' ', normalize-space(@Class), ' '), ' book-chapter-1 ')]"
max_levels: 1.0
margin_left: 62.0
level3_toc: u"descendant-or-self::
[contains(concat(' ', normalize-space(@Class), ' '), ' book-chapter-3 ')]"
page_breaks_before: u'/'
margin_top: 56.0
pdf_mono_font_size: 12.0
publisher: u'GitBook'
pdf_header_template: u''
pdf_sans_family: u'Arial'
level2_toc: u"descendant-or-self::[contains(concat(' ', normalize-space(@Class), ' '), ' book-chapter-2 ')]"
breadth_first: True
margin_right: 62.0
chapter: u"descendant-or-self::
[contains(concat(' ', normalize-space(@Class), ' '), ' book-chapter ')]"
book_producer: u'GitBook'
paper_size: u'a4'
margin_bottom: 56.0
1% Converting input to HTML...
Python function terminated unexpectedly
DLL load failed: The specified module could not be found. (Error Code: 1)
Traceback (most recent call last):
File "site.py", line 114, in main
File "site.py", line 88, in run_entry_point
File "site-packages\calibre\ebooks\conversion\cli.py", line 401, in main
File "site-packages\calibre\ebooks\conversion\plumber.py", line 1106, in run
File "site-packages\calibre\ebooks\conversion\plugins\pdf_output.py", line 151, in specialize_options
File "site.py", line 44, in load_module
ImportError: DLL load failed: The specified module could not be found.

The PDF generation failed`

@zeusongit
Copy link
Contributor Author

zeusongit commented Sep 11, 2020

@zeusongit @QilongTang On the container tests with these updates:

@alfredo-pozo Can you share the commands you used to build and generate the PDF?
Also, what version of gitbook are you using?

@alfredo-pozo
Copy link
Contributor

@zeusongit The version is this one:
Installing GitBook 3.2.3 gitbook@3.2.3 ..\..\Users\ContainerAdministrator\AppData\Local\Temp\tmp-25720ZzsadlhGO1K\node_modules\gitbook +-- escape-string-regexp@1.0.5 +-- escape-html@1.0.3
And these are the commands we use to install and build L20 and L21
https://github.com/DynamoDS/DynamoPrimer/blob/master/DockerUtilities/DockerCommands.ps1#L20

@zeusongit
Copy link
Contributor Author

After testing by @alfredo-pozo and me it turns out that on the docker image the compatible versions are:
NodeJS 10.16.3
Calibre 3.48

Which will be updated after this PR.

@QilongTang
Copy link
Contributor

After testing by @alfredo-pozo and me it turns out that on the docker image the compatible versions are:
NodeJS 10.16.3
Calibre 3.48

Which will be updated after this PR.

Good finding!

@QilongTang QilongTang merged commit 951249d into DynamoDS:Staging-branch Sep 17, 2020
QilongTang pushed a commit that referenced this pull request Oct 23, 2020
* Update gitbook version

* Updater other languages
QilongTang added a commit that referenced this pull request Oct 23, 2020
* Update gitbook version

* Updater other languages

Co-authored-by: Ashish Aggarwal <aggarwal.ash@husky.neu.edu>
QilongTang added a commit that referenced this pull request Oct 26, 2020
QilongTang added a commit that referenced this pull request Oct 26, 2020
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

3 participants