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

Re: [PATCH 2/2] Revert "rev-parse: remove restrictions on some options"

From
Jeff King <peff@peff.net>
Date
Feb 29, 2016, 21:29 UTC
Message-ID
<20160229212943.GA25342@sigill.intra.peff.net>
In-Reply-To
<xmqqy4a3cti1.fsf@gitster.mtv.corp.google.com>
On Mon, Feb 29, 2016 at 09:32:22AM -0800, Junio C Hamano wrote:
Show 7 quoted lines
> Jeff King <peff@peff.net> writes:
> 
> > IOW, I think my 2/2 should be replaced with this:
> 
> This looks sensible.
> 
> Don't we still want the documentation updates from the previous 2/2?

I don't think so. They were primarily about moving those option blocks to the "these options must come first..." section, which is no longer true.

It also added "you don't have to be in a git repository for these" to that section, but I think that is less important. We could add that individually to each option, I guess.

-Peff
Previous: Junio C HamanoNext: John Keeping
Message 9 of 13 in “fix some rev-parse options in non-repos”
  1. 0/2 fix some rev-parse options in non-reposJeff King, Feb 26, 2016
  2. 1/2 t1515: add tests for rev-parse out-of-repo helpersJeff King, Feb 26, 2016
  3. 2/2 Revert "rev-parse: remove restrictions on some options"Jeff King, Feb 26, 2016
  4. Jeff KingFeb 26, 2016
  5. Junio C HamanoFeb 26, 2016
  6. Jeff KingFeb 27, 2016
  7. Jeff KingFeb 29, 2016
  8. Junio C HamanoFeb 29, 2016
  9. Jeff KingFeb 29, 2016
  10. John KeepingFeb 27, 2016
  11. Jeff KingFeb 29, 2016
  12. Eric SunshineFeb 28, 2016
  13. Jeff KingFeb 29, 2016

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.