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

Visual Studio のインストールオプションの設定ファイルを追加する #1162

Merged
merged 6 commits into from
Jan 19, 2020

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Jan 18, 2020

PR の目的

Visual Studio 2019 では .vsconfig というファイルを sln と同じディレクトリに置いておくと
ソリューションのビルドに必要なコンポーネントが足りないと自動的にユーザーに通知して
クリック一つでインストールすることができる。

カテゴリ

  • ビルド手順

PR の背景

#518

サクラエディタをビルドしてみようと Visual Studio Community 2017 を
クリーンインストールしてみたところ、いくつか追加で必要だったので README.md を修正してみました。

とあり、README.md に必要なコンポーネントの説明を記載しているが、
足りない場合に自動的にユーザーに通知してワンタッチでインストールできたら便利なため。

PR のメリット

ユーザーが簡単に Sakura Editor のビルド環境を構築できる。

PR のデメリット (トレードオフとかあれば)

なし

PR の影響範囲

Visaul Studio のインストール

関連チケット

#518
#936
#6

残件

Visaul Studio 2017 での確認 (完了)

参考資料

@m-tmatma
Copy link
Member Author

https://docs.microsoft.com/ja-jp/visualstudio/install/import-export-installation-configurations?view=vs-2019#automatically-install-missing-components

のような感じで、インストールを求められるのでインストールを押すだけ。

vs2019

vs2019-2

@m-tmatma
Copy link
Member Author

m-tmatma commented Jan 18, 2020

VS2017 を以下でインストールしてコンパイルできた。

vs_community__2147242378.1579356070.exe --config <.vsconfig のファイルパス>

@AppVeyorBot
Copy link

Build sakura 1.0.2548 completed (commit 9d4a66ce99 by @m-tmatma)

@m-tmatma m-tmatma marked this pull request as ready for review January 18, 2020 14:16
@m-tmatma
Copy link
Member Author

ドキュメントは後で書きます。

@m-tmatma m-tmatma added this to the v2.4.0 milestone Jan 18, 2020
@berryzplus
Copy link
Contributor

文句なし合意。

この辺どうします?

  • microsoft.visualstudio.component.vc.atl
    VC++特有のテンプレートクラスを使うのに必要な標準コンポーネント。
    (コミットログを見た感じ、意図的に外している風に見えたけど何故。)
  • component.github.visualstudio
    Visual Studioをgithubと連携させるのに便利なプラグイン。
  • Microsoft.VisualStudio.Workload.Python
    Visual Studio で .py を編集するのに必要。
  • component.cpython3.x64
    個別にPython実行環境を用意しない場合に必要。(つまり「一般向け」とするなら必要。)

大半の人にとって要らなさそうだけど(ぼくは入れてる)

  • microsoft.visualstudio.component.vc.cmake.project
    VC++ で CMake プロジェクト を開くのに必要。
  • component.linux.cmake
    Visual Studio で MinGW ビルド(CMake)を行うのに必要。

あと、ちょっと微妙なんですがこういう変更をしています。
#1044 プロジェクトのビルドにWindows10 SDKを使う

Windows 10 SDKの特定バージョンをインストールさせる設定方法が分からんかったです。
そして、vs2017とvs2019で入れるべきバージョンが違うっている・・・。

@m-tmatma
Copy link
Member Author

この辺どうします?

  • microsoft.visualstudio.component.vc.atl
    VC++特有のテンプレートクラスを使うのに必要な標準コンポーネント。
    (コミットログを見た感じ、意図的に外している風に見えたけど何故。)

対応当時、必要だったと思って入れたはずでしたが、
いま試したらなくてもコンパイルできたので revert しました。

  • component.github.visualstudio
    Visual Studioをgithubと連携させるのに便利なプラグイン。
  • Microsoft.VisualStudio.Workload.Python
    Visual Studio で .py を編集するのに必要。
  • component.cpython3.x64
    個別にPython実行環境を用意しない場合に必要。(つまり「一般向け」とするなら必要。)

大半の人にとって要らなさそうだけど(ぼくは入れてる)

  • microsoft.visualstudio.component.vc.cmake.project
    VC++ で CMake プロジェクト を開くのに必要。
  • component.linux.cmake
    Visual Studio で MinGW ビルド(CMake)を行うのに必要。

とりあえずはビルドに最低限必要なものと考えています。
必要に応じて追加をするのを別 PR で検討すればいいと思います。

@m-tmatma
Copy link
Member Author

この辺どうします?

  • microsoft.visualstudio.component.vc.atl
    VC++特有のテンプレートクラスを使うのに必要な標準コンポーネント。
    (コミットログを見た感じ、意図的に外している風に見えたけど何故。)

対応当時、必要だったと思って入れたはずでしたが、
いま試したらなくてもコンパイルできたので revert しました。

#1044 の変更を確認したら UseOfMfc が false になっています。
当時は必要だったが、現在は不要になっているのだと思います。

ちなみに

microsoft.visualstudio.component.vc.atl → ATL のみ
microsoft.visualstudio.component.vc.atlmfc → ATL + MFC

という意味です。

@AppVeyorBot
Copy link

Build sakura 1.0.2549 completed (commit a2c237bc8a by @m-tmatma)

@m-tmatma
Copy link
Member Author

大半の人にとって要らなさそうだけど(ぼくは入れてる)

  • microsoft.visualstudio.component.vc.cmake.project
    VC++ で CMake プロジェクト を開くのに必要。
  • component.linux.cmake
    Visual Studio で MinGW ビルド(CMake)を行うのに必要。

あと、ちょっと微妙なんですがこういう変更をしています。
#1044 プロジェクトのビルドにWindows10 SDKを使う

vs_community__2147242378.1579356070.exe --remove Microsoft.VisualStudio.Component.Windows10SDK.17763

を実行してみると、 microsoft.visualstudio.component.vc.cmake.project も同時に削除されて結果としてテストプロジェクトのビルドが通らなくなったので、追加しました。

ただ Microsoft.VisualStudio.Component.Windows10SDK.17763 を削除しても sakura.exe 本体のコンパイルは成功しました。

そこのところはわからないですが、17763 に依存しているので念のために入れておきます。

9860ac7

@AppVeyorBot
Copy link

Build sakura 1.0.2550 completed (commit 8677d9824f by @m-tmatma)

@berryzplus
Copy link
Contributor

とりあえずはビルドに最低限必要なものと考えています。
必要に応じて追加をするのを別 PR で検討すればいいと思います。

了解。

@m-tmatma m-tmatma merged commit 2fa247d into sakura-editor:master Jan 19, 2020
@m-tmatma m-tmatma deleted the feature/dot-vsconfig branch January 19, 2020 04:09
@berryzplus
Copy link
Contributor

ただ Microsoft.VisualStudio.Component.Windows10SDK.17763 を削除しても sakura.exe 本体のコンパイルは成功しました。

vs2019 なら 18362 を参照するので 17763 がなくても通るはずです。
vs2017 でビルドできたんだとしたら原因は分からんです。

HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 16, 2020
Visual Studio のインストールオプションの設定ファイルを追加する
@ghost ghost mentioned this pull request Apr 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants