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

.github: Autotest build need debug flag, rover also need enable-math-… #26170

Merged
merged 1 commit into from
Feb 10, 2024

Conversation

khancyr
Copy link
Contributor

@khancyr khancyr commented Feb 8, 2024

…check-indexes. Fix caching

This allow all autotest to reuse the cached build from 1rst step as intended. We gain about 4min CI time per autotest
Previously :
image

now :
image

(Size difference is because that is 2 differents branch)

Copy link
Collaborator

@Ryanf55 Ryanf55 left a comment

Choose a reason for hiding this comment

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

Nice improvement! No need to build in release then rebuild in debug :)

@khancyr
Copy link
Contributor Author

khancyr commented Feb 9, 2024

yep. It has been like this for some years, and nobody notice it ...
We still have some other test doing the build in release, so that we test both debug and release in CI

@peterbarker peterbarker merged commit e728176 into ArduPilot:master Feb 10, 2024
92 checks passed
@peterbarker
Copy link
Contributor

Merged, thanks!

@khancyr khancyr deleted the feature/CI-fix-builds branch February 10, 2024 23:40
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.

3 participants