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

Re: [PATCH v2 0/5] contrib/subtree/Makefile: Standardisation pass

From
Jeff King <peff@peff.net>
Date
May 5, 2014, 22:01 UTC
Message-ID
<20140505220112.GA17610@sigill.intra.peff.net>
In-Reply-To
<69f827ea-0ba2-4ca0-b711-002e1a0010b7@email.android.com>
[fixed David's address in cc list]
On Tue, May 06, 2014 at 07:54:30AM +1000, James Denholm wrote:
> Given that subtree subtree doesn't really generate a lot of discussion,
> would it be advisable to wrap this up (barring further discussion) and send
> it off to Junio rather than waiting for further community consensus?

I do not know if "lack of discussion" is a good reason to consider something in good shape; oftentimes it is a sign that nobody is interested in the area. We usually rely on "area maintainers" to give an OK to the patches if they are not something that the maintainer himself has an interest in.

However, in this case, you did get review, and I think it is pretty easy to see the patches are good even if one does not care about the particular area. So I think they are fine to pass on and apply.

Note also that patches like this are a great place to get started, as they help build trust in a contributor, who can later help out with area maintenance.

-Peff
Previous: James DenholmNext: James Denholm
Message 13 of 14 in “contrib/subtree/Makefile: Standardisation pass”
  1. 0/5 contrib/subtree/Makefile: Standardisation passJames Denholm, May 3, 2014
  2. 1/5 contrib/subtree/Makefile: scrap unused $(gitdir)James Denholm, May 3, 2014
  3. 2/5 contrib/subtree/Makefile: Use GIT-VERSION-FILEJames Denholm, May 3, 2014
  4. 3/5 contrib/subtree/Makefile: s/libexecdir/gitexecdirJames Denholm, May 3, 2014
  5. 4/5 contrib/subtree/Makefile: Doc-gen rules cleanupJames Denholm, May 3, 2014
  6. 5/5 contrib/subtree/Makefile: clean rule cleanupJames Denholm, May 3, 2014
  7. Jeff KingMay 5, 2014
  8. James DenholmMay 5, 2014
  9. Jeff KingMay 5, 2014
  10. James DenholmMay 5, 2014
  11. Jeff KingMay 5, 2014
  12. James DenholmMay 5, 2014
  13. Jeff KingMay 5, 2014
  14. James DenholmMay 6, 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.