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

Enable create-spdx when building images. #567

Merged
merged 2 commits into from
Apr 20, 2023

Conversation

dylan-matthew-turner
Copy link
Contributor

Because we now skip creating SPDX when a package comes from a remote feed, we can enable creating the spdx zst archives when building images.

To enable it, we need to inherit from create-spdx at the top-level bitbake config, and add it to the list of DISTRO_FEATURES.

I verified this produces the expected archive by building packagefeed-ni-core followed by nilrt-base-system-image

@dylan-matthew-turner dylan-matthew-turner requested a review from a team April 18, 2023 14:50
conf/distro/nilrt.inc Outdated Show resolved Hide resolved
conf/distro/nilrt.inc Outdated Show resolved Hide resolved
Under DISTRO_FEATURES, all features listed used spaces not tabs except
virtualization. Make it consistent with the rest of the formatting

Signed-off-by: Dylan Turner <dylan.turner@ni.com>
Enable create-spdx by inheriting the class and set it to run on image
creation by adding it to the distro feature set

Signed-off-by: Dylan Turner <dylan.turner@ni.com>
@dylan-matthew-turner
Copy link
Contributor Author

dylan-matthew-turner commented Apr 18, 2023

I moved the changes to nilrt.conf from nilrt.inc (and thus also split the INHERIT), but I left a change in nilrt.inc to convert one tab in a string of spaces to all spaces for the sake of consistency.

I rebuilt after moving, and the spdx is still being created

@amstewart amstewart merged commit 52a9afa into ni:nilrt/master/kirkstone Apr 20, 2023
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