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

Installer のx64 対応 #180

Merged
merged 6 commits into from
Jun 27, 2018
Merged

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Jun 24, 2018

#131: Installer のx64 対応

対応内容

  • x64 で x64 のネイティブ用のディレクトリにインストールされるようにする
  • x86 と x64 でインストーラのファイル名を変える
  • x86 と x64 で共通部分を二重管理しなくてもいいように共通部分を別ファイルに抽出し、インクルートで個別部分を別ファイルで管理する。
  • x86 と x64 で別ファイルで iss ファイルを指定できるように bat ファイルを修正する。

変更内容

  • sakura.iss を sakura-common.iss に変更
  • sakura-Win32.iss と sakura-x64.iss を 追加
  • sakura-Win32.iss と sakura-x64.iss で sakura-common.iss をインクルード
  • MyArchitecture のマクロを sakura-Win32.iss と sakura-x64.iss でそれぞれ定義
  • sakura-common.iss で MyArchitecture が "x86" 以外の場合に ArchitecturesInstallIn64BitMode, ArchitecturesAllowed を有効にする
  • OutputBaseFilename の定義でMyArchitecture のマクロ定義を使って x86 版と x64 版でファイル名を変える

@m-tmatma m-tmatma added the installer installer 関連 label Jun 24, 2018
@m-tmatma m-tmatma added this to the next release milestone Jun 24, 2018
@m-tmatma m-tmatma added the x64 x64 対応 label Jun 24, 2018
@m-tmatma
Copy link
Member Author

@m-tmatma
Copy link
Member Author

#131 (comment)

には別件で対応したほうがいいかと思ったが、
本件で対応したほうがレビューしやすいし、
動作確認も二度手間になるのでこのPR で対応しました。

@m-tmatma
Copy link
Member Author

https://ci.appveyor.com/project/m-tmatma/sakura/build/1.0.200

  • x86, x64 でインストール、アンインストールができること
  • インストーラのファイル名にそれぞれ x86, x64 がつくこと
  • x64 版で C:\Program Files\sakura にインストールされること

を確認しました。

@m-tmatma
Copy link
Member Author

Rebase してひとつのコミットにまとめた方が良ければそうします。

@m-tmatma
Copy link
Member Author

なお masterをベースに作業したので、テストするにはx64 ブランチのマージが必要です

@kobake
Copy link
Member

kobake commented Jun 26, 2018

なお masterをベースに作業したので、テストするにはx64 ブランチのマージが必要です

master に x64 入ったので、この PR のブランチを master をもとに rebase いただきたいです。

@m-tmatma
Copy link
Member Author

テスト用にこのブランチから別のブランチを作ったので、rebase ではなくマージで対応しようと思います。

@kobake
Copy link
Member

kobake commented Jun 27, 2018

テスト用にこのブランチから別のブランチを作ったので、rebase ではなくマージで対応しようと思います。

マージコミットをこの PR に積むという理解で合ってますか?

@m-tmatma
Copy link
Member Author

m-tmatma commented Jun 27, 2018

マージコミットをこの PR に積むという理解で合ってますか?

2356e1e で master をマージしました。

以下を確認しました。
appveyor から Win32 Release, x64 Release 版のインストーラをダウンロードする

  1. 既存のインストールしている sakura (Win32 版) をアンインストールする
  2. ログオフする
  3. x64 Release 版をインストールする (インストール時のチェックはすべて入れる)
  4. x64 Release 版のインストールに成功することを確認する
  5. C:\Program Files\sakura 以下にインストールされていることを確認する
  6. sakura.exe を起動してバージョンを確認して64bit 版であることを確認する
  7. sakura を全終了する。
  8. x64 Release 版をアンインストーする。
  9. ログオフする
  10. Win32 Release 版をインストールする (インストール時のチェックはすべて入れる)
  11. Win32 Release 版のインストールに成功することを確認する
  12. C:\Program Files (x86)\sakura 以下にインストールされていることを確認する
  13. sakura.exe を起動してバージョンを確認して32bit 版であることを確認する
  14. sakura を全終了する。
  15. Win32 Release 版をアンインストーする。
  16. ログオフする
  17. Win32 Release 版をインストールする (インストール時のチェックはすべて入れる)
  18. Win32 Release 版のインストールに成功することを確認する
  19. sakura.exe を起動してバージョンを確認して32bit 版であることを確認する

@m-tmatma
Copy link
Member Author

↑ 17 が抜けていたので手順を追加

@m-tmatma
Copy link
Member Author

レビューよろしくお願いします。

@kobake
Copy link
Member

kobake commented Jun 27, 2018

動作問題なさそうです。

32bit

32bit_1

32bit_2

64bit

64bit_1

64bit_2

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 36f9951 into sakura-editor:master Jun 27, 2018
@m-tmatma m-tmatma deleted the feature/x64-installer branch June 28, 2018 13:58
@ds14050 ds14050 added x64 x64 対応 installer installer 関連 labels Sep 18, 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
installer installer 関連 x64 x64 対応
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants