Changeset View
Changeset View
Standalone View
Standalone View
ports-mgmt/portlint/src/portlint.pl
Show First 20 Lines • Show All 3,448 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."); | ||||
mat: This should more like error out saying "DON'T USE THAT, USE THE LICENSE FRAMEWORK" ;-) | |||||
Done Inline ActionsI'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 |
This should more like error out saying "DON'T USE THAT, USE THE LICENSE FRAMEWORK" ;-)