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

Add the EUPL license to Software::License #58

Closed
wants to merge 5 commits into from

Conversation

waterkip
Copy link
Contributor

No description provided.

@haarg
Copy link
Member

haarg commented Mar 20, 2017

The meta_name and meta2_name methods should return valid license strings for CPAN meta v1 and meta v2.

The full license text should be included, not just a reference to a URL.

@Leont
Copy link
Member

Leont commented Mar 20, 2017

The meta_name and meta2_name methods should return valid license strings for CPAN meta v1 and meta v2.

In both cases that should be open_source.

SPDX support would be helpful here.

The full license text should be included, not just a reference to a URL.

Agreed.

@karenetheridge
Copy link
Member

The meta_name and meta2_name methods should return valid license strings for CPAN meta v1 and meta v2.

As a breadcrumb to contributors, it would be useful to add a test that checked that all licence classes returned correct strings here.

@waterkip
Copy link
Contributor Author

waterkip commented Mar 20, 2017

I've updated the PR with your comments.

Cheers,
Wesley

@waterkip
Copy link
Contributor Author

waterkip commented Apr 1, 2017

Is there any indication when this PR can/will be merged?

@Leont
Copy link
Member

Leont commented Apr 2, 2017

Is there any indication when this PR can/will be merged?

I expect we'll look over it at the Perl Toolchain Summit next month. A new release of Software::License is due.

@Leont
Copy link
Member

Leont commented Aug 22, 2017

Given that this license is versioned, the class should probably also be (also, this is 1.1 but 1.2 has been released since this ticket was opened. Is it desirable to add both?)

@Leont
Copy link
Member

Leont commented Oct 27, 2017

I've squashed this, fixed it up to make dzil happy (PkgVersion wants a newline after the package statement) and merged it.

Enjoy Software::License 0.103013 :-)

@Leont Leont closed this Oct 27, 2017
@waterkip waterkip deleted the EUPL_inclusion branch June 13, 2022 14:03
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.

4 participants