Page MenuHomeFreeBSD
Differential D30061 Diff 89116 ports-mgmt/portlint/src/portlint.pl

Changeset View

Standalone View

View Options

ports-mgmt/portlint/src/portlint.pl

Show First 20 Lines • Show All 3,437 Lines • ▼ Show 20 Lines if ($nowrksubdir eq '') {
} }
} }
# check RESTRICTED/NO_CDROM/NO_PACKAGE # check RESTRICTED/NO_CDROM/NO_PACKAGE
print "OK: checking RESTRICTED/NO_CDROM/NO_PACKAGE.\n" if ($verbose); print "OK: checking RESTRICTED/NO_CDROM/NO_PACKAGE.\n" if ($verbose);
my $lps = $makevar{LICENSE_PERMS} // ''; my $lps = $makevar{LICENSE_PERMS} // '';
if ($committer && ($tmp =~ /\n(RESTRICTED|NO_CDROM|NO_PACKAGE)[+?]?=/ || if ($committer && ($tmp =~ /\n(RESTRICTED|NO_CDROM|NO_PACKAGE)[+?]?=/ ||
$lps =~ /\bno-\b/)) { $lps =~ /\bno-\b/)) {
&perror("WARN", $file, -1, "Possible restrictive licensing found. ". &perror("WARN", $file, -1, "Possible restrictive licensing found.");
matUnsubmitted
Done
Inline Actions

This should more like error out saying "DON'T USE THAT, USE THE LICENSE FRAMEWORK" ;-)

mat: This should more like error out saying "DON'T USE THAT, USE THE LICENSE FRAMEWORK" ;-)
marcusUnsubmitted
Done
Inline Actions

I'm not sure we should have a warning like this unless there's an action. Since they wouldn't need to touch LEGAL anymore, I think we can leave this whole block out.

marcus: I'm not sure we should have a warning like this unless there's an action. Since they wouldn't…
"If there are, in fact, limitations to use or distribution, please update ports/LEGAL.");
} }
if ($tmp =~ /\nNO_PACKAGE[+?]?=/) { if ($tmp =~ /\nNO_PACKAGE[+?]?=/) {
&perror("WARN", $file, -1, "NO_PACKAGE is obsolete. It should be ". &perror("WARN", $file, -1, "NO_PACKAGE is obsolete. It should be ".
"replaced with \"LICENSE_PERMS=no-pkg-mirror\""); "replaced with \"LICENSE_PERMS=no-pkg-mirror\"");
} }
# check NO_STAGE # check NO_STAGE
▲ Show 20 Lines • Show All 487 Lines • Show Last 20 Lines