svn commit: r251404 - in head/lib/msun: . src

David Schultz das at freebsd.org
Wed Jun 5 08:13:09 UTC 2013


On Wed, Jun 05, 2013, Chris Rees wrote:
> On 5 Jun 2013 06:33, "David Schultz" <das at freebsd.org> wrote:
> >
> > Author: das
> > Date: Wed Jun  5 05:33:01 2013
> > New Revision: 251404
> > URL: http://svnweb.freebsd.org/changeset/base/251404
> >
> > Log:
> >   Style fixes.
> >
> >   Submitted by: bde
> >
> > Modified:
> >   head/lib/msun/Makefile
> >   head/lib/msun/src/catrig.c
> >   head/lib/msun/src/catrigf.c
> >   head/lib/msun/src/e_log2.c
> >
> > Modified: head/lib/msun/Makefile
> >
> ==============================================================================
> > --- head/lib/msun/Makefile      Wed Jun  5 01:22:59 2013        (r251403)
> > +++ head/lib/msun/Makefile      Wed Jun  5 05:33:01 2013        (r251404)
> > @@ -21,6 +21,10 @@ ARCH_SUBDIR= ${MACHINE_CPUARCH}
> >  .include "${ARCH_SUBDIR}/Makefile.inc"
> >
> >  .PATH: ${.CURDIR}/${ARCH_SUBDIR}
> > +.if ${MACHINE_CPUARCH} == "i386" || ${MACHINE_CPUARCH} == "amd64"
> > +.PATH: ${.CURDIR}/x86
> > +CFLAGS+=       -I${.CURDIR}/x86
> > +.endif
> 
> Since you're doing style fixes, it is worth noting that quoting strings in
> Makefiles is nearly always incorrect, and only works in our make because of
> an extension.

Thanks -- I didn't know that, and apparently neither did whoever
wrote that line originally.  But quoted strings are pervasive in
our makefiles; if there's a good reason to fix it, perhaps you
could go through and clean them up. :)


More information about the svn-src-head mailing list