Re: [PATCH RESEND v2] make: add install-stripped target

Bagas Sanjaya [off-list ref] writes:

Add the target that install Git with stripped executables.
...
+
+install-stripped: INSTALL_OPTS = -s --strip-program=$(STRIP)

Not everybody's $(INSTALL) supports --strip-program, does it?  

FreeBSD seems to lack the option, and instead uses the $STRIPBIN
environment variable to specify which program to use.

The manual page for macOS [*] does not mention --strip-program or
STRIPBIN.

If the user leaves STRIP as the default (i.e. strip), this target
fails, not because the user does not have 'strip', but because the
'install' used by the user does not know the --strip-program option,
even though there is no need to use the option in such a case.

Given that, I am not sure if we really want "install-stripped"
target.  Adding INSTALL_OPTS make variable that defaults to an empty
string is a good idea, but we probably can and should stop there.

Then users can do any of the following:

	$ make INSTALL_OPTS=-s install

	$ STRIPBIN=my-strip make INSTALL_OPTS=-s install

depending on their system, which sounds sufficient, and is a vast
improvement over the current

	$ make ; make strip ; make install 

I would think.


[Reference]

* https://opensource.apple.com/source/file_cmds/file_cmds-321.40.3/install/install.1.auto.html

On 01/09/21 00.06, Junio C Hamano wrote:

Given that, I am not sure if we really want "install-stripped"
target.  Adding INSTALL_OPTS make variable that defaults to an empty
string is a good idea, but we probably can and should stop there.

Then users can do any of the following:

	$ make INSTALL_OPTS=-s install

	$ STRIPBIN=my-strip make INSTALL_OPTS=-s install

depending on their system, which sounds sufficient, and is a vast
improvement over the current

	$ make ; make strip ; make install

I would think.

If we make $(INSTALL_OPTS) applies to both compiled executables and 
scripts, we have problem that $(INSTALL) -s only works for the former.

-- 
An old man doll... just what I always wanted! - Clara