conf/96834: [PATCH] Update Fortran related variables in sys.mk

[Ruslan Ermilov]

The following reply was made to PR conf/96834; it has been noted by GNATS.

From: Ruslan Ermilov <ru at FreeBSD.org>
To: "Steven G. Kargl" <kargl at troutmask.apl.washington.edu>
Cc: bug-followup at FreeBSD.org
Subject: Re: conf/96834: [PATCH] Update Fortran related variables in sys.mk
Date: Sat, 6 May 2006 16:50:34 +0300

 On Fri, May 05, 2006 at 11:19:39AM -0700, Steven G. Kargl wrote:
 > 
 > The sys.mk file currently defines "FFLAGS = -O" for the default
 > Fortran compilation options while "CFLAGS = -O2 -pipe".
 > 
 I don't have an opinion about this, since I don't know what
 -O2 does exactly when applied to f77(1).
 
 > Futhermore,
 > the .if (POSIX) conditional is unnecessary.  FreeBSD has never had a
 > fort77 program and it never will.
 > 	
 This is irrelevant.  POSIX requires that these variables be set
 as follows:
 
 FC=fort77
 FFLAGS=-O 1
 
 So we do.
 
 > The patch removes the POSIX conditional and updates
 > the default Fortran flags.
 > 
 > --- sys.mk.orig	Fri May  5 11:12:58 2006
 > +++ sys.mk	Fri May  5 11:13:46 2006
 > @@ -63,13 +63,8 @@
 >  _+_		?=	+
 >  .endif
 >  
 > -.if defined(%POSIX)
 > -FC		?=	fort77
 > -FFLAGS		?=	-O 1
 > -.else
 >  FC		?=	f77
 > -FFLAGS		?=	-O
 > -.endif
 > +FFLAGS		?=	-O2 -pipe
 >  EFLAGS		?=
 >  
 >  INSTALL		?=	install
 
 -- 
 Ruslan Ermilov
 ru at FreeBSD.org
 FreeBSD committer