On Fri, 29 Oct 2010, Paolo Bonzini wrote:
On 10/29/2010 08:14 PM, Andi Kleen wrote:
+ifeq ($(enable_plugin),yes)
+ $(INSTALL_SCRIPT) lto/gcc-ar $(DESTDIR)$(bindir)/$(AR_INSTALL_NAME)
+ $(INSTALL_SCRIPT) lto/gcc-ranlib
$(DESTDIR)$(bindir)/$(RANLIB_INSTALL_NAME)
+endif
In addition to what pointed out by Joseph (including the preference for a C
implementation), this is also wrong for cross-compiling. It needs to be
installed as $(DESTDIR)$(gcc_tooldir)/bin/$(AR_INSTALL_NAME) in that case.
I thought the point was to install a binary for users to use, not for
internal use by other programs - that is, something in bindir, which is
the place for user binaries, not gcc_tooldir/bin, which is the place for
internal binaries. AR_INSTALL_NAME is the transformed name, suitable for
use in bindir.