Skip to content

Conversation

chenrui333
Copy link

Looks like the license got updated into Artistic-2.0 per 298c06c, which is not aligned with perl5, https://github.com/Perl/perl5/blob/blead/Copying. Thus, filing this PR to update license, @toddr let me know if it makes sense. Thanks!

relates to:

@chenrui333
Copy link
Author

cc @toddr

@chenrui333
Copy link
Author

@toddr gentle ping

@toddr
Copy link
Member

toddr commented Jan 10, 2024

Larry's license is the Artistic license, not GPL.

arnout pushed a commit to buildroot/buildroot that referenced this pull request Feb 5, 2024
The license file 'README' has changed due to the following diff:

-      $p3->parse($foo, ProtocolEncoding => 'ISO-8859-1');
-      close($foo);
+      $p3->parse($fh, ProtocolEncoding => 'ISO-8859-1');
+      close($fh);

This does not affect the licensing terms.

A 'LICENSE' file was added in 2.47, but it contains the terms of the
wrong license (Artistic 2.0), while this package is under Artistic or
GPL-1.0+. This has been reported upstream at
cpan-authors/XML-Parser#99.

Signed-off-by: Kevin Cui <[email protected]>
Signed-off-by: Thomas Petazzoni <[email protected]>
@perlpunk
Copy link

The metadata says
https://metacpan.org/release/TODDR/XML-Parser-2.47/source/META.json#L9

   "license" : [
      "perl_5"
   ],

https://metacpan.org/release/TODDR/XML-Parser-2.47/source/META.yml#L13

license: perl

This includes Artistic (1.0?) and GPL 1.0: https://dev.perl.org/licenses/

Hoewever, the LICENSE file only contains Artistic 2.0

That's also problematic for automated tools that try to get the license from a module. They might get the wrong one.

btw, not sure how to specify something like "perl5, but Artistic 2.0 instead of 1.0 in the META.(json|yml).

bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Mar 1, 2025
https://build.opensuse.org/request/show/1249251
by user pgajdos + dimstar_suse
- Fix license manually (module metadata is contradicting).

See also cpan-authors/XML-Parser#99 (forwarded request 1249249 from tinita)
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.

3 participants