Skip to content
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

sakura.iss を UTF8 に変更 #170

Merged
merged 1 commit into from
Jun 23, 2018

Conversation

m-tmatma
Copy link
Member

#132: sakura.iss を UTF8 に変更

sakura.iss の文字コード変換前後で(ローカルビルドの)インストーラの
バイナリが完全一致しているのを確認した。

@m-tmatma m-tmatma added installer installer 関連 refactoring リファクタリング 【ChangeLog除外】 labels Jun 23, 2018
@m-tmatma m-tmatma added this to the next release milestone Jun 23, 2018
@kobake
Copy link
Member

kobake commented Jun 23, 2018

sakura.iss 自体の文字コード以外のファイル内容の一致も確認。

char

@kobake
Copy link
Member

kobake commented Jun 23, 2018

#132: sakura.iss を UTF8 に変更

sakura.iss の文字コード変換前後で(ローカルビルドの)インストーラの
バイナリが完全一致しているのを確認した。

インストーラのバイナリって完全一致するものです?

以下は AppVeyor のバイナリを比較してみた結果です。

diff

タイムスタンプとか何かしらの動的なものが埋め込まれる都合で、このように diff が出るのは仕方ないと思っていたのですが。

@kobake
Copy link
Member

kobake commented Jun 23, 2018

バイナリ差分はさておきインストーラ実行時の日本語文言は正しく表示されているようなので LGTM です。

installer-char

Copy link
Member

@kobake kobake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ファイル差分および動作内容ともに問題なしです!

@kobake kobake merged commit 4ea1b49 into sakura-editor:master Jun 23, 2018
@m-tmatma
Copy link
Member Author

インストーラのバイナリって完全一致するものです?

appveyor のビルドでは EXE 等のリビルドが走るし、GitHash が異なるので違いますが
コミット前の状態で EXE や CHM だけをローカルビルドした状態でEXE などのリビルドを
せずにインストーラだけを UTF8 変換前後で、ビルドして比較した結果です。
(これでもタイムスタンプが違うことを予想しましたが、一致してました)

@m-tmatma m-tmatma deleted the feature/sakura-iss-utf8 branch June 23, 2018 12:48
@kobake
Copy link
Member

kobake commented Jun 23, 2018

なるほどど。また新しい知識が増えました。ありがとうございます!

@ds14050 ds14050 added installer installer 関連 refactoring リファクタリング 【ChangeLog除外】 labels Sep 18, 2018
@KENCHjp KENCHjp added the CI appveyor など CI 関連 【ChangeLog除外】 label Dec 5, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI appveyor など CI 関連 【ChangeLog除外】 installer installer 関連 refactoring リファクタリング 【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants