cvs commit: ports/ports-mgmt/portlint Makefile
ports/ports-mgmt/portlint/src portlint.pl
Gerald Pfeifer
gerald at pfeifer.com
Sun Jan 18 19:42:51 PST 2009
Hi Marcus,
On Sun, 18 Jan 2009, Joe Marcus Clarke wrote:
> Requested by: Dominic Fandrey <kamikaze at bsdforen.de> [1]
> gerlad [3]
who is that gerlad guy? :-P
Seriously, thanks a lot for the fast reaction to my suggestion. I never
would have come up with this code myself, but now that it is there, I
managed to debug a problem and make some other fixes. (Checking for
FCFLAGS instead of FFLAGS was a specification problem in my original
mail, sorry about that.)
Gerald
Commit log:
Check for CONFIGURE_ENV+=, too. Warn about FFLAGS instead of FCFLAGS
being passed to CONFIGURE_ENV and rework the associated error message.
--- /usr/local/bin/portlint 2009-01-18 20:24:16.000000000 +0100
+++ x 2009-01-19 04:37:47.132874000 +0100
@@ -1894,7 +1894,7 @@
#
# whole file: CONFIGURE_ENV
#
- if ($whole =~ /\nCONFIGURE_ENV[?:]?=\s*([^\\\n]+(\\\n[^\\\n]+)*)/) {
+ if ($whole =~ /\nCONFIGURE_ENV[?:+]?=\s*([^\\\n]+(\\\n[^\\\n]+)*)/) {
my $configure_env = $1;
my $cflags = undef;
my $cxxflags = undef;
@@ -1937,10 +1937,10 @@
if ($configure_env =~ /(FC)=/ ||
$configure_env =~ /(F77)=/ ||
- $configure_env =~ /(FCFLAGS)=/) {
- &perror("FATAL", $file, -1, "The Fortran flag, $1 is already ".
+ $configure_env =~ /(FFLAGS)=/) {
+ &perror("FATAL", $file, -1, "$1 is already ".
"passed in CONFIGURE_ENV via bsd.gcc.mk. If you need to ".
- "override the the default value, alter $1 in the Makefile ".
+ "override the default value, alter $1 in the Makefile ".
"instead with $1=...");
}
}
More information about the cvs-all
mailing list