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

C#ツールの.vsconfigを追加する #1199

Conversation

berryzplus
Copy link
Contributor

PR の目的

C#ツールの.vsconfigを追加することにより、ツールをビルドしやすくする。

カテゴリ

  • その他

PR の背景

#1195 (comment)

とりあえず、インストーラのビルドができればいいので この PR でもいいです。

了解っす。

マニュアルをもう一度確認しましたが、「ソリューションのルート」に vsconfig があれば読み込まれるみたいなんで、C# ツール向けの vsconfig を用意しときたいと思っています。

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

PR のメリット

  • 基本的には、.vsconfigを用意することそのものがメリットになると思っています。
  • 必要なコンポーネントが明確になる結果、ビルドしてみようと思った人が迷わなくて済むようになります。

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

  • バージョンを net461 に揃えたの失敗だったかもです 😢
vsバージョン 既定の.NETバージョン 備考
vs2017 .NET Framework 4.6.1 ここに合わせた。
vs2019 .NET Framework 4.7.2 .vsconfigの恩恵を受けられるのはこっち。

前に書いた通り、既定のバージョンを外すとvsの開発ツールが動かなくなるので、
「net461に合わせる」ってことは、vs2019での追加インストールが必須ということになります。

.vsconfigによる自動追加インストールの恩恵を受けられるのは主にvs2019のほうなので、
どこかでvs2019中心で考える方向にシフトしていったほうがよいのかも知れません。

PR の影響範囲

  • C#ツールを使ってみようとした人が初めてソリューションを開いたときに、「デスクトップ開発(C#)」を有効にしていなかった場合の visual studio の挙動に影響します。

関連チケット

#1195
#1192

参考資料

@AppVeyorBot
Copy link

@m-tmatma m-tmatma added this to the v2.4.0 milestone Feb 14, 2020
@berryzplus
Copy link
Contributor Author

レビューありがとうございます。
とりあえず、マージしていきます。

@berryzplus berryzplus merged commit 403a0e2 into sakura-editor:master Feb 15, 2020
@berryzplus berryzplus deleted the feature/add_vsconfig_for_csharp_tools branch February 15, 2020 04:50
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 16, 2020
…onfig_for_csharp_tools

C#ツールの.vsconfigを追加する
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