Re: [PATCH] make: add INSTALL_STRIP variable

From: Bagas Sanjaya <bagasdotme@gmail.com>
Date: Aug 21, 2021, 08:24 UTC
Message-ID: <70e609f5-bf40-658a-a021-ab7a0ad4baa1@gmail.com>
In-Reply-To: <YR+Tp2AGeeKyRKoC@danh.dev>

On 20/08/21 18.36, Đoàn Trần Công Danh wrote:

Show 12 quoted lines

> I believe it's better to write like this:
> 
> ----- 8< ------
> ifdef INSTALL_STRIP
> install: strip
> endif
> 
> install: all
> 	....
> ---- >8-------
> 
> IOW, install depends on strip, not install invoke strip.

Oh, I missed that.

> I think it would work better for:
> 
> 	make install strip
>

Wouldn't it install unstripped binaries to the prefix then stripping them in the build directory?

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