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.

Sign up for GitHub

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

Jump to bottom

x64 ビルド用コンフィグ (sakura, sakura_lang_en_US) #84

Merged
merged 2 commits into from
Jun 10, 2018
Merged

x64 ビルド用コンフィグ (sakura, sakura_lang_en_US) #84

merged 2 commits into from
Jun 10, 2018

Conversation

kobake
Copy link
Member

@kobake kobake commented Jun 10, 2018 •
edited
Loading

x64 ビルド設定をこの PR で行います。コンパイル警告への対応は他 PR で分担して対応予定。

対応内容

  • ソリューションの SolutionConfigurationPlatforms に Debug|x64 と Release|x64 を追加
  • sakura, sakura_lang_en_US の2プロジェクトの Platform に x64 を追加
  • ソリューションと各プロジェクトの関連は以下の通り
    • Solution: Win32 Debug
      • sakura: Win32 Debug
      • sakura_lang_en_US: Win32 Debug
      • HeaderMake: Win32 Debug
      • MakefileMake: Win32 Debug
    • Solution: Win32 Release
      • sakura: Win32 Release
      • sakura_lang_en_US: Win32 Release
      • HeaderMake: Win32 Release
      • MakefileMake: Win32 Release
    • Solution: x64 Debug
      • sakura: x64 Debug
      • sakura_lang_en_US: x64 Debug
      • HeaderMake: Win32 Debug
      • MakefileMake: Win32 Debug
    • Solution: x64 Release
      • sakura: x64 Release
      • sakura_lang_en_US: x64 Release
      • HeaderMake: Win32 Release
      • MakefileMake: Win32 Release

※HeaderMake, MakefileMake については x64 不要と判断。Win32 でビルドする。

成果物一覧

Win32/Debug/
  sakura.exe
  sakura.pdb
  sakura_lang_en_US.dll
  sakura_lang_en_US.pdb
  
Win32/Release/
  sakura.exe
  sakura.pdb
  sakura_lang_en_US.dll
  sakura_lang_en_US.pdb
  
x64/Debug/
  sakura.exe
  sakura.pdb
  sakura_lang_en_US.dll
  sakura_lang_en_US.pdb
  
x64/Release/
  sakura.exe
  sakura.pdb
  sakura_lang_en_US.dll
  sakura_lang_en_US.pdb

@kobake kobake changed the title [WIP] x64 ビルド用コンフィグ (sakura, sakura_lang_en_US) x64 ビルド用コンフィグ (sakura, sakura_lang_en_US) Jun 10, 2018
@kobake kobake changed the title x64 ビルド用コンフィグ (sakura, sakura_lang_en_US) [WIP] x64 ビルド用コンフィグ (sakura, sakura_lang_en_US) Jun 10, 2018
@kobake kobake changed the title [WIP] x64 ビルド用コンフィグ (sakura, sakura_lang_en_US) x64 ビルド用コンフィグ (sakura, sakura_lang_en_US) Jun 10, 2018
@kobake
Copy link
Member Author

kobake commented Jun 10, 2018

整備完了しました。レビューお願いします。

@m-tmatma
Copy link
Member

修正は問題ないですが、また失敗してますね。
https://ci.appveyor.com/project/sakuraeditor/sakura/build/job/a1aos12ltont0nbj

@kobake kobake closed this Jun 10, 2018
@kobake kobake reopened this Jun 10, 2018
@kobake
Copy link
Member Author

kobake commented Jun 10, 2018

ぐぬぬ。4発ビルド走らせると失敗率上がりますね……

@berryzplus
Copy link
Contributor

ビルド回数が倍になってますからねぇ。
内容は問題なさそうと見ています。

berryzplus
berryzplus previously approved these changes Jun 10, 2018
View reviewed changes
Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

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

LGTMです。
1件コメント入れましたが警告出てないので対応しなくても良さそうです。

sakura/sakura.vcxproj Outdated
<AdditionalDependencies>comctl32.lib;Imm32.lib;mpr.lib;imagehlp.lib;Shlwapi.lib;%(AdditionalDependencies)</AdditionalDependencies>
<GenerateMapFile>true</GenerateMapFile>
<SubSystem>Windows</SubSystem>
<LargeAddressAware>true</LargeAddressAware>
Copy link
Contributor

Choose a reason for hiding this comment

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

警告出てないので無視されてるんだと思いますが外してもいいのでは。

Copy link
Member

@m-tmatma m-tmatma Jun 10, 2018 •
edited
Loading

Choose a reason for hiding this comment

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

https://msdn.microsoft.com/ja-jp/library/wz223b1z.aspx

によれば 64 ビット コンパイラでは、このオプションは既定で有効になっています。
とあります。

Win32 構成も x64 構成も Release では /LARGEADDRESSAWARE が有効になっていますが、
Debug 構成では指定されていませんね。

Copy link
Member

Choose a reason for hiding this comment

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

ちょっと前のコメント修正

Copy link
Member Author

@kobake kobake Jun 10, 2018 •
edited
Loading

Choose a reason for hiding this comment

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

今回はWin32構成をもとにx64構成を構築しましたので不本意に引き継がれている設定があったかもしれません。
VS2017で新規プロジェクト作成したときの規定値を調べてそれに合わせようと思います。
今出先なので帰宅後に。

Copy link
Contributor

Choose a reason for hiding this comment

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

によれば 64 ビット コンパイラでは、このオプションは既定で有効になっています。
とあります。
Win32 構成も x64 構成も Release では /LARGEADDRESSAWARE が有効になっていますが、
Debug 構成では指定されていませんね。

別のオプションと勘違いしていたようです。
オプション名がはっきりしないんですが、
32bitビルドで64bit移行に向けた警告を出させるようなものがあって
それが有効になっていると思って書きました。

Copy link
Member Author

Choose a reason for hiding this comment

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

VS2017 で新規プロジェクトを作成したときの値に合わせて x64 Release の LargeAddressAware 指定を外しました。指定しない場合は true として振る舞うので挙動は変わりませんが。

Win32 Release に LargeAddressAware が入っている理由は知りませんが、今回のスコープとは別なので一旦スルーしておきます。

Copy link
Member

Choose a reason for hiding this comment

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

#85 に登録しました。

Copy link
Member

Choose a reason for hiding this comment

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

オプション名がはっきりしないんですが、
32bitビルドで64bit移行に向けた警告を出させるようなものがあって
それが有効になっていると思って書きました。

/Wp64 は廃止されたそうです。
https://msdn.microsoft.com/ja-jp/library/yt4xw8fh.aspx

@kobake kobake closed this Jun 10, 2018
@kobake kobake reopened this Jun 10, 2018
@kobake
Copy link
Member Author

kobake commented Jun 10, 2018

ビルド再試行します

@berryzplus
Copy link
Contributor

1ビルド2分かかるのに4本目で失敗とか...orz

@kobake kobake closed this Jun 10, 2018
@kobake kobake reopened this Jun 10, 2018
@kobake
Copy link
Member Author

kobake commented Jun 10, 2018

再トライ！

@kobake
Copy link
Member Author

kobake commented Jun 10, 2018

やっとビルド通りました(´д｀)

@m-tmatma m-tmatma merged commit 0159e92 into sakura-editor:x64 Jun 10, 2018
@m-tmatma m-tmatma mentioned this pull request Jun 10, 2018
Closed
@m-tmatma m-tmatma added this to the next release milestone Jun 23, 2018
@kobake kobake added the x64 x64 対応 label Aug 12, 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
@m-tmatma
Merge pull request sakura-editor#84 from kobake/x64-configuration
44aff59 
x64 ビルド用コンフィグ (sakura, sakura_lang_en_US)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m-tmatma m-tmatma m-tmatma approved these changes

@berryzplus berryzplus berryzplus left review comments

Assignees
No one assigned
Labels
CI appveyor など CI 関連 【ChangeLog除外】 x64 x64 対応
Projects
None yet
Milestone
v2.4.0
Development

Successfully merging this pull request may close these issues.
4 participants
@kobake @m-tmatma @berryzplus @KENCHjp