make_fossa_deps_conan.py conan 2.22 graph info compatability#1626
Closed
TheIceman2706 wants to merge 1 commit intofossas:masterfrom
Closed
make_fossa_deps_conan.py conan 2.22 graph info compatability#1626TheIceman2706 wants to merge 1 commit intofossas:masterfrom
TheIceman2706 wants to merge 1 commit intofossas:masterfrom
Conversation
Working with the make_fossa_deps_conan.py script and conan 2.22 some errors were encountered. Conan now wraps the graph in a "graph" object and exports the nodes as named objects. Also some more information is directly available. The changes in this PR correspond to the new structure and are actively used at Miele & Cie. KG in multiple CI pipelines to check license compliance. For questions please contact friedrich.may@miele.com (my work email). Submitted on behalf of Miele.
6 tasks
Contributor
|
Thanks for this contribution! Because of the way our CI works, PRs have to be made by a FOSSA employee so I've pulled this change and made a new PR over here: #1629. You will still be credited, thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview
This PR makes make_fossa_deps_conan.py work with conan 2.22.
Conan now wraps the graph in a "graph" object and exports the nodes as named objects. Also some more information is directly available.
The changes in this PR correspond to the new structure and are actively used at Miele & Cie. KG in multiple CI pipelines to check license compliance.
Acceptance criteria
When users use conan 2.22 for their package management, they can now integrate fossa into their workflow.
Risks
This may break compatibility to earlier conan versions.
Tests
Defining automated tests for this script is not feasible in the current test setup.
References
Checklist
docs/.docs/README.msand gave consideration to how discoverable or not my documentation is.Changelog.md. If this PR did not mark a release, I added my changes into an## Unreleasedsection at the top..fossa.ymlorfossa-deps.{json.yml}, I updateddocs/references/files/*.schema.jsonAND I have updated example files used byfossa initcommand. You may also need to update these if you have added/removed new dependency type (e.g.pip) or analysis target type (e.g.poetry).docs/references/subcommands/<subcommand>.md.For questions please contact friedrich.may@miele.com (my work email). Submitted on behalf of Miele.