[Ffmpeg-devel] [MacIntel] Testers welcome!

[Diego Biurrun]

On Sun, Aug 13, 2006 at 11:27:04AM +0200, Ulrich von Zadow wrote:
> Diego Biurrun wrote:
> >On Sat, Aug 12, 2006 at 08:29:34PM +0200, Ulrich von Zadow wrote:
> >>Guillaume POIRIER wrote:
> >>>The last part of patches to add MacIntel support has been committed.
> >>>Could the ppl who have access to any of these machines confirm that it 
> >>>works?
> >>Works here using plain ./configure and ./configure --disable-shared 
> >>--disable-debug --disable-encoders.
> >>
> >>It is still nessesary to run ranlib on the static libraries after make 
> >>install, though, so make install is missing a step.
> >
> >Hmmm?  ranlib is run on the static libraries, look at the rule in the
> >Makefile that creates them:
> >
> >$(LIB): $(STATIC_OBJS)
> >        rm -f $@
> >        $(AR) rc $@ $^ $(EXTRAOBJS)
> >        $(RANLIB) $@
> 
> The error is '../lib/libavformat.a is out of date; rerun ranlib(1) 
> (can't load from it)'. Running ranlib after install solves the issue. 
> This is Mac-specific.

Please try the attached patch, if you can test it on PPC as well as
Intel that would be a plus.

Diego
-------------- next part --------------
Index: configure
===================================================================
--- configure	(revision 5991)
+++ configure	(working copy)
@@ -580,6 +580,7 @@
 SLIBNAME_WITH_FULLVERSION='$(SLIBPREF)$(NAME).$(LIBVERSION)$(SLIBSUF)'
 SLIBNAME_WITH_MAJOR='$(SLIBPREF)$(NAME).$(LIBMAJOR)$(SLIBSUF)'
 FFSERVERLDFLAGS=-Wl,-bind_at_load
+LIB_INSTALL_EXTRA_CMD="$ranlib"
 ;;
 MINGW32*)
 # Note: the rest of the mingw32 config is done afterwards as mingw32
@@ -1761,6 +1762,7 @@
   echo "SLIBNAME_WITH_VERSION=${SLIBNAME_WITH_VERSION}" >> config.mak
   echo "SLIBNAME_WITH_MAJOR=${SLIBNAME_WITH_MAJOR}" >> config.mak
   echo "SLIB_EXTRA_CMD=${SLIB_EXTRA_CMD}" >> config.mak
+  echo "LIB_INSTALL_EXTRA_CMD=${LIB_INSTALL_EXTRA_CMD}" >> config.mak
 fi
 echo "EXTRALIBS=$extralibs" >> config.mak
 
Index: common.mak
===================================================================
--- common.mak	(revision 5991)
+++ common.mak	(working copy)
@@ -69,6 +69,7 @@
 install-lib-static: $(LIB)
 	install -d "$(libdir)"
 	install -m 644 $(LIB) "$(libdir)"
+	$(LIB_INSTALL_EXTRA_CMD)
 
 install-headers:
 	install -d "$(incdir)"