Re: [dwm] [dmenu] OPTFLAGS in config.mk + patch

Hi,

Honestly, I think introducing OPTFLAGS is kind of arbitrary, even if
it's done in existing software (well existing software uses autohell
quite a lot). In my opinion CFLAGS is the right variable to include
compiler flags, and -Ox is one of them.

Kind regards,
Anselm

2009/4/8 Jan Blazek <appolito2_AT_gmail.com>:
> I would like to make package of dmenu for Fedora and in the review process
> someone (Till Maas) suggested it would be better to have ability to pass gcc
> optimization flags to make on the command line. [1]
>
> Best regards,
> Jan Blazek
>
> PS: Patch attached.
>
> [1] https://bugzilla.redhat.com/show_bug.cgi?id=485638#c3
>
> --- dmenu-3.9/config.mk.optflags        2008-09-09 15:45:00.000000000 -0400
> +++ dmenu-3.9/config.mk 2009-04-02 15:26:41.000000000 -0400
> @@ -20,7 +20,8 @@ LIBS = -L/usr/lib -lc -L${X11LIB} -lX11
>
>  # flags
>  CPPFLAGS = -DVERSION=\"${VERSION}\" ${XINERAMAFLAGS}
> -CFLAGS = -std=c99 -pedantic -Wall -Os ${INCS} ${CPPFLAGS}
> +OPTFLAGS = -Os
> +CFLAGS = -std=c99 -pedantic -Wall ${OPTFLAGS} ${INCS} ${CPPFLAGS}
>  LDFLAGS = -s ${LIBS}
>
>  # Solaris
Received on Thu Apr 09 2009 - 08:21:06 UTC