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

Custom proprietary license support #69

Open
Ivashkaization opened this issue Apr 8, 2021 · 1 comment
Open

Custom proprietary license support #69

Ivashkaization opened this issue Apr 8, 2021 · 1 comment

Comments

@Ivashkaization
Copy link

Ivashkaization commented Apr 8, 2021

Library has following license metadata:

License: X Proprietary
Classifier: License :: Other/Proprietary License

And liccheck.ini file has following authorized license in it:

[Licenses]
authorized_licenses:
        x proprietary

But liccheck fails, because it needs Other/Proprietary License (as classifier license) to be in liccheck.ini file

However, this doesn't reproduce on versions below 0.5.2

I suppose changing:

licenses = get_license(dist) + get_license_OSI_classifiers(dist)

to:
licenses = get_licenses_from_classifiers(dist) or get_license(dist) or []

caused it.

Maybe liccheck should handle proprietary licenses differently to avoid allowing all Other/Proprietary License instead of custom proprietary licenses?

@Niccolum
Copy link

Niccolum commented Aug 6, 2021

What about progress? Still waiting?

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

No branches or pull requests

2 participants