Build Requirementsの記述を現状に合わせる #1952
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced May 19, 2024
suconbu
reviewed
May 24, 2024
suconbu
approved these changes
May 26, 2024
|
レビューありがとうございました。 |
|
せっかく最近 #1948 でメンテしたのに、MinGWのビルドエラーが出ている件。 |
AppVeyorのログを見てみました。 |
|
最新版で再現しなかったので、MinGWビルドエラーへの対処は行わないものとします。 PS C:\work\sakura-editor\sakura> choco list --local-only msys2
Chocolatey v1.1.0
msys2 20240507.0.0
1 packages installed.
PS C:\work\sakura-editor\sakura> C:\msys64\mingw64\bin\gcc.exe --version
gcc.exe (Rev1, Built by MSYS2 project) 12.2.0 |
beru
pushed a commit
that referenced
this pull request
Jun 14, 2024
#1953) * `project-PlatformToolset.md`を2022対応に合わせて修正 * Express 2017 for Windows Desktopは対象外であることを明記 * 企業でもCommunity版を使って貢献できることを明記 * Visual Studio 2025以降を見据えた表記に修正 * 参照形式リンクから不要なバージョン情報除去 * Update README.md Co-authored-by: berryzplus <berryzplus@gmail.com> * READMEからVS2017への言及箇所削除 * TOC修正 --------- Co-authored-by: berryzplus <berryzplus@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR対象
カテゴリ
PR の背景
仕様・動作説明
ドキュメント修正のみです。
現状： Visual Studioの特定バージョン・特定エディションでないとビルドできないように見える。
実態： Windows 10 SDKとC++17に対応したC++開発環境ならビルドは可能。（ビルドだけならMinGWでもできる）
Community Editionを使う必要はありません。
PR の影響範囲
ドキュメント修正のみです。
テスト内容
ドキュメント修正のみなので、ありません。
関連 issue, PR
参考資料
特になし
残課題
この記述が正しいか、自信がありません。
sakura/README.md
Line 76 in 7eb997a
GitHub ActionsとAzure pipelinesによるビルドではEnterprise Editionが使われます。
現状では #1809 を対応できておらず、リリースには AppVeyor で生成されたモジュールを使っています。
AppVeyorではビルドに Community Edition を指定できるのかも知れません。
自分の記憶する限り合意は「リリース版にはvs2017でビルドしたモジュールを使う」までで、Community Editonを使うことまでは合意してなかったような・・・。