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

Pass along unknown licenses unclassified #185

Merged
merged 1 commit into from
May 6, 2019

Conversation

andre-rosa
Copy link
Contributor

@andre-rosa andre-rosa commented May 2, 2019

@andre-rosa
Copy link
Contributor Author

@allenh1 Ptal

superflore/utils.py Outdated Show resolved Hide resolved
superflore/utils.py Outdated Show resolved Hide resolved
superflore/utils.py Outdated Show resolved Hide resolved
@andre-rosa andre-rosa changed the title Add new licenses Proprietary and Government Purpose Rights Pass along unknown licenses unclassified May 3, 2019
@tfoote
Copy link
Member

tfoote commented May 3, 2019

This looks good to me. @allenh1 Does Gentoo require that they be specifically classified or from a certain list? If so we could always add an optional argument to this that would trigger the exception if the license can't be classified. And this could be used by Gentoo packaging, but not set by the OpenEmbedded invocation.

@allenh1
Copy link
Collaborator

allenh1 commented May 3, 2019

Does Gentoo require that they be specifically classified or from a certain list?

Gentoo doesn't, but a lot of users do. In the past, we had this sort of rule in place (though we just copied the license string in that case). I think that, for now, having an "unclassified" license will be just fine.

I think we do a release after this & the rest of the preliminary open embedded PR's come in, then I'll get the automatic license pulling on the docker for the next superflore release.

Copy link
Collaborator

@allenh1 allenh1 left a comment

Choose a reason for hiding this comment

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

This just needs a bit more work. PTAL at the tests for unknown licenses, and satisfy the linter.

  - Fixes ros-infrastructure#184 generating qt_metapackages recipes in Melodic
  - Currently Proprietary and United States Government Purpose licenses
    aren't matched in SuperFlore and it aborts generation of some melodic
    packages like qt_metapackages/libqt_core
    https://github.com/swri-robotics/qt_metapackages/blob/master/libqt_core/package.xml
  - With the patch, license is output as:
    ...
    LICENSE = "United States Government Purpose & SwRI Proprietary"
    LIC_FILES_CHKSUM = "file://package.xml;beginline=11;endline=11;md5=8e473b26ead33548c945bdaca143a08d"
    ...
@allenh1
Copy link
Collaborator

allenh1 commented May 6, 2019

@andre-rosa lgtm. Thanks for iterating on this.

@allenh1 allenh1 merged commit d87f8dd into ros-infrastructure:master May 6, 2019
@andre-rosa andre-rosa deleted the new_licenses branch May 6, 2019 20:36
@andre-rosa
Copy link
Contributor Author

Thank you, @allenh1 !

zffgithub pushed a commit to zffgithub/superflore that referenced this pull request Apr 11, 2023
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.

Melodic qt_metapackages cannot be generated
3 participants