Re: make install-strip with binutils

[Ralf Wildenhues <Ralf dot Wildenhues at gmx dot de>]

http://gcc.gnu.org/ml/gcc-patches/2010-11/msg00192.html

* Ralf Wildenhues wrote on Tue, Nov 02, 2010 at 09:08:41PM CET:
> * NightStrike wrote on Wed, Oct 27, 2010 at 11:08:51PM CEST:
> > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46202

> OK to commit to GCC and sync to src?  The patch has src-specific bits in
> etc/.  install-strip will fail for parts of the src tree outside of
> binutils+gdb, but fixing that should be straight-forward.
> 
> I'll gladly split the .PHONY fixlets and the minor gcc/doc cleanups in
> separate patches if you prefer.

Now, as GCC build maintainer, it seems most of this patch is free to
proceed on (still, I'd welcome any review!), and the src bits are
trivial.  The web update has been OKed off-list by Gerald, so the only
remaining bits are in gcc/doc/install.texi.  Is that part of the build
maintainer role too, or OK to go ahead with?

Thanks,
Ralf

> bits only for src:
> 
> etc/ChangeLog:
> 2010-11-01  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
> 
> 	* Makefile.in (install-strip): New target.

> bits for GCC and shared GCC/src bits:
> 
> PR other/46202: implement install-strip.
> 
> ChangeLog:
> 2010-11-01  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
> 
> 	PR other/46202
> 	* configure.ac: Fix just-built in-tree STRIP name to be
> 	binutils/strip-new.
> 	* configure: Regenerate.
> 	* Makefile.def (install-strip-gcc, install-strip-binutils)
> 	(install-strip-opcodes, install-strip-ld, install-strip-itcl)
> 	(install-strip-sid): Mirror dependencies on non-strip variants
> 	of these targets on the respective -strip prerequisites.
> 	* Makefile.tpl (install-strip, install-strip-host)
> 	(install-strip-target): New targets.
> 	(install-strip-[+module+], install-strip-target-[+module+]):
> 	New targets.
> 	* Makefile.in: Regenerate.
> 
> gcc/ChangeLog:
> 2010-11-01  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
> 
> 	PR other/46202
> 	* Makefile.in (install_sh, INSTALL_STRIP_PROGRAM): New
> 	variables.
> 	(AR_FOR_TARGET, RANLIB_FOR_TARGET, STRIP_FOR_TARGET): Fix
> 	shell quoting.
> 	(STRIP_FOR_TARGET): Look for in-tree strip under name strip-new.
> 	(install-strip): New target.
> 	* doc/install.texi (Final install): Minor markup and code style
> 	fixes.  Document install-strip target.
> 
> fixincludes/ChangeLog:
> 2010-11-01  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
> 
> 	PR other/46202
> 	* Makefile.in (install-strip): New phony target.
> 	(all, check, install): Also mark as phony.
> 
> libgcc/ChangeLog:
> 2010-11-01  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
> 
> 	PR other/46202
> 	* Makefile.in (install-strip): New phony target.
> 
> libiberty/ChangeLog:
> 2010-11-01  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
> 
> 	PR other/46202
> 	* Makefile.in (install-strip): New phony target.
> 	(install): Also mark as phony.
> 
> gnattools/ChangeLog:
> 2010-11-01  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
> 
> 	PR other/46202
> 	* Makefile.in (install-strip): New phony target.
> 	(check, installcheck, info, dvi, pdf, html, install)
> 	(install-info, install-pdf, install-html, mostlyclean)
> 	(clean, distclean, maintainer-clean): Mark phony.
> 
> libada/ChangeLog:
> 2010-11-01  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
> 
> 	PR other/46202
> 	* Makefile.in (install-strip): New phony target.
> 	(check, installcheck, info, dvi, pdf, html, install)
> 	(install-info, install-pdf, install-html, mostlyclean)
> 	(clean, distclean, maintainer-clean): Mark phony.

[...]
> --- a/gcc/doc/install.texi
> +++ b/gcc/doc/install.texi
> @@ -2638,7 +2638,7 @@ messages may be automatically processed.
>  
>  Now that GCC has been built (and optionally tested), you can install it with
>  @smallexample
> -cd @var{objdir}; make install
> +cd @var{objdir} && make install
>  @end smallexample
>  
>  We strongly recommend to install into a target directory where there is
> @@ -2674,7 +2674,8 @@ jail can be achieved with the command
>  make DESTDIR=@var{path-to-rootdir} install
>  @end smallexample
>  
> -@noindent where @var{path-to-rootdir} is the absolute path of
> +@noindent
> +where @var{path-to-rootdir} is the absolute path of
>  a directory relative to which all installation paths will be
>  interpreted.  Note that the directory specified by @code{DESTDIR}
>  need not exist yet; it will be created if necessary.
> @@ -2688,6 +2689,12 @@ it will not be created otherwise.  This is regarded as a feature,
>  not as a bug, because it gives slightly more control to the packagers
>  using the @code{DESTDIR} feature.
>  
> +You can let installed programs and libraries be stripped with
> +
> +@smallexample
> +make install-strip
> +@end smallexample
> +
>  If you are bootstrapping a released version of GCC then please
>  quickly review the build status page for your release, available from
>  @uref{http://gcc.gnu.org/buildstat.html}.