CI: only generate a contrib tarball if we're on the main branch
There's no need to generate the tarball since it's not going to be used by anyone else. It's only used on master after a merge or in nightly builds.
This should save a few minutes during each contrib MR builds.
The values are
pipeline type | CI_COMMIT_BRANCH | CI_DEFAULT_BRANCH |
---|---|---|
MR pipeline | '' | 'master' |
merged on master | 'master' | 'master' |
nightly build | 'master' | 'master' |
Merge request reports
Activity
changed milestone to %4.0
added Component:: CI label
requested review from @ePirat
added MRStatus::Reviewable label
- Resolved by Alexandre Janniaux
What about 3.0.x?
added MRStatus::InReview label and removed MRStatus::Reviewable label
added MRStatus::Acceptable label and removed MRStatus::InReview label
added MRStatus::Accepted label and removed MRStatus::Acceptable label
MR Acceptance result
This MergeRequest has been Accepted! Congratulations.MR acceptance checks details:
-
MR should be considered mergeable by Gitlab -
Last pipeline should be successful -
MergeRequest should have at least one external review and/or vote -
All threads should be resolved, and score >= 0 -
MergeRequest should have no activity (threads/votes) for (24h/24h)
-
added 1 commit
- 5928a080 - CI: log the branch used to build in Debian job
added 8 commits
-
5928a080...3b6a553c - 6 commits from branch
videolan:master
- 70ccc951 - CI: only generate a contrib tarball if we're on the main branch
- bdf08ca5 - CI: log the branch used to build in Debian job
-
5928a080...3b6a553c - 6 commits from branch
- Resolved by Steve Lhomme
For safety we should probably merge the second commit first and verify the condition works between MR and merged/nightly builds.
added MRStatus::InReview label and removed MRStatus::Accepted label
mentioned in merge request !2715 (merged)
added MRStatus::NotCompliant label and removed MRStatus::InReview label
added 15 commits
-
bdf08ca5...b147b800 - 14 commits from branch
videolan:master
- a54e8016 - CI: only generate a contrib tarball if we're on the main branch
-
bdf08ca5...b147b800 - 14 commits from branch