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

Re: [PATCH] subtree/Makefile: Standardize (esp. for packagers)

From
nod.helm@gmail.com <nod.helm@gmail.com>
Date
Apr 26, 2014, 04:56 UTC
Message-ID
<3cb4338e-de68-404d-86dc-70cac7e13606@email.android.com>
In-Reply-To
<CAHYYfeGNDLVxzP6zMyJnSi8GxpQaUKGAkqaLfXbZ=8B1k7vvyQ@mail.gmail.com>
On 24 Apr 2014 11:52, <nod.helm@gmail.com> wrote:
Show 16 quoted lines
>
> From: James Denholm <nod.helm@gmail.com>
>
> contrib/subtree/Makefile is a shambles in regards to it's consistency
> with other makefiles, which makes subtree overly painful to include in
> build scripts.
>
> Two major issues are present:
>
> Firstly, calls to git itself (for $(gitdir) and $(gitver)), making
> building difficult on systems that don't have git.
>
> Secondly, the Makefile uses the variable $(libexecdir) for defining the
> exec path.
>
> (...)

I hate to be that guy, but could I get an opinion on the proposed patch? Is git interested in purely makefile patches, or should I find further improvements to make in subtree and purpose this again with those?

Regards, James Denholm.

Previous: nod.helm@gmail.comNext: Jeff King
Message 2 of 10 in “subtree/Makefile: Standardize (esp. for packagers)”
  1. subtree/Makefile: Standardize (esp. for packagers)nod.helm@gmail.com, Apr 24, 2014
  2. nod.helm@gmail.comApr 26, 2014
  3. Jeff KingApr 26, 2014
  4. James DenholmApr 27, 2014
  5. Jeff KingApr 27, 2014
  6. James DenholmApr 27, 2014
  7. Matthew OgilvieApr 30, 2014
  8. James DenholmMay 3, 2014
  9. Felipe ContrerasMay 3, 2014
  10. James DenholmMay 3, 2014

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.