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: retain only licenses as build artifacts #655

Merged
merged 2 commits into from
Sep 25, 2024
Merged

Conversation

lumag
Copy link
Collaborator

@lumag lumag commented Sep 17, 2024

There is little point in retaining full set of build artifacts for the test builds, which is what the empty "artifacts" config does. Keep just the licences and throw away everything else.

@lumag
Copy link
Collaborator Author

lumag commented Sep 17, 2024

cc @ricardosalveti

There is little point in retaining full set of build artifacts for the
test builds, which is what the empty "artifacts" config does. Keep just
the licences and throw away everything else.

Signed-off-by: Dmitry Baryshkov <[email protected]>
Now as we are not publishing build artifacts, also build
initramfs-firmware-mega-image. This verifies that there are no conflicts
between firmware packages, at the same time verifying that all files can
be downloaded.

Signed-off-by: Dmitry Baryshkov <[email protected]>
@ricardosalveti
Copy link
Contributor

Is there any value on retaining the licenses?

@lumag
Copy link
Collaborator Author

lumag commented Sep 25, 2024

@ricardosalveti it's mostly a bandaid for TuxOE, which populates all artifacts if it finds empty or non-existing artifacts array. I think it's better than having artifacts: [ "nonexisting"]. But I can use that if you'd prefer it.

@ricardosalveti
Copy link
Contributor

Sure, works for me, was just curious why it was needed.

@lumag lumag merged commit cff2e79 into Linaro:master Sep 25, 2024
2 checks passed
@lumag lumag deleted the CI-artifacts branch September 25, 2024 22:11
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.

2 participants