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

「簡易リビルド」機能が非推奨になったことへの対応 #676

Closed
berryzplus opened this issue Dec 4, 2018 · 2 comments · Fixed by #688
Closed

「簡易リビルド」機能が非推奨になったことへの対応 #676

berryzplus opened this issue Dec 4, 2018 · 2 comments · Fixed by #688

Comments

@berryzplus
Copy link
Contributor

さっき sakura.sln をクリーンビルドして気付きました。
最近の変更で -Gm オプションが非推奨になったみたいです。

ログに警告が増えてます・・・。

>cl : Command line warning D9035: option 'Gm' has been deprecated and will be removed in a future release

注:最新版(v15.9.3)の挙動です。appveyor のビルドで発生していたかどうかは見ていませんが、appveyorのvs2017は15.8だったはずなので発生しないんじゃないかと思います。

リリースノートにそれらしき記載はありませんが、ドキュメント側の更新日付が新しいので最近投入された変更と考えられます。

対象は headermakemakefilemake です。
緊急じゃありませんが、appveyorのvsバージョンが上がった時にビックリするのもあれなので、なんとなく対応方向を考えときたい感じです。

@berryzplus berryzplus changed the title 「簡易リビルド」機能が非対応になったことへの対応 「簡易リビルド」機能が非推奨になったことへの対応 Dec 4, 2018
@berryzplus
Copy link
Contributor Author

マズいとこ(タイトル)を誤字ってたので修正 😭

@ds14050
Copy link
Contributor

ds14050 commented Dec 7, 2018

チラ見して理解したところでは /Gm オプションというのは、ヘッダファイルが更新された場合でも再コンパイルが必要ない場合があるから独自のメタデータを参照して判断するよ、ってなオプションみたいですね。

ちょっとしたずるというかビルド時間最小化に貪欲だったというか、今ではインクリメンタルビルド、パラレルビルドという真っ当な代替手段で間に合うからそっちを使えと。

HeaderMake も MakefileMake も完成したプロジェクトですからオプションを取り除いてもなんの影響もないのではないでしょうか。

berryzplus added a commit to berryzplus/sakura-editor that referenced this issue Dec 9, 2018
clのオプション-Gmが非推奨になることに対処するため、/MPオプションを有効にします。
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this issue Jun 11, 2019
clのオプション-Gmが非推奨になることに対処するため、/MPオプションを有効にします。
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 a pull request may close this issue.

2 participants