On 2013/03/22 17:10, Brian Callahan wrote:
> This patch does not include sthen's nitems patch from last month, but willing 
> to discuss it.

Hold off on the nitems patch, haven't checked but this is likely
in the big headers cleanup diff.

> -SHARED_LIBS= fltk            7.0 \
> -             fltk_forms      7.0 \
> -             fltk_gl         7.0 \
> -             fltk_images     7.0
> +SHARED_LIBS= fltk            7.1 \
> +             fltk_forms      7.1 \
> +             fltk_gl         7.1 \
> +             fltk_images     7.1

fltk bump is correct, but there's no ABI change in fltk_forms,
fltk_gl, fltk_images

> Index: patches/patch-configure
> ===================================================================
> RCS file: patches/patch-configure
> diff -N patches/patch-configure

It's better to patch the autoconf input files as was done in the
previous version, it reduces chance of problems if a future upstream
version moves to a newer autoconf.

> -@@ -1147,7 +1147,7 @@ if test -n "$GCC"; then
> -         if test "x$with_optim" != x; then
> -         OPTIM="$with_optim $OPTIM"
> -     else
> --            OPTIM="-Os $OPTIM"
> -+            : # OPTIM="-Os $OPTIM"
> -     fi

so now it builds like this,

c++ -I..     -Os -Wall -Wunused -Wno-format-y2k  -fPIC -fno-exceptions 
-fno-strict-aliasing -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_THREAD_SAFE 
-D_REENTRANT -I/usr/X11R6/include/freetype2 -I/usr/X11R6/include 
-I/usr/local/include -I/usr/local/include/libpng -I/usr/X11R6/include/freetype2 
-I/usr/X11R6/include -O2 -pipe  -I/usr/local/include  -I/usr/X11R6/include 
-DFL_LIBRARY -c Fl_Gl_Device_Plugin.cxx -o Fl_Gl_Device_Plugin.o

I forget the precedence order of -O options, but probably better to keep
this in the patch.

Reply via email to