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

vs2022対応(仮) #1763

Closed

Conversation

berryzplus
Copy link
Contributor

PR の目的

#1755 のレビューで気になったところを自分で実装してみたブランチを共有します。
PRにしておけばブランチ消しても残せるのでPRにしておきます。

カテゴリ

  • 実験 (master へのマージを目的としない)
  • その他の問題

PR の背景

PR のメリット

vs2022でビルドできるようになります。
find-tools.batの潜在的なバグ(?)が直ります。

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

現状で複数バッチから利用されていない2ツールの探索ロジックをfind-tools.batから外します。
複数から使ってないツールは他にもあるやんけ、とか色々文句が出る懸念があります。

仕様・動作説明

変更点

  • build-sln.bat の第3引数にバージョンを指定できるようになります。
  • build-all.bat の第3引数にバージョンを指定できるようになります。
  • 前回指定したバージョンと異なるバージョンを指定した場合に洗い替えする機能を追加します。

PR の影響範囲

ビルド全般に影響する変更です。
主にローカルビルドの、MSVCによるビルドに影響します。
アプリ機能への影響はないと考えられます。

テスト内容

テスト1

手順

cd sakura
build-sln Win32 Debug 2017
build-sln Win32 Release 2019
build-sln x64 Debug 2022

x86デバッグ版はvs2017でビルドされ、
x86リリース版はvs2019でビルドされ、
x64リリース版はvs2022でビルドされます。

関連 issue, PR

参考資料

@sonarcloud
Copy link

sonarcloud bot commented Jan 10, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@AppVeyorBot
Copy link

@berryzplus
Copy link
Contributor Author

berryzplus commented Jan 12, 2022

#1764 がマージされて当初目的は果たされたと思うので、閉じておきます。
018ef0aafaf9e5 の変更は、入れたほうがいいのかも知れませんが。。。

@ghost
Copy link

ghost commented Jan 30, 2022

816b239 3485c28 4890089 の取り込み要望( #1785 )を出しました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants