PDF-Translator icon indicating copy to clipboard operation
PDF-Translator copied to clipboard

pdf.js をnpmからインストールする

Open kazushisan opened this issue 4 years ago • 5 comments

pdf.jsライブラリのコードをリポジトリで管理するのではなく、npmからインストールすることを検討したほうがいいと思いました。

kazushisan avatar Jun 30 '20 13:06 kazushisan

ありがとうございます. ここで用いているpdf.jsのコードはnpmなどでインストールできるpdf.jsから少し修正を加えたものを用いています. そのため,単純にpdf.jsのライブラリをリポジトリから削除して,npmからインストールさせるようにすることはできないと思っています(あまり詳しくないため,他に何かいい方法があるかもしれません).

この問題に対処するとしたら,修正を加えた処理を別のファイルのコードで置き換えることだと思っており,この方針で進めていきたいと思います.

yoshinobc avatar Jun 30 '20 16:06 yoshinobc

@yoshinobc そうなんですね…

修整を加えたコードを分離するのが難しそうであれば patch-package を使う方法を取ることもできます!

今週は時間を確保できないのですが、それでもよければアサインしていただけませんか

kazushisan avatar Jun 30 '20 23:06 kazushisan

patch-package今調べましたが便利そうですね! これを使えば問題を解決できると思います. ありがとうございます.できるだけすぐに修正するようにします(直近は課題や研究を優先してしまうかもしれませんが...)

yoshinobc avatar Jul 01 '20 11:07 yoshinobc

@yoshinobc

今週は時間を確保できないのですが、それでもよければアサインしていただけませんか

せっかくなのでこちらで作業してプルリク投げてもいいですか??

kazushisan avatar Jul 01 '20 23:07 kazushisan

そうしていただけると助かります…

yoshinobc avatar Jul 04 '20 14:07 yoshinobc