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

Re: [PATCH] Makefile(s): avoid recipe prefix in conditional statements

From
Paul Smith <psmith@gnu.org>
Date
Apr 9, 2024, 20:41 UTC
Message-ID
<2b392b30614abd9a110515448853aa43eac42d8b.camel@gnu.org>
In-Reply-To
<xmqqh6gb5szm.fsf@gitster.g>
On Mon, 2024-04-08 at 16:34 -0700, Junio C Hamano wrote:
Show 5 quoted lines
> I am not sure what benefits are gained by the existing users with
> this change to ease fixing some parser bug (I didn't bother to see
> your bug tracker) so I cannot judge if the benefit outweighs the
> cost of them all having to scramble and adjust to the new world
> order.

Just to point out that it's actually unusual (in my experience) for makefiles to use TAB as indentation. There are so many situations where this comes back to bite you (see my other emails) that people simply don't do it.

I realize Git and Linux (using Linus's coding style) are committed to each-TAB-is-one-indentation-level, even insofar as using it inside makefiles not just C code, but they are outliers IME.

So I'm not sure I'm ready to concede (yet) that the "cost of them all" is actually very wide. Certainly two projects as popular as Git and Linux with this problem are very concerning.

Previous: Junio C HamanoNext: Junio C Hamano
Message 6 of 13 in “Makefiles are broken as of GNU Make commit 07fcee35f058a876447c8a021f9eb1943f902534”
  1. Dario GjorgjevskiApr 8, 2024
  2. Makefile(s): avoid recipe prefix in conditional statementsTaylor Blau, Apr 8, 2024
  3. Junio C HamanoApr 8, 2024
  4. Paul SmithApr 8, 2024
  5. Junio C HamanoApr 8, 2024
  6. Paul SmithApr 9, 2024
  7. Junio C HamanoApr 8, 2024
  8. Paul SmithApr 9, 2024
  9. Junio C HamanoApr 9, 2024
  10. Jeff KingApr 9, 2024
  11. Jeff KingApr 9, 2024
  12. Paul SmithApr 9, 2024
  13. Junio C HamanoApr 8, 2024

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.