[Bug libffi/52223] [4.5/4.6/4.7 Regression] libffi's man page install breaks with multilibs and overridden mandir

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52223

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jakub at gcc dot gnu.org

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-02-22 15:03:34 UTC ---
(In reply to comment #4)
> (In reply to comment #3)
> > This ought to work.  Does applying the fix I discuss in 
> > <http://gcc.gnu.org/ml/gcc-patches/2011-09/msg01187.html> - setting 
> > FLAGS_TO_PASS = $(AM_MAKEFLAGS) in Makefile.am - for libffi help here?
> 
> Yes, with that applied overriding mandir at install does work.
> 
> I had to adapt the patch a little, this is what I tested:
> 
> --- gcc-4.7-20120211/libffi/Makefile.am.~1~
> +++ gcc-4.7-20120211/libffi/Makefile.am
> @@ -76,6 +76,9 @@ AM_MAKEFLAGS = \
>         "RANLIB=$(RANLIB)" \
>         "DESTDIR=$(DESTDIR)"
> 
> +# Subdir rules rely on $(FLAGS_TO_PASS)
> +FLAGS_TO_PASS = $(AM_MAKEFLAGS)
> +
>  MAKEOVERRIDES=
> 
>  toolexeclib_LTLIBRARIES = libffi.la
> --- gcc-4.7-20120211/libffi/Makefile.in.~1~
> +++ gcc-4.7-20120211/libffi/Makefile.in
> @@ -455,6 +455,8 @@ AM_MAKEFLAGS = \
>         "RANLIB=$(RANLIB)" \
>         "DESTDIR=$(DESTDIR)"
> 
> +# Subdir rules rely on $(FLAGS_TO_PASS)
> +FLAGS_TO_PASS = $(AM_MAKEFLAGS)
>  MAKEOVERRIDES = 
>  toolexeclib_LTLIBRARIES = libffi.la
>  noinst_LTLIBRARIES = libffi_convenience.la

Are you going to post this to gcc-patches?  Please CC the build maintainers.