Erik de Castro Lopo
2005-Jan-29 21:32 UTC
[Flac-dev] Patch : fix configure.in and Makefile.am problems.
Note : I'm subscribed to the list. Please don't CC replies to me. Hi all, I am trying to compiler current CV head on a PowerPC G3 Debain system. Debian test has automake 1.9 which is a little more stringent about errors than 1.8. The following patch has the following fixes. 0) Remove AC_CANNONIAL_HOST because it has already been invoked earlier 1) Make sure FLaC__HAS_AS__TEMPORARILY_DISABLED and FLaC__HAS_GAS__TEMPORARILY_DISABLED have been defined on PPC. 2) Fix all instances of CFLAGS/CXXFLAGS being set in Makefile.am. These two variables are automaticall set in Makefile.in. If you need extra flag foo in Makefile.am, the correct way to do it is AM_CFLAGS/AM_CXXFLAGS. Probably more to come as this thing still doesn't compile. Cheers, Erik Index: configure.in ==================================================================RCS file: /cvsroot/flac/flac/configure.in,v retrieving revision 1.105 diff -u -r1.105 configure.in --- configure.in 27 Jan 2005 03:59:55 -0000 1.105 +++ configure.in 30 Jan 2005 05:23:53 -0000 @@ -42,7 +42,6 @@ dnl AC_CHECK_FUNCS(getopt_long , , [LIBOBJS="$LIBOBJS getopt.o getopt1.o"] ) AC_CHECK_FUNCS(getopt_long, [], []) -AC_CANONICAL_HOST case "$host_cpu" in i*86) cpu_ia32=true ; AC_DEFINE(FLAC__CPU_IA32) ;; powerpc) cpu_ppc=true ; AC_DEFINE(FLAC__CPU_PPC) ;; @@ -209,6 +208,9 @@ CFLAGS="$OUR_CFLAGS_HEAD $CFLAGS" CXXFLAGS="$OUR_CFLAGS_HEAD $CXXFLAGS" +AM_CONDITIONAL(FLaC__HAS_AS__TEMPORARILY_DISABLED, test "yes" = "no") +AM_CONDITIONAL(FLaC__HAS_GAS__TEMPORARILY_DISABLED, test "yes" = "no") + AM_CONFIG_HEADER(config.h) AH_TEMPLATE(FLAC__ALIGN_MALLOC_DATA, [define to align allocated memory on 32-byte boundaries]) AH_TEMPLATE(FLAC__CPU_IA32, [define if building for ia32/i386]) Index: src/flac/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/flac/Makefile.am,v retrieving revision 1.33 diff -u -r1.33 Makefile.am --- src/flac/Makefile.am 25 Jan 2005 04:18:27 -0000 1.33 +++ src/flac/Makefile.am 30 Jan 2005 05:23:53 -0000 @@ -16,7 +16,7 @@ # Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. bin_PROGRAMS = flac -CFLAGS = @CFLAGS@ @OGG_CFLAGS@ +AM_CFLAGS = @OGG_CFLAGS@ if FLaC__HAS_OGG NEED_OGGFLAC_LIB = $(top_builddir)/src/libOggFLAC/libOggFLAC.la Index: src/libFLAC/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/libFLAC/Makefile.am,v retrieving revision 1.57 diff -u -r1.57 Makefile.am --- src/libFLAC/Makefile.am 29 Jan 2005 06:10:58 -0000 1.57 +++ src/libFLAC/Makefile.am 30 Jan 2005 05:23:53 -0000 @@ -43,7 +43,7 @@ CPUCFLAGS = -maltivec -mabi=altivec -force_cpusubtype_ALL -DFLAC__NO_ASM endif endif -CFLAGS = @CFLAGS@ $(DEBUGCFLAGS) $(CPUCFLAGS) +AM_CFLAGS = $(DEBUGCFLAGS) $(CPUCFLAGS) if FLaC__NO_ASM else Index: src/libFLAC++/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/libFLAC++/Makefile.am,v retrieving revision 1.23 diff -u -r1.23 Makefile.am --- src/libFLAC++/Makefile.am 25 Jan 2005 04:18:35 -0000 1.23 +++ src/libFLAC++/Makefile.am 30 Jan 2005 05:23:54 -0000 @@ -29,7 +29,6 @@ # SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. lib_LTLIBRARIES = libFLAC++.la -CXXFLAGS = @CXXFLAGS@ m4datadir = $(datadir)/aclocal m4data_DATA = libFLAC++.m4 Index: src/libOggFLAC/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/libOggFLAC/Makefile.am,v retrieving revision 1.23 diff -u -r1.23 Makefile.am --- src/libOggFLAC/Makefile.am 25 Jan 2005 04:18:35 -0000 1.23 +++ src/libOggFLAC/Makefile.am 30 Jan 2005 05:23:54 -0000 @@ -32,7 +32,7 @@ if DEBUG DEBUGCFLAGS = endif -CFLAGS = @CFLAGS@ @OGG_CFLAGS@ $(DEBUGCFLAGS) +AM_CFLAGS = @OGG_CFLAGS@ $(DEBUGCFLAGS) SUBDIRS = include . Index: src/libOggFLAC++/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/libOggFLAC++/Makefile.am,v retrieving revision 1.17 diff -u -r1.17 Makefile.am --- src/libOggFLAC++/Makefile.am 25 Jan 2005 04:18:38 -0000 1.17 +++ src/libOggFLAC++/Makefile.am 30 Jan 2005 05:23:55 -0000 @@ -29,7 +29,6 @@ # SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. lib_LTLIBRARIES = libOggFLAC++.la -CXXFLAGS = @CXXFLAGS@ m4datadir = $(datadir)/aclocal m4data_DATA = libOggFLAC++.m4 Index: src/metaflac/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/metaflac/Makefile.am,v retrieving revision 1.19 diff -u -r1.19 Makefile.am --- src/metaflac/Makefile.am 25 Jan 2005 04:18:38 -0000 1.19 +++ src/metaflac/Makefile.am 30 Jan 2005 05:23:55 -0000 @@ -16,7 +16,6 @@ # Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. bin_PROGRAMS = metaflac -CFLAGS = @CFLAGS@ EXTRA_DIST = \ Makefile.lite \ Index: src/plugin_xmms/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/plugin_xmms/Makefile.am,v retrieving revision 1.28 diff -u -r1.28 Makefile.am --- src/plugin_xmms/Makefile.am 25 Jan 2005 04:18:40 -0000 1.28 +++ src/plugin_xmms/Makefile.am 30 Jan 2005 05:23:55 -0000 @@ -29,7 +29,7 @@ plugin.h \ tag.h -CFLAGS = @CFLAGS@ @XMMS_CFLAGS@ +AM_CFLAGS = @XMMS_CFLAGS@ INCLUDES = -I$(top_srcdir)/src if FLaC__INSTALL_XMMS_PLUGIN_LOCALLY xmmsinputplugindir = $(HOME)/.xmms/Plugins Index: src/test_grabbag/cuesheet/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/test_grabbag/cuesheet/Makefile.am,v retrieving revision 1.8 diff -u -r1.8 Makefile.am --- src/test_grabbag/cuesheet/Makefile.am 25 Jan 2005 04:18:41 -0000 1.8 +++ src/test_grabbag/cuesheet/Makefile.am 30 Jan 2005 05:23:55 -0000 @@ -19,8 +19,6 @@ Makefile.lite \ test_cuesheet.dsp -CFLAGS = @CFLAGS@ - noinst_PROGRAMS = test_cuesheet test_cuesheet_SOURCES = \ main.c Index: src/test_libFLAC/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/test_libFLAC/Makefile.am,v retrieving revision 1.16 diff -u -r1.16 Makefile.am --- src/test_libFLAC/Makefile.am 25 Jan 2005 04:18:41 -0000 1.16 +++ src/test_libFLAC/Makefile.am 30 Jan 2005 05:23:56 -0000 @@ -19,7 +19,6 @@ Makefile.lite \ test_libFLAC.dsp -CFLAGS = @CFLAGS@ INCLUDES = -I$(top_srcdir)/src/libFLAC/include noinst_PROGRAMS = test_libFLAC Index: src/test_libFLAC++/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/test_libFLAC++/Makefile.am,v retrieving revision 1.17 diff -u -r1.17 Makefile.am --- src/test_libFLAC++/Makefile.am 25 Jan 2005 04:18:42 -0000 1.17 +++ src/test_libFLAC++/Makefile.am 30 Jan 2005 05:23:56 -0000 @@ -15,8 +15,6 @@ # along with this program; if not, write to the Free Software # Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. -CFLAGS = @CFLAGS@ - EXTRA_DIST = \ Makefile.lite \ README \ Index: src/test_libOggFLAC/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/test_libOggFLAC/Makefile.am,v retrieving revision 1.15 diff -u -r1.15 Makefile.am --- src/test_libOggFLAC/Makefile.am 25 Jan 2005 04:18:43 -0000 1.15 +++ src/test_libOggFLAC/Makefile.am 30 Jan 2005 05:23:56 -0000 @@ -19,7 +19,7 @@ Makefile.lite \ test_libOggFLAC.dsp -CFLAGS = @CFLAGS@ @OGG_CFLAGS@ +AM_CFLAGS = @OGG_CFLAGS@ INCLUDES = noinst_PROGRAMS = test_libOggFLAC Index: src/test_libOggFLAC++/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/test_libOggFLAC++/Makefile.am,v retrieving revision 1.15 diff -u -r1.15 Makefile.am --- src/test_libOggFLAC++/Makefile.am 25 Jan 2005 04:18:44 -0000 1.15 +++ src/test_libOggFLAC++/Makefile.am 30 Jan 2005 05:23:56 -0000 @@ -20,7 +20,7 @@ README \ test_libOggFLAC++.dsp -CFLAGS = @CFLAGS@ @OGG_CFLAGS@ +AM_CFLAGS = @OGG_CFLAGS@ noinst_PROGRAMS = test_libOggFLAC++ test_libOggFLAC___LDADD = \ Index: src/test_seeking/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/test_seeking/Makefile.am,v retrieving revision 1.3 diff -u -r1.3 Makefile.am --- src/test_seeking/Makefile.am 25 Jan 2005 04:18:44 -0000 1.3 +++ src/test_seeking/Makefile.am 30 Jan 2005 05:23:56 -0000 @@ -19,7 +19,7 @@ Makefile.lite \ test_seeking.dsp -CFLAGS = @CFLAGS@ @OGG_CFLAGS@ +AM_CFLAGS = @OGG_CFLAGS@ INCLUDES = if FLaC__HAS_OGG Index: src/test_streams/Makefile.am ==================================================================RCS file: /cvsroot/flac/flac/src/test_streams/Makefile.am,v retrieving revision 1.13 diff -u -r1.13 Makefile.am --- src/test_streams/Makefile.am 25 Jan 2005 04:18:44 -0000 1.13 +++ src/test_streams/Makefile.am 30 Jan 2005 05:23:56 -0000 @@ -19,8 +19,6 @@ Makefile.lite \ test_streams.dsp -CFLAGS = @CFLAGS@ - noinst_PROGRAMS = test_streams test_streams_SOURCES = \ main.c -- +-----------------------------------------------------------+ Erik de Castro Lopo nospam@mega-nerd.com (Yes it's valid) +-----------------------------------------------------------+ "C++ is an atrocity, the bletcherous scab of the computing world, responsible for more buffer overflows, more security breaches, more blue screens of death, more mysterious failures than any other computer language in the history of the planet Earth." -- Eric Lee Green
Josh Coalson
2005-Jan-30 10:45 UTC
[Flac-dev] Patch : fix configure.in and Makefile.am problems.
--- Erik de Castro Lopo <erikd-flac@mega-nerd.com> wrote:> Hi all, > > I am trying to compiler current CV head on a PowerPC G3 Debain > system. > > Debian test has automake 1.9 which is a little more stringent about > errors than 1.8. The following patch has the following fixes. > > 0) Remove AC_CANNONIAL_HOST because it has already been invoked > earlier > 1) Make sure FLaC__HAS_AS__TEMPORARILY_DISABLED and > FLaC__HAS_GAS__TEMPORARILY_DISABLED have been defined on PPC. > 2) Fix all instances of CFLAGS/CXXFLAGS being set in Makefile.am. > These two variables are automaticall set in Makefile.in. If you > need extra flag foo in Makefile.am, the correct way to do it is > AM_CFLAGS/AM_CXXFLAGS. > > Probably more to come as this thing still doesn't compile.OK, thanks, checked in. once linux-ppc is ironed out the release should be ready to go. Josh __________________________________________________ Do You Yahoo!? Tired of spam? Yahoo! Mail has the best spam protection around http://mail.yahoo.com
Erik de Castro Lopo
2005-Jan-30 12:12 UTC
[Flac-dev] Patch : fix configure.in and Makefile.am problems.
On Sun, 30 Jan 2005 10:45:05 -0800 (PST) Josh Coalson <xflac@yahoo.com> wrote:> OK, thanks, checked in. once linux-ppc is ironed out the > release should be ready to go.Long way to go yet. Mine is a G3 PowerPC so it doesn't have Altivec. I've tried --disable-assembler but there's still something screwed up int the Makefile.am. Erik -- +-----------------------------------------------------------+ Erik de Castro Lopo nospam@mega-nerd.com (Yes it's valid) +-----------------------------------------------------------+ "Any sufficiently complicated C or Fortran program contains an ad-hoc, informally-specified, bug-ridden, slow implementation of half of CommonLisp." -- Greenspuns Tenth Rule Of Programming