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

[CI] Optimize MSYS2 on Appveyor #6573

Merged

Conversation

oxillo
Copy link
Contributor

@oxillo oxillo commented Apr 20, 2020

Reorganize YAML

  • setup platform: matrix
  • use for: to separate build logics MSYS2/VS - avoid if BUILDER= ... statements

Reduce MSYS2 installation/upgrade time using a clean MSYS2 install.
For comparison :

  • baseline : upgrade current install and install OF packages : 10min or more
  • remove not needed package, upgrade, install OF packages : 7-8min
  • clean install, upgrade, install OF packages : 4min

With the additional gain of CCache, MSYS2 builds are around 15min for MINGW32 and 25min for MINGW64

setup platform matrix
use for: to better separate MSYS2 vs Visual Studio

reinstall MSYS2 from repo
Only install/update required component.
Doing so, decreased installation from 10min to 4 min
@arturoc arturoc merged commit 5453d35 into openframeworks:patch-release Apr 20, 2020
@arturoc
Copy link
Member

arturoc commented Apr 20, 2020

thanks!!

@oxillo oxillo deleted the appveyor_msys2_optimisation branch April 21, 2020 05:35
@sittichai318
Copy link

Reorganize YAML

  • setup platform: matrix
  • use for: to separate build logics MSYS2/VS - avoid if BUILDER= ... statements

Reduce MSYS2 installation/upgrade time using a clean MSYS2 install.
For comparison :

  • baseline : upgrade current install and install OF packages : 10min or more
  • remove not needed package, upgrade, install OF packages : 7-8min
  • clean install, upgrade, install OF packages : 4min

With the additional gain of CCache, MSYS2 builds are around 15min for MINGW32 and 25min for MINGW64

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.

3 participants