na2hiro / Kifu-for-JS

JavaScriptで動く将棋の棋譜再生盤とそのブックマークレット `kifu-for-js`、JSON棋譜フォーマット(JKF)の定義とKIF, KI2, CSAからの変換ライブラリ `json-kifu-format`、将棋のモデル`shogi.js`を含むmonorepo
MIT License
112 stars 16 forks source link

Fix import error of shogi.js #33

Closed orangain closed 7 years ago

orangain commented 7 years ago

READMEの手順通りにbuildするとBrowserifyでエラーになるのを修正しました。最近のnpmは依存ライブラリをflatに置くのがエラーの原因だと思います。

json-kifu-format -> shogi.js の推移的な依存関係があるので、dependenciesにshogi.jsを追加しなくてもimportできますが、shogi.jsのクラスを使用しているので依存することを明確にしたほうがいいと判断して追加しました。

Before

$ ./node_modules/.bin/gulp build
[22:18:04] Using gulpfile ~/Kifu-for-JS/gulpfile.js
[22:18:04] Starting 'browserify'...
browserify starting
[22:18:04] Starting 'uglify'...
[22:18:04] Finished 'uglify' after 454 ms
browserify Error: Cannot find module 'json-kifu-format/node_modules/shogi.js' from '/Users/orange/Kifu-for-JS/src'

After

$ ./node_modules/.bin/gulp build
[22:19:53] Using gulpfile ~/Kifu-for-JS/gulpfile.js
[22:19:53] Starting 'browserify'...
browserify starting
[22:19:53] Starting 'uglify'...
[22:19:54] Finished 'uglify' after 176 ms
[22:20:03] Finished 'browserify' after 9.82 s
[22:20:03] Starting 'build'...
[22:20:03] Finished 'build' after 69 μs

Environment

$ node -v
v7.7.1
$ npm -v
4.1.2
na2hiro commented 7 years ago

ありがとうございます.