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

Vivliostyle-viewwerとPDF出力のためのdocker環境 #1

Open
wants to merge 5 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@at-grandpa

at-grandpa commented Aug 20, 2018

以下のことをできるようにしました。make hoge コマンドはリポジトリルートで実行する想定です。

  • make setup
    • 環境構築
  • make css/pdf
    • articles/book.pdf を出力
  • make css/browser
    • デフォルトブラウザにて Vivliostyle-viewer を使用しての確認
  • circleciでPDFの生成

やっていることは以下です。

  • make setup
    • dockerコンテナの build & up
    • review環境のsetup
      • 既存の setup.sh を叩く
    • vivliostyle環境のsetup
      • vivliostyleのzipをダウンロードして解凍
  • make css/pdf
    • htmlのbuild
    • httpサーバを起動して book.html のserve
    • vivliostyleのサーバを起動
    • puppeteer を用いて vivliostyle サーバ経由で book.html をpdf出力
  • make css/browser
    • htmlのbuild
    • httpサーバを起動して book.html のserve
    • vivliostyleのサーバを起動
    • ブラウザで Vivliostyle viewer にアクセス
  • circleci
    • サーバセットアップからpuppeteerでpdf出力

詳しくはPR内の README.md を参照してください。


nodejsやnpmにあまり詳しくないため、とりあえず動くものを作成しました。「こうしたほうが良い」などの指摘は大変ありがたいので、どしどしレビューをお願いします。

@vvakame vvakame self-assigned this Aug 20, 2018

@vvakame

This comment has been minimized.

Show comment
Hide comment
@vvakame

vvakame Aug 20, 2018

Owner

ありがたや〜〜 すみませんが内容を詳しく見られるのはちょっと後になりそうです…!
暇が出来たら見ます!
pull requestが来て大変うれしいです!

Owner

vvakame commented Aug 20, 2018

ありがたや〜〜 すみませんが内容を詳しく見られるのはちょっと後になりそうです…!
暇が出来たら見ます!
pull requestが来て大変うれしいです!

@at-grandpa

This comment has been minimized.

Show comment
Hide comment
@at-grandpa

at-grandpa Aug 20, 2018

@vvakame

いえいえー!おそらく今はめちゃくちゃ忙しい時期ですよね ...!
お手すきで良いので、お気になさらず!

at-grandpa commented Aug 20, 2018

@vvakame

いえいえー!おそらく今はめちゃくちゃ忙しい時期ですよね ...!
お手すきで良いので、お気になさらず!

kbaba1001 and others added some commits Sep 1, 2018

Merge pull request #2 from kbaba1001/fix_on_linux
linux(debian)環境で動かなかった部分を修正しました
var titleList: HTMLElement[] = [];
if (children != undefined) {
titleList = Array.from(children.querySelectorAll("li"));
}

This comment has been minimized.

@at-grandpa

at-grandpa Sep 14, 2018

この修正が無い場合に、例えば catalog.yml から syntax-example.re をコメントアウトすると以下のエラーになります。

image

原因は main.ts の160行目の el.children[i + 1] の要素が存在しない場合に undefined になってしまうからでした。このため、暗黙の条件として「最後の章は必ず節なければならない」となっていました。

それを改修しました。一度確認お願いいたします。

@at-grandpa

at-grandpa Sep 14, 2018

この修正が無い場合に、例えば catalog.yml から syntax-example.re をコメントアウトすると以下のエラーになります。

image

原因は main.ts の160行目の el.children[i + 1] の要素が存在しない場合に undefined になってしまうからでした。このため、暗黙の条件として「最後の章は必ず節なければならない」となっていました。

それを改修しました。一度確認お願いいたします。

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