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.exe と sakura_lang_en_US.dll の生成フォルダをソリューションフォルダをもとに決める #76

Closed
m-tmatma opened this issue Jun 9, 2018 · 12 comments

Comments

@m-tmatma
Copy link
Member

m-tmatma commented Jun 9, 2018

sakura.exe と sakura_lang_en_US.dll の生成フォルダはプロジェクトのフォルダをもとに決められるが、
ソリューションフォルダをもとに決める。

プロジェクトのフォルダをもとに成果物が生成されるので、sakura_lang_en_US.dll が別のフォルダに
作成されるので英語リソースのテストをするのに手間がかかる。

@m-tmatma
Copy link
Member Author

m-tmatma commented Jun 9, 2018

#72 の後に対応する

@berryzplus
Copy link
Contributor

berryzplus commented Jun 10, 2018

OutDirを変更するならIntDirもソリューション基準にしませんか?

appveyorには影響ありませんし、気分的なものなのでどちらでもいいですがこんな感じ。
$(SolutionDir)$(Configuration)\$(ProjectName).obj</IntDir>

あと、出力先にプロジェクト名+.dll(exe)で出したい場合、OutputFileは省略可能です。
行を削ってしまえば、今後メンテしなくてよくなるメリットがあります。

m-tmatma added a commit to m-tmatma/sakura that referenced this issue Jun 10, 2018
m-tmatma added a commit to m-tmatma/sakura that referenced this issue Jun 10, 2018
@kobake
Copy link
Member

kobake commented Jun 10, 2018

新規プロジェクトを作ったときのデフォルト設定が今の (Visual Studio Community 2017 の) 推奨なのかな、と思っています。

試しに新規プロジェクト (Windows Console Application) を作ってみましたが、以下のようになりました。

  • Output Directory: $(SolutionDir)$(Platform)\$(Configuration)\
  • Intermediate Directory: $(Platform)\$(Configuration)\
  • Output File: $(OutDir)$(TargetName)$(TargetExt)

@m-tmatma
Copy link
Member Author

あと、出力先にプロジェクト名+.dll(exe)で出したい場合、OutputFileは省略可能です。

sakura_lang_en_US はプロジェクト名 ≠ ファイル名なのでそこも変えたほうがいいと思います。
ただその場合別の PR、チケット にしたほうがいいと思います。

@m-tmatma
Copy link
Member Author

Output Directory: $(SolutionDir)$(Platform)$(Configuration)
Intermediate Directory: $(Platform)$(Configuration)
Output File: $(OutDir)$(TargetName)$(TargetExt)

そうですね。x64 対応するときに、$(Platform) が入っていると
x86 と x64 でフォルダ構成が対象になるので扱いやすいと思います。

Intermediate Directory はかぶらなければどこに作ってもいいと思います。

HeaderMake.exe と MakefileMake.exe の生成場所はソリューション直下ではなく
各ビルド構成毎にちがうフォルダにしたいと思います。

#24 で見直してもいいと思います。

@berryzplus
Copy link
Contributor

sakura_lang_en_US はプロジェクト名 ≠ ファイル名なのでそこも変えたほうがいいと思います。

ボケてました。

@kobake
Copy link
Member

kobake commented Jun 10, 2018

#80 でざっくり対応中です。
Configuration 名が今は Win32 になっているのですが、appveyor.yml に合わせて x86 に変更します?(とくにこだわりはないです)

@m-tmatma
Copy link
Member Author

sakura_lang_en_US はプロジェクト名 ≠ ファイル名なのでそこも変えたほうがいいと思います。

ボケてました。

ファイル名は sakura_lang.vcxproj です。

プロジェクト名は sakura_lang_en_US で
出力ファイル名は sakura_lang_en_US.dll なので

出力ファイル名をプロジェクト名+.dll(exe) とすることは現時点でも可能です。

@kobake
Copy link
Member

kobake commented Jun 10, 2018

あ、すみません PR 被ってたみたいです

@kobake
Copy link
Member

kobake commented Jun 10, 2018

Configuration 名が今は Win32 になっているのですが、appveyor.yml に合わせて x86 に変更します?(とくにこだわりはないです)

Platform 名の間違いでした

m-tmatma added a commit to m-tmatma/sakura that referenced this issue Jun 10, 2018
- 中間ファイル用のフォルダを $(Platform)\$(Configuration)\ に変更
- sakura と sakura_lang_en_US の出力先を $(SolutionDir)$(Platform)\$(Configuration)\ に変更
- Debug や sakura_lang となっているところを $(OutDir) と $(TargetName) に変更
- appveyor.yml を追従
@m-tmatma m-tmatma added this to the next release milestone Jun 10, 2018
@m-tmatma
Copy link
Member Author

#79 がマージされた。

@m-tmatma
Copy link
Member Author

sakura_lang_en_US はプロジェクト名 ≠ ファイル名なのでそこも変えたほうがいいと思います。

#82 で対応

HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this issue Jun 11, 2019
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this issue Jun 11, 2019
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this issue Jun 11, 2019
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this issue Jun 11, 2019
- 中間ファイル用のフォルダを $(Platform)\$(Configuration)\ に変更
- sakura と sakura_lang_en_US の出力先を $(SolutionDir)$(Platform)\$(Configuration)\ に変更
- Debug や sakura_lang となっているところを $(OutDir) と $(TargetName) に変更
- appveyor.yml を追従
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants