search

pear / pear-core

This is the definitive source of PEAR's core files.
http://pear.php.net/package/PEAR
Other
108 stars 103 forks source link

update PHP License to 3.01 #116

Closed remicollet closed 9 months ago

sonarcloud[bot] commented 3 years ago

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

ashnazg commented 3 years ago

@remicollet are these license changes supposed to align with the BSD license in the package2.xml?

remicollet commented 3 years ago

@remicollet are these license changes supposed to align with the BSD license in the package2.xml?

No.

This update to latest version of the PHP license (notice, IIRC, some distro refuse PHP 3.0 as a valid license, e.g. Debian)

As some files use the PHP License, this means this package is "PHP", not "BSD" So package.xml also needs to be fixed

ashnazg commented 3 years ago

The overall package was switched from PHP 3.0 to New BSD way back (https://github.com/pear/pear-core/commit/48f5f4f932dad88dc32c40e337b257ed631814cc), so I have to assume any files still showing PHP license in them must have simply been overlooked at the time.

@kenguest @cweiske @CloCkWeRX @helgi -- anyone else see this differently?

kenguest commented 3 years ago

I concur - any package that still is licensed with the PHP License should be changed over to BSD.

On Wed, 11 Aug 2021, 23:07 Chuck Burgess, @.***> wrote:

The overall package was switched from PHP 3.0 to New BSD way back (48f5f4f https://github.com/pear/pear-core/commit/48f5f4f932dad88dc32c40e337b257ed631814cc), so I have to assume any files still showing PHP license in them must have simply been overlooked at the time.

@kenguest https://github.com/kenguest @cweiske https://github.com/cweiske @CloCkWeRX https://github.com/CloCkWeRX @helgi https://github.com/helgi -- anyone else see this differently?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pear/pear-core/pull/116#issuecomment-897193329, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABZFBVVU3OTPG4W33F25QLT4LYCLANCNFSM47C7RC6A .

ashnazg commented 3 years ago

Well, the package itself was already BSD... it's just that some files inside the package were still showing PHP in their contents/comments.

kenguest commented 3 years ago

Ok. To rephrase then, any package that still is licensed with the PHP License should be changed over to BSD and any files included therein that specify a license should be updated to remove any ambiguity.

ashnazg commented 9 months ago

Updated files to New BSD (#128) based on your found list... thanks @remicollet !