On Wed, Jun 05, 2013, Chris Rees wrote: > On 5 Jun 2013 06:33, "David Schultz" <d...@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. :) _______________________________________________ svn-src-head@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
