Skip to content

Exclude packages changed from packageconfig diff#2

Merged
jeremie-dautheribes merged 1 commit intobootlin:mainfrom
Louson:main
Mar 25, 2026
Merged

Exclude packages changed from packageconfig diff#2
jeremie-dautheribes merged 1 commit intobootlin:mainfrom
Louson:main

Conversation

@Louson
Copy link

@Louson Louson commented Mar 3, 2026

When a package that has some PACKAGECONFIG is added or remove, all its packageconfigs are shown in the diff, which may drastically increase the output.

The aim of spdx-diff is to get a clear output of changes from two builds. It makes sense to ignore packages changes in the packageconfig diff.

Requires bootlin/meta-spdx-diff-test#1 for test

When a package that has some PACKAGECONFIG is added or remove, all its
packageconfigs are shown in the diff, which may drastically increase the
output.

The aim of spdx-diff is to get a clear output of changes from two
builds. It makes sense to ignore packages changes in the packageconfig
diff.

Signed-off-by: Louis Rannou <louis.rannou@non.se.com>
@jeremie-dautheribes
Copy link
Member

@Louson some tests are still failing, could you have a look please?

@benjarobin
Copy link
Collaborator

@jeremie-dautheribes Well I think bootlin/meta-spdx-diff-test#1 needs to be merged first

jeremie-dautheribes added a commit to bootlin/meta-spdx-diff-test that referenced this pull request Mar 25, 2026
Following bootlin/spdx-diff#2, update the
generated SBOMs.

Signed-off-by: Jérémie Dautheribes <jeremie.dautheribes@bootlin.com>
@jeremie-dautheribes
Copy link
Member

Following bootlin/meta-spdx-diff-test@141b231 the tests now run correctly, so I'll merge it. Thanks!

@jeremie-dautheribes jeremie-dautheribes merged commit ad57989 into bootlin:main Mar 25, 2026
7 of 16 checks passed
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