Skip to content

Conversation

@eccles
Copy link
Contributor

@eccles eccles commented Nov 9, 2022

Problem:
Validator complains about 'author' appering before 'group' or name'.

Solution:
Enforce latest versions of syft, cdx cmds and check ordering guarantees
with Elementtree python module.
Upgrade to cyclonedx 1.4.
Create Dockerfile to enforce dependency versions and enable local
testing.
Remove xmllinting of cyclonedx as upstream now handles this.
Executing xmllint was failing to invalidate the xml file because of the
need to execute in a pipeline.sbom_scraper.sh can now be executed from any location and is not tied to
being executed in the repo.

Signed-off-by: Paul Hewlett phewlett76@gmail.com

@eccles eccles force-pushed the dev/eccles/dockerfile branch 3 times, most recently from 931baf3 to a3d86f2 Compare November 9, 2022 18:39
@eccles eccles force-pushed the dev/eccles/dockerfile branch 2 times, most recently from 3a4c99b to b23c7da Compare November 10, 2022 10:15
Problem:
Validator complains about 'author' appearing before 'group' or name'.

Solution:
Enforce latest versions of syft, cdx cmds and check ordering guarantees
with Elementtree python module.
Upgrade to cyclonedx 1.4.
Create Dockerfile to enforce dependency versions and enable local
testing.
Remove xmllinting of cyclonedx as upstream now handles this.
Executing xmllint was failing to invalidate the xml file because of the
need to execute in a pipeline.
sbom_scraper.sh can now be executed from any location and is not tied to
being executed in the repo.

Signed-off-by: Paul Hewlett <phewlett76@gmail.com>
@eccles eccles force-pushed the dev/eccles/dockerfile branch from b23c7da to 8db703e Compare November 10, 2022 14:16
Copy link

@landintrees landintrees left a comment

Choose a reason for hiding this comment

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

LGTM

@eccles eccles merged commit 3845ec0 into main Nov 10, 2022
@eccles eccles deleted the dev/eccles/dockerfile branch November 10, 2022 14:18
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