bregonig.dll を sakura.exe と同じ場所に自動配置 #329
Conversation
kobake
changed the title
There was a problem hiding this comment.
LGTMです。
一応ローカルでビルドして以下メッセージが出力されたのを見ました。
3>=======================
3>postBuild
3>=======================
3>PLATFORM=Win32
3>CONFIGURATION=Debug
3>Copy bregonig.dll to destination directory.
3> 1 file(s) copied.ビルドされた sakura.exe で ctrl + f して検索ダイアログで bregonig.dll が認識されているのも見ました。実際に正規表現検索もしてみましたが問題なさそうです。
|
|
||
| : ---- copy bregonig.dll ---- : | ||
| @echo Copy bregonig.dll to destination directory. | ||
| copy /B ..\installer\externals\bregonig\%PLATFORM%\bregonig.dll ..\%PLATFORM%\%CONFIGURATION%\bregonig.dll |
There was a problem hiding this comment.
末尾 ..\%PLATFORM%\%CONFIGURATION%\bregonig.dll ですが ..\%PLATFORM%\%CONFIGURATION%\ でもいけるはずです。そうすることのメリットはコピー対象ファイルが変わったとき、変更が1箇所で済む、です。まぁ、ファイル名変わらないでしょうけどｗ
感想なので変更は任意で。
There was a problem hiding this comment.
僕もここはどっちにしようか迷いましたがなんとなく明示的にファイル名書きました。こだわりはあんまりないです。
| @@ -118,7 +118,7 @@ | |||
| <TargetMachine>MachineX86</TargetMachine> | |||
| </Link> | |||
| <PostBuildEvent> | |||
| <Command>call postBuild.bat "$(TargetPath).manifest"</Command> | |||
| <Command>call postBuild.bat $(Platform) $(Configuration)</Command> | |||
There was a problem hiding this comment.
変更後の引数にダブルクォーテーションがないですが意図的でしょうか？
Configuration には 空白を含む名前 を付けたくなることもあるかも知れません。
・・・たぶんないですよね。外したのが意図的でなければ付けといたほうが無難な気がします。
There was a problem hiding this comment.
実は全部スペースは入らない想定で組んでます。スペース入ると .bat 側も実はうまく動かなくなります。運用として割り切りたいな～というお気持ち。
|
レビューありがとうございます。 |
bregonig.dll を sakura.exe と同じ場所に自動配置することにより 38行目のコピーで bregonig.dll が bron412.zip のものではなく、リポジトリに登録しているものが直接組み込まれるようになってしまう。 このため 38行目のコピーではワイルドカードでコピーせずに、ビルドした DLL を明示的にファイル名指定でコピーする。
|
appveyor 用のドキュメント更新します。 |
|
また #332 を投げています。 |
#329 によるデグレを修正 (bron412.zip の bregonig.dll が使われなくなる)
bregonig.dll を sakura.exe と同じ場所に自動配置
bregonig.dll を sakura.exe と同じ場所に自動配置することにより 38行目のコピーで bregonig.dll が bron412.zip のものではなく、リポジトリに登録しているものが直接組み込まれるようになってしまう。 このため 38行目のコピーではワイルドカードでコピーせずに、ビルドした DLL を明示的にファイル名指定でコピーする。
sakura-editor#329 によるデグレを修正 (bron412.zip の bregonig.dll が使われなくなる)
sakuraプロジェクトのビルド時に、自動的にビルド出力先に bregonig.dll がコピーされるようにする。
主目的
手動で bregonig.dll をコピーしてくる手間を省くことにより、以下を実現します。
注記事項
installer/externals/bregonig 内に bregonig.dll を直接配置しています。