git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] make: add INSTALL_STRIP variable

From
Bagas Sanjaya <bagasdotme@gmail.com>
Date
Aug 24, 2021, 09:39 UTC
Message-ID
<6e0ef7f2-3dd0-09e4-5a1d-7e59b979d624@gmail.com>
In-Reply-To
<xmqq5yvwrw8s.fsf@gitster.g>
On 23/08/21 22.55, Junio C Hamano wrote:
Show 12 quoted lines
> Perhaps.  One bad thing with the current "strip" arrangement is that
> it is done in the built directory, and because "make install" would
> blindly install whatever in the built directory, if you truly care
> that you install unstripped binaries, you need to see if they are
> stripped and rebuild them as needed, because "make strip" may or may
> not have been done.  From that point of view, getting rid of the
> current "make strip" and introducing either "make strip-installed"
> ("we've installed things earlier---go strip them") or "make
> install-stripped" ("we've built (or if we haven't please build them
> first), now install them and strip them in the installed directory")
> may make more sense.  And for that, any idea that came up in this
> discussion that relies on the current "strip" target would not help.

But often the installed directory (install prefix) is owned by root, so one has to `sudo make install-strip`, right?

-- 
An old man doll... just what I always wanted! - Clara
Previous: Junio C HamanoNext: Johannes Schindelin
Message 6 of 10 in “make: add INSTALL_STRIP variable”
  1. make: add INSTALL_STRIP variableBagas Sanjaya, Aug 20, 2021
  2. Đoàn Trần Công DanhAug 20, 2021
  3. Junio C HamanoAug 20, 2021
  4. Đoàn Trần Công DanhAug 21, 2021
  5. Junio C HamanoAug 23, 2021
  6. Bagas SanjayaAug 24, 2021
  7. Johannes SchindelinAug 24, 2021
  8. Junio C HamanoAug 24, 2021
  9. Bagas SanjayaAug 21, 2021
  10. Đoàn Trần Công DanhAug 21, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.