appveyor への対応 (Visual Studio 2017)

[m-tmatma]

#10: appveyor への対応 (Visual Studio 2017)

• /source-charset:shift_jis を指定して、英語環境 (appveyor) 上でもコンパイルできるようにする
• appveyor.yml を追加

参考サイト

• https://docs.microsoft.com/ja-jp/cpp/build/reference/source-charset-set-source-character-set
• https://msdn.microsoft.com/ja-jp/library/dd317756.aspx
• http://teitoku-window.hatenablog.com/entry/2017/09/21/201750
• Support a different code page appveyor/ci#846

[kobake]

/source-charset:shift_jis を指定して、英語環境 (appveyor) 上でもコンパイルできるようにする

これについてですが、もういまどきソースコードの文字セットが SJIS ってのも古すぎるので、近いうちに全ソースコードを UTF-8 に一括変換しちゃうつもりです。ツールが誤認識しないように BOM も付けちゃう予定です。

このあたりの変更が衝突する可能性あります、ということを先に伝えておきます。

[m-tmatma]

これについてですが、もういまどきソースコードの文字セットが SJIS ってのも古すぎるので、
近いうちに全ソースコードを UTF-8 に一括変換しちゃうつもりです。ツールが誤認識しないように BOM も付けちゃう予定です。

一般的には私も最近のトレンドに沿ってソースコードを UTF-8 で保存するのがいいと思いますが、

sakura editor のソースコードの中では shift-jis を前提とした文字列定数が
いろんな箇所で使われています。

UTF-8 に変換したときに正しく動くのかを個別に十分に検討する必要があります。

十分な単体テストが整備されていれば、変更前と変更後でデグレが発生しないことを
容易に確認できますが、現状では単体テストは存在しないので、結構リスクが高い
と思います。

すくなくともすぐにできるものではないと思います。

[kobake]

そですね、リテラル定義のところ気をつけたほうが良さそうです。
先にテスト機構 (このPRによるビルド機構含め) 用意しておくのが良さそうですね。

[kobake]

動作確認できました、ご対応ありがとうございます！マージします！

AppVeyor 便利ですね～ 👍👍👍👍

動作確認情報

• https://github.com/kobake/sakura/commits/master
• https://ci.appveyor.com/project/kobake/sakura-qiaye/build/1.0.6

メモ

postBuild.bat の影響で AppVeyor ビルドログ内に変な文字化けメッセージが混じってしまう問題を先に直して master マージしました。

• VS2017 では postBuild.bat の処理内容は不要なので削除 VS2017 では postBuild.bat の処理内容は不要なので削除 #14

[kobake]

今後、sakura リポジトリの master ブランチは以下の AppVeyor プロジェクトでビルドされるようになります。
https://ci.appveyor.com/project/kobake/sakura

本PRマージ後のビルド結果
https://ci.appveyor.com/project/kobake/sakura/build/1.0.10

ちょっと URL 内に kobake って入ってるのカッコ悪いですが…。もうちょっと良い感じの置き方があればアドバイスいただけると助かります。

[m-tmatma]

ちょっと URL 内に kobake って入ってるのカッコ悪いですが…。
もうちょっと良い感じの置き方があればアドバイスいただけると助かります

#15 を作成しました。

[m-tmatma]

#43 の原因になっている可能性あり。
#96 で workaround を導入

• destination path 'xxx' already exists and is not an empty directory appveyor/ci#2433
• If 'restart-computer' cmdlet is in init stage, creating the clone folder may fail with error appveyor/ci#2435
• If 'restart-computer' cmdlet is in init stage, creating the clone folder may fail with error appveyor/ci#2435 (comment)