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

azure pipelineのmsys2インストールをやめる。 #1352

Merged

Conversation

berryzplus
Copy link
Contributor

PR の目的

azure pipelineのmsys2インストールをやめて、MinGWビルドのビルド時間を短縮します。

カテゴリ

  • ビルド関連
    • Azure Pipelines

PR の背景

当初、azure pipelinesにはmsys2とmingw-w64がインストールされていなかった。
 ↓
いつの間にか、msys2とmingw-w64が標準でプリインストールされるようになった。
 ↓
msys2とmingw-w64のインストールに時間がかかるようになった。
 ↓
msys2とmingw-w64のインストールを廃止して、ビルド時間を短縮したい。

PR のメリット

azure pipelinesのMinGWビルドの時間を10分程度短縮できます。

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

将来的にMicrosoftがmsys2のプリインストールを外した場合、MinGWビルドが失敗します。

仕様・動作説明

テスト内容

テスト1

手順

PR の影響範囲

azure pipelinesのMinGWビルド

関連 issue, PR

参考資料

msys2とmingw-w64が標準でプリインストールされるようになったため廃止する。
@k-takata
Copy link
Member

GitHub Actionsには先月くらいからmsys2が入ったようですね。
https://github.com/actions/virtual-environments
azpも同じころからなのだろうか?

@berryzplus
Copy link
Contributor Author

レビューありがとうございます。マージしちゃいます。

@berryzplus berryzplus merged commit 66cda09 into sakura-editor:master Jul 28, 2020
@berryzplus berryzplus deleted the feature/use_pre-installed_msys2 branch July 28, 2020 16:21
@k-takata k-takata added azure pipelines CI appveyor など CI 関連 【ChangeLog除外】 labels Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
azure pipelines CI appveyor など CI 関連 【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants