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

Re: [PATCH v2] Makefile: error out invoking strip target

From
Junio C Hamano <gitster@pobox.com>
Date
Nov 26, 2021, 07:29 UTC
Message-ID
<xmqqilwf8je9.fsf@gitster.g>
In-Reply-To
<20211125122607.26602-1-bagasdotme@gmail.com>
Bagas Sanjaya <bagasdotme@gmail.com> writes:
> Now that $INSTALL_STRIP variable can be defined since 3231f41009 (make:
> add INSTALL_STRIP option variable, 2021-09-05), it is redundant to have
> 'strip' target when $INSTALL_STRIP does the job. Error out when invoking
> the target so that users are forced to define the variable instead.

It is not exactly redundant for folks who like to build and use in place without installing.

What is the reason why we might want to eventually remove the "strip" target, making "make strip" an error? I do not quite see much downsides for having just a target with a simple one-liner recipe.

Previous: Bagas SanjayaNext: Bagas Sanjaya
Message 2 of 3 in “Makefile: error out invoking strip target”
  1. Makefile: error out invoking strip targetBagas Sanjaya, Nov 25, 2021
  2. Junio C HamanoNov 26, 2021
  3. Bagas SanjayaNov 26, 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.