Page MenuHomeFreeBSD

comms/conserver-com: Update licensing details and pet portlint
ClosedPublic

Authored by 0mp on Oct 12 2018, 8:42 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 20, 11:38 AM
Unknown Object (File)
Sat, Apr 20, 11:38 AM
Unknown Object (File)
Sat, Apr 20, 11:38 AM
Unknown Object (File)
Thu, Apr 18, 8:32 AM
Unknown Object (File)
Wed, Apr 10, 12:04 AM
Unknown Object (File)
Wed, Apr 3, 4:52 AM
Unknown Object (File)
Mar 17 2024, 3:24 AM
Unknown Object (File)
Feb 16 2024, 10:23 AM
Subscribers

Details

Summary
  1. For whatever reason I'm getting this fatal error from portlint:

    > FATAL: Makefile: license specified is BSD3CLAUSE OTHERBSDLIKELICENSES, but LICENSE_FILE specified is for BSD3CLAUSE.

    Why do I get it?
  1. This change has to be committed after D17524 because the ${WRKSRC}/LICENSES file was added in 8.2.2.

Makefile changes:

  • Switch the main license from BSD2CLAUSE to BSD3CLAUSE
  • Mention other licenses under whose terms this software is distributed
  • Sort variables
Test Plan
  • portlint
  • poudriere fail at the moment due to the missing ${WRKSRC}/LICENSES file

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Oct 12 2018, 1:43 PM
sbruno requested changes to this revision.Oct 12 2018, 2:15 PM

FATAL: Makefile: license specified is BSD3CLAUSE OTHERBSDLIKELICENSES, but LICENSE_FILE specified is for BSD3CLAUSE.

portlint seems to dislike the multiple licenses, and to be fair I don't understand why we are defining multiple entries here.

This revision now requires changes to proceed.Oct 12 2018, 2:15 PM

FATAL: Makefile: license specified is BSD3CLAUSE OTHERBSDLIKELICENSES, but LICENSE_FILE specified is for BSD3CLAUSE.

portlint seems to dislike the multiple licenses, and to be fair I don't understand why we are defining multiple entries here.

I've added OTHERBSDLIKELICENSES because I don't know how to categorize those different licenses: https://github.com/conserver/conserver/blob/master/LICENSES

I don't know what's our approach to the LICENSE variable: is its role to be as precise as possible or to just give a clue of what sort of licenses you should expect if you have a closer look at the licensing.

antoine requested changes to this revision.Oct 12 2018, 3:13 PM
antoine added a subscriber: antoine.

There is no ${WRKSRC}/LICENSES , was this tested?

There is no ${WRKSRC}/LICENSES , was this tested?

Isn't there one if you apply the parent revision first?

There is no ${WRKSRC}/LICENSES , was this tested?

First of all, thanks for testing out my patch :)

You are right, the file is not there (yet) and I'm aware of it. See the description of the revision.

This revision was not accepted when it landed; it landed in state Needs Revision.Oct 12 2018, 4:08 PM
This revision was automatically updated to reflect the committed changes.