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.

Sign up for GitHub

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

Jump to bottom

Update CycloneDX to use syft namespace and output multiple CPEs #849

Merged
merged 3 commits into from
Mar 1, 2022
Merged

Update CycloneDX to use syft namespace and output multiple CPEs #849

merged 3 commits into from
Mar 1, 2022

Conversation

kzantow
Copy link
Contributor

@kzantow kzantow commented Feb 25, 2022

This fixes: #818 and #842 and will complete the encoder/decoder work for anchore/grype#481

@kzantow
update cyclonedx to use syft namespace and output multiple cpes
472030e 
Signed-off-by: Keith Zantow <kzantow@gmail.com>
@kzantow kzantow linked an issue Feb 25, 2022 that may be closed by this pull request
Use syft: property namespace in CycloneDX #842
Closed
@kzantow
lint
e27e48a 
Signed-off-by: Keith Zantow <kzantow@gmail.com>
@kzantow kzantow changed the title Update CycloneDX to use syft: namespace and output multiple CPEs Update CycloneDX to use syft namespace and output multiple CPEs Feb 25, 2022
kzantow
kzantow commented Feb 25, 2022
View reviewed changes
syft/source/location.go Outdated Show resolved Hide resolved
@kzantow kzantow requested a review from a team February 25, 2022 18:42
wagoodman
wagoodman approved these changes Mar 1, 2022
View reviewed changes
Copy link
Contributor

@wagoodman wagoodman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all sorts of reflect-fun! Took a while for me to get through it. Nice job 🙌

@kzantow
PR feedback
4bd479a 
Signed-off-by: Keith Zantow <kzantow@gmail.com>
@kzantow kzantow merged commit edac8c7 into anchore:main Mar 1, 2022
@kzantow kzantow deleted the multiple-cyclonedx-cpes branch March 1, 2022 22:37
@kzantow kzantow mentioned this pull request Mar 1, 2022
Closed
GijsCalis pushed a commit to GijsCalis/syft that referenced this pull request Feb 19, 2024
@kzantow
Update CycloneDX to use syft namespace and output multiple CPEs (anch…
6b483c1 
…ore#849)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wagoodman wagoodman wagoodman approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Use syft: property namespace in CycloneDX Add support for multiple CPEs in CycloneDX
2 participants
@kzantow @wagoodman