Skip to content

CI-mingw.yml: use pre-installed MinGW / also use lld and ccache for faster builds - #4400

Merged
danmar merged 5 commits into
cppcheck-opensource:mainfrom
firewave:mingw-ci
Sep 4, 2023
Merged

CI-mingw.yml: use pre-installed MinGW / also use lld and ccache for faster builds#4400
danmar merged 5 commits into
cppcheck-opensource:mainfrom
firewave:mingw-ci

Conversation

@firewave

@firewavefirewave commented Aug 25, 2022

Copy link
Copy Markdown
Collaborator

This used to be one of the longest running jobs because of the slow setup and linking. Now it will take only ~2 minutes if everything is cached with half the time taken up by the tests.

@firewavefirewave changed the title CI-mingw.yml: removed MinGW setup step since it is pre-installed in windows-* imagesCI-mingw.yml: use pre-installed MinGWAug 25, 2022
@firewave
firewaveforce-pushed the mingw-ci branch 5 times, most recently from eab43fc to 586d271CompareAugust 29, 2022 17:16
@firewave
firewaveforce-pushed the mingw-ci branch 3 times, most recently from fe5ea3f to 9e8658bCompareSeptember 25, 2022 20:16
chrchr-github added a commit to chrchr-github/cppcheck that referenced this pull request May 10, 2023
@firewave

Copy link
Copy Markdown
CollaboratorAuthor

Accidentally closed by having a Trac ticket match a GitHub PR - re-opening.

@firewavefirewave reopened this Sep 1, 2023
@firewave
firewaveforce-pushed the mingw-ci branch 4 times, most recently from c6d2964 to c29ac28CompareSeptember 2, 2023 13:36
@firewavefirewave changed the title CI-mingw.yml: use pre-installed MinGWCI-mingw.yml: use pre-installed MinGW, lld and ccache for faster buildSep 2, 2023
@firewave
firewaveforce-pushed the mingw-ci branch 2 times, most recently from 56e30ee to f755ca8CompareSeptember 2, 2023 13:53
@firewavefirewave changed the title CI-mingw.yml: use pre-installed MinGW, lld and ccache for faster buildCI-mingw.yml: use pre-installed MinGW / also use lld and ccache for faster buildsSep 2, 2023
@firewave
firewaveforce-pushed the mingw-ci branch 2 times, most recently from 3869256 to 7acbe59CompareSeptember 2, 2023 14:02
@firewave

Copy link
Copy Markdown
CollaboratorAuthor

I have lowered the recursion limit for TestLeakAutoVarRecursiveCountLimit several times but I still keeps crashes with a stack overflow. I do not want to adjust the limit in the code since it was fine outside MSYS2 (and probably will be for all users which are not as memory limited as the CI) and (hopefully temporarily) disabled the test for MinGW instead.

@firewave

Copy link
Copy Markdown
CollaboratorAuthor

@firewave
firewave marked this pull request as ready for review September 2, 2023 14:05
@danmar

Copy link
Copy Markdown
Collaborator

I feel I can merge this.. do you feel it's ready?

@firewave

Copy link
Copy Markdown
CollaboratorAuthor

I feel I can merge this.. do you feel it's ready?

Yup. That's why I marked it for review.

@danmar
danmar merged commit c068c74 into cppcheck-opensource:mainSep 4, 2023
@firewave
firewave deleted the mingw-ci branch September 4, 2023 12:53
Sign up for freeto 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

@firewave@danmar