Skip to content

fix(generate-cdx): correctly split distro supplier into components#186

Merged
Urist-McGit merged 3 commits intomainfrom
fm/fix-supplier
Mar 4, 2026
Merged

fix(generate-cdx): correctly split distro supplier into components#186
Urist-McGit merged 3 commits intomainfrom
fm/fix-supplier

Conversation

@fmoessbauer
Copy link
Member

The distro supplier is a supplier, hence has to be processed. Previously it was just taken as-is, which added the e-mail address to the name field. We fix this by correctly splitting the supplier.

@fmoessbauer fmoessbauer requested a review from Urist-McGit March 3, 2026 15:21
@fmoessbauer
Copy link
Member Author

@Urist-McGit Please also check the supplier handling on merge. This was out of scope in this bugfix.

Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
The distro supplier is a supplier, hence has to be processed. Previously
it was just taken as-is, which added the e-mail address to the name
field. We fix this by correctly splitting the supplier.

Fixes: 958c97d Initial commit
Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
@Urist-McGit
Copy link
Collaborator

Please also check the supplier handling on merge. This was out of scope in this bugfix.

mrrge only makes a new distribution component, which is covered by your fix.

@Urist-McGit Urist-McGit merged commit 735fc9d into main Mar 4, 2026
14 checks passed
@Urist-McGit Urist-McGit deleted the fm/fix-supplier branch March 4, 2026 08:15
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