Bug 46202 - Makefile doesn't support install-strip
Summary: Makefile doesn't support install-strip
Status: RESOLVED FIXED
Alias: None
Product: gcc
Classification: Unclassified
Component: other (show other bugs)
Version: unknown
: P3 enhancement
Target Milestone: ---
Assignee: Ralf Wildenhues
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-27 21:08 UTC by nightstrike
Modified: 2010-11-20 19:40 UTC (History)
2 users (show)

See Also:
Host:
Target:
Build:
Known to work:
Known to fail:
Last reconfirmed: 2010-11-01 16:41:30

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description nightstrike 2010-10-27 21:08:08 UTC 
The GNU Standards list install-strip as a required makefile target.  This target isn't present, however.  See discussion here:

http://www.sourceware.org/ml/binutils/2010-10/msg00377.html
Comment 1 Ralf Wildenhues 2010-11-02 21:44:12 UTC 
Proposed patch at <http://gcc.gnu.org/ml/gcc-patches/2010-11/msg00192.html>.
Comment 2 Ralf Wildenhues 2010-11-20 19:37:12 UTC 
Author: rwild
Date: Sat Nov 20 19:37:08 2010
New Revision: 166980

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166980
Log:
PR other/46202: implement install-strip.

/:
	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/:
	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.
	(STRIPPROG): New variable, exported if STRIP is set.
	* doc/install.texi (Final install): Minor markup and code style
	fixes.  Document install-strip target.

fixincludes/:
	PR other/46202
	* Makefile.in (install-strip): New phony target.
	(all, check, install): Also mark as phony.

libgcc/:
	PR other/46202
	* Makefile.in (install-strip): New phony target.

libiberty/:
	PR other/46202
	* Makefile.in (install-strip): New phony target.
	(install): Also mark as phony.

gnattools/:
	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/:
	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.

Modified:
    trunk/ChangeLog
    trunk/Makefile.def
    trunk/Makefile.in
    trunk/Makefile.tpl
    trunk/configure
    trunk/configure.ac
    trunk/fixincludes/ChangeLog
    trunk/fixincludes/Makefile.in
    trunk/gcc/ChangeLog
    trunk/gcc/Makefile.in
    trunk/gcc/config/i386/t-cygming
    trunk/gcc/doc/install.texi
    trunk/gnattools/ChangeLog
    trunk/gnattools/Makefile.in
    trunk/libada/ChangeLog
    trunk/libada/Makefile.in
    trunk/libgcc/ChangeLog
    trunk/libgcc/Makefile.in
    trunk/libiberty/ChangeLog
    trunk/libiberty/Makefile.in
Comment 3 Ralf Wildenhues 2010-11-20 19:40:13 UTC 
Fixed in GCC trunk and src CVS.