Hello,

Here is a patch for backend/Makefile.am. I can do similar patches for other
Makefile.am if you agree with the approach.

* use AM_CPPFLAGS instead of the obsolete INCLUDES
* using AM_LDFLAGS for library link switchs seems very confusing to me, for
  2 reasons: because it isn't used for the only link done in that file
  (maybe because there is also a libgnashbackend_la_LDFLAGS variable) and 
  also because it is not LDFLAGS-like, but LIBS-like. So I propose renaming
  it libgnashbackend_LIBS.

--
Pat
Index: Makefile.am
===================================================================
RCS file: /sources/gnash/gnash/backend/Makefile.am,v
retrieving revision 1.44.2.3
diff -u -3 -p -r1.44.2.3 Makefile.am
--- Makefile.am 11 Nov 2006 12:33:17 -0000      1.44.2.3
+++ Makefile.am 13 Nov 2006 15:42:29 -0000
@@ -20,11 +20,11 @@
 
 AUTOMAKE_OPTIONS =
 
-AM_CPPFLAGS = # -Wall
+#AM_CPPFLAGS = # -Wall
 
 ## WARNING: make sure GLIB_LIBS appears first
 ## See: http://lists.gnu.org/archive/html/gnash-dev/2006-07/msg00076.html
-AM_LDFLAGS =  \
+libgnashbackend_LIBS =  \
        $(GLIB_LIBS) \
        $(JPEG_LIBS) \
        $(PNG_LIBS) \
@@ -39,7 +39,7 @@ AM_LDFLAGS =  \
        $(PTHREAD_LIBS) \
        $(NULL)
 
-INCLUDES = -I.. \
+AM_CPPFLAGS = -I.. \
         -I$(top_srcdir) \
         -I$(top_srcdir)/server \
         -I$(top_srcdir)/server/parser \
@@ -69,46 +69,46 @@ lib_LTLIBRARIES = libgnashbackend.la
 
 if USE_RENDERER_CAIRO
 RENDER_SOURCES = render_handler_cairo.cpp render_handler_tri.cpp
-AM_LDFLAGS += $(CAIRO_LIBS)
-INCLUDES += $(CAIRO_CFLAGS)
+libgnashbackend_LIBS += $(CAIRO_LIBS)
+AM_CPPFLAGS += $(CAIRO_CFLAGS)
 endif
 
 if USE_RENDERER_OPENGL
 RENDER_SOURCES = render_handler_ogl.cpp render_handler_tri.cpp
-AM_LDFLAGS += $(OPENGL_LIBS) # $(GLEXT_LIBS)           
-INCLUDES += $(OPENGL_CFLAGS) # $(GLEXT_CFLAGS) 
+libgnashbackend_LIBS += $(OPENGL_LIBS) # $(GLEXT_LIBS)         
+AM_CPPFLAGS += $(OPENGL_CFLAGS) # $(GLEXT_CFLAGS)      
 endif
 
 if USE_SOUND_GST
 SOUND_SOURCES = sound_handler_gst.cpp 
-AM_LDFLAGS += $(GSTREAMER_LIBS) 
-AM_LDFLAGS += $(LIBXML_LIBS) 
-INCLUDES += $(GSTREAMER_CFLAGS)
-INCLUDES += $(LIBXML_CFLAGS)
+libgnashbackend_LIBS += $(GSTREAMER_LIBS) 
+libgnashbackend_LIBS += $(LIBXML_LIBS) 
+AM_CPPFLAGS += $(GSTREAMER_CFLAGS)
+AM_CPPFLAGS += $(LIBXML_CFLAGS)
 endif
 
 if USE_SOUND_SDL
 SOUND_SOURCES = sound_handler_sdl.cpp sound_handler_sdl.h
 if USE_FFMPEG_ENGINE
-AM_LDFLAGS += $(FFMPEG_LIBS)
-INCLUDES += $(FFMPEG_CFLAGS)
+libgnashbackend_LIBS += $(FFMPEG_LIBS)
+AM_CPPFLAGS += $(FFMPEG_CFLAGS)
 endif
 if USE_MAD_ENGINE
-AM_LDFLAGS += $(MAD_LIBS)
-INCLUDES += $(MAD_CFLAGS)
+libgnashbackend_LIBS += $(MAD_LIBS)
+AM_CPPFLAGS += $(MAD_CFLAGS)
 endif
 endif
 
 if USE_RENDERER_AGG
 RENDER_SOURCES = render_handler_tri.cpp render_handler_agg.cpp
-AM_LDFLAGS += $(AGG_LIBS)
-INCLUDES += $(AGG_CFLAGS)
+libgnashbackend_LIBS += $(AGG_LIBS)
+AM_CPPFLAGS += $(AGG_CFLAGS)
 endif
 
 libgnashbackend_la_LIBADD = \
        $(top_builddir)/libbase/libgnashbase.la \
        $(top_builddir)/server/libgnashserver.la  \
-       $(AM_LDFLAGS)
+       $(libgnashbackend_LIBS)
 
 libgnashbackend_la_SOURCES = \
        $(RENDER_SOURCES) \
_______________________________________________
Gnash-dev mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/gnash-dev

Reply via email to