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

sakura.vcxprojのビルド設定を「手作り仕様」に書き替える #1730

Merged

Conversation

sanomari
Copy link
Contributor

PR の目的

開発をしやすくすることが目的です。

カテゴリ

  • ビルド関連
    • ローカルビルド

(主にローカルビルドの利便性向上が目的ですが、実態はCICDビルドにも影響はします。)

PR の背景

いま、サクラエディタのビルド設定は4つあります。

  1. Win32 - Debug構成
  2. Win32 - Release構成
  3. x64 - Debug構成
  4. x64 - Release構成

しかし、実際の設定は以下の理由に応じて決めていると思います。

  1. プロジェクトの基本設定
  2. Debugビルドするときの設定
  3. Releaseビルドするときの設定
  4. x86ビルドするときの設定
  5. x64ビルドするときの設定

現状だと基本設定は4か所に記載されていて、他の設定も各2か所ずつ設定しています。

変更が少し面倒です。

PR のメリット

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

仕様・動作説明

既存の設定は変えずに、設定の組み方だけ変更します。
間違って変えてるところはないつもりですが、あったら指摘してください。

PR の影響範囲

テスト内容

テスト1

手順

関連 issue, PR

参考資料

https://kyabatalian.hatenablog.com/entry/2019/01/06/170759

従来IDEで作成した設定をベースに手作りしていたのを変更。
これにより、複数のビルド環境に対する共通的な変更を簡単に行えるようになります。
@sonarcloud
Copy link

sonarcloud bot commented Sep 27, 2021

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

Build sakura 1.0.3923 completed (commit 7273df25fa by @sanomari)

@beru
Copy link
Contributor

beru commented Sep 27, 2021

Property Sheetを使う事で同じような対応は取れないでしょうか?結局は手書きする方が柔軟性は高いと思いますが。

@berryzplus
Copy link
Contributor

https://github.com/sakura-editor/sakura/blob/master/tests/unittests/tests1.vcxproj と合わせてくれてるのかな?
個人的にはこういうほうが好きです。
変更内容が適切か?と言われると自信ないですが、良さそうに見えます。

Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

いいんじゃないでしょうか。
と見解を示しておかないと進まなそうなので「えいやっ!」とapproveしときます。

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.

None yet

4 participants