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.

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

Modification of package format for pip #1136

Merged
merged 1 commit into from
Mar 23, 2022

Conversation

ThiefaineM
Copy link

Packages download with pip or pip3 are taken from the index Pypi.
To be more accurate for the purl / in general, it could be better to use Pypi
to describe the packages format.
Here for the purl for a python component:

The scond one is recognized but not the first.

Resolves: #1130

Signed-off-by: Thiéfaine Mercier thiefaine.mercier@avisto.com

@rnjudge
Copy link
Contributor

rnjudge commented Mar 21, 2022

@nishakm I'm okay with this change, wdyt? The sample reports will need to be updated as well but I can do that in another PR.

@rnjudge
Copy link
Contributor

rnjudge commented Mar 21, 2022

@ThiefaineM I think your PR commit message is just a little too long for the linting... must be less than 72 characters in the body :) Once you update, this looks good to me. You can ignore the prospector error.

Packages download with pip or pip3 are taken
from the index Pypi.
To be more accurate for the purl / in general,
it could be better to use Pypi
to describe the packages format.

Resolves: tern-tools#1130

Signed-off-by: Thiéfaine Mercier <thiefaine.mercier@avisto.com>
@ThiefaineM
Copy link
Author

It is updated, sorry for the inconvenience !

@nishakm
Copy link
Contributor

nishakm commented Mar 22, 2022

LGTM!

@rnjudge
Copy link
Contributor

rnjudge commented Mar 22, 2022

Shoot, @ThiefaineM - I think there is an extra commit attached to this PR now. Can you remove the "Merge branch 'tern-tools:main' into purl-gen-pip" commit?

@ThiefaineM
Copy link
Author

ThiefaineM commented Mar 22, 2022

It's done ! Everything should be okay :)

@rnjudge rnjudge merged commit cdc6732 into tern-tools:main Mar 23, 2022
@ThiefaineM ThiefaineM deleted the purl-gen-pip branch March 23, 2022 16:39
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.

Purl generation
3 participants