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

ビルド要件の精査と記述 #6

Closed
kobake opened this issue May 21, 2018 · 9 comments
Closed

ビルド要件の精査と記述 #6

kobake opened this issue May 21, 2018 · 9 comments
Labels
document ドキュメント management 運営に関する話題 【ChangeLog除外】
Milestone

Comments

@kobake
Copy link
Member

kobake commented May 21, 2018

Visual Studio Community 2017 インストール時に Windows SDK を入れておかないとビルド通らない(当然といえば当然だけど README に明記はしたほうが良い情報)。

他、諸々ビルド要件について精査して README に書いておく。

参考事例

https://twitter.com/arigayas/status/998445467418574848
この例では Windows SDK 8.1 で通っているけど、このあたりのバージョン挙動についても調べておきたい。

@arigayas
Copy link

Twitterの画像を見に行くのが手間なので貼付けます。

以前、他のオープンソフトプロジェクトをビルドする時に
「Windows XP Support for C++」が必要ですと
教えてもらったので、このスクリーンショットだと
「C++に関するWindows XP サポート」にチェックが入ってます。

vs2017_

@kobake
Copy link
Member Author

kobake commented May 21, 2018

なるほど!追加情報ありがとうございます!
けっこう考えるところ多いですね。

Win10でビルドしたexeがWinXPで動くかどうかとか、網羅的にやろうとするとなかなか大変そう…。
地道に情報集めていきます…!!

※場合によってはサポートを切り捨てるOSも今後出てくると思います(そのために古いバージョンの実行ファイルの配布は残しておく)

@arigayas
Copy link

Visual Studioがインストールされていないまっさらな環境で
ビルド環境構築を試さないと他に必要なものがあるかどうかわからないですね。

しかし私の手持ちの環境では、まっさらな環境はもう用意できません

@kobake
Copy link
Member Author

kobake commented May 21, 2018

大丈夫です!基本的に報告のあったものから順次対応という感じで考えています!
いろんな人からのケース報告を待つことにします!

動作の再現性>>>>>>ビルドの再現性

ってことで、どちらかというとビルド環境よりも動作環境の検証を優先して考えているので、実は本 Issue の対応優先度は低めです。でも忘れないように Issue 化してます(ちょっとこの Issue の粒度は大きすぎるので、後々分割予定)。

@kobake
Copy link
Member Author

kobake commented May 22, 2018

今のところいただいた情報をもとに README に情報を追記しました。
このあたりは今後ビルドでコケる人からの報告をいただくたびに Issue 化して随時 README に追記するスタンスで行こうと思います。

@kobake kobake closed this as completed May 22, 2018
@m-tmatma m-tmatma added management 運営に関する話題 【ChangeLog除外】 document ドキュメント labels Jun 11, 2018
@m-tmatma m-tmatma added this to the next release milestone Aug 19, 2018
@berryzplus
Copy link
Contributor

いまさらですが、追加情報です。

現状では 「Windows 8.1 SDK と UCRT SDK」も必須のようです。
MsBuildの PlatformSDK を探しに行く機構が SDK バージョンを要求していて、
何も指定しない場合 8.1 を入力したのと同じ動きをするようです。

#6 (comment)
このスクショではチェックが入ってますね・・・

最新版 vs2017 をクリーンインストールする機会があって、ビルドできなくて地味に焦りました。

@m-tmatma m-tmatma reopened this Sep 16, 2018
@m-tmatma
Copy link
Member

現状では 「Windows 8.1 SDK と UCRT SDK」も必須のようです。
MsBuildの PlatformSDK を探しに行く機構が SDK バージョンを要求していて、
何も指定しない場合 8.1 を入力したのと同じ動きをするようです。

PR 可能だったらお願いします。

@ds14050 ds14050 added management 運営に関する話題 【ChangeLog除外】 document ドキュメント labels Sep 18, 2018
beru added a commit that referenced this issue Oct 5, 2018
Visual Studio Install options を修正(#6 の反映など)
@m-tmatma
Copy link
Member

m-tmatma commented Nov 8, 2018

これは #518 の適用で完了ですか?

@berryzplus
Copy link
Contributor

yesです。閉じておきますね。

HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this issue Jun 11, 2019
Visual Studio Install options を修正(sakura-editor#6 の反映など)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document ドキュメント management 運営に関する話題 【ChangeLog除外】
Projects
None yet
Development

No branches or pull requests

5 participants