8000 Add "supported-licenses" command by weeman1337 · Pull Request #401 · fsfe/reuse-tool · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add "supported-licenses" command #401

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 18, 2021
Merged

Add "supported-licenses" command #401

merged 1 commit into from
Sep 18, 2021

Conversation

weeman1337
Copy link
Contributor

Starting simple:

  • new supported-licenses command
  • prints all supported licences with id, name and reference
    • each one in a single line
    • the fields are tab separate

closes #363

mxmehl
mxmehl approved these changes Sep 17, 2021
View reviewed changes
@mxmehl
Copy link
Member
mxmehl commented Sep 17, 2021 •
edited
Loading

Thanks a bunch!

In c442e33 I just fixed a typo.

In 8d9b95a I made a proposal for a more table-style look avoiding external libraries. The fields can also be easily parsed (I tested cut and awk). Please feel free to test and also revert if you disagree :)

@mxmehl mxmehl force-pushed the feature-list-licenses branch from d0180ff to 8d9b95a Compare September 17, 2021 14:21
@weeman1337 weeman1337 force-pushed the feature-list-licenses branch 2 times, most recently from 3da2ff7 to ffeda62 Compare September 17, 2021 17:46
@weeman1337
add list-licenses command
8aa5516 
Signed-off-by: Michael Weimann <mail@michael-weimann.eu>
@weeman1337 weeman1337 force-pushed the feature-list-licenses branch from ffeda62 to 8aa5516 Compare September 17, 2021 17:48
@weeman1337
Copy link
Contributor Author

Thank @mxmehl for the suggestions. I rebased everything and solved the conflicts.
As well as some Git cleanup to keep the log tidy.

@weeman1337 weeman1337 requested a review from mxmehl September 17, 2021 17:50
@mxmehl mxmehl merged commit b7b9c7c into fsfe:master Sep 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mxmehl mxmehl mxmehl approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

List all SPDX license identifiers (compatible with REUSE)
2 participants
@weeman1337 @mxmehl
0