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

Re: [PATCH 2/2] ls-files: fix broken --no-empty-directory

From
Miklos Vajna <vmiklos@frugalware.org>
Date
Mar 10, 2009, 22:16 UTC
Message-ID
<20090310221648.GB4371@genesis.frugalware.org>
In-Reply-To
<20090310191111.GA27662@sigill.intra.peff.net>
On Tue, Mar 10, 2009 at 03:11:11PM -0400, Jeff King <peff@peff.net> wrote:
> which is even still a little confusing, as you get "--empty-directory"
> in the usage message. But you would almost never want to use that, as it
> is already the default.

Exactly, that's why I suggested the usage of PARSE_OPT_NONEG, which would avoid a new no-op option. ;-)

But I'm fine with the above patch as well, in case having the "no-" prefix in an option name is considered as an improper negation.

Previous: Jeff King
Message 6 of 6 in “fix ls-files parseopt regression”
  1. 0/2 fix ls-files parseopt regressionJeff King, Mar 8, 2009
  2. 1/2 t3000: use test_cmp instead of diffJeff King, Mar 8, 2009
  3. 2/2 ls-files: fix broken --no-empty-directoryJeff King, Mar 8, 2009
  4. Miklos VajnaMar 8, 2009
  5. Jeff KingMar 10, 2009
  6. Miklos VajnaMar 10, 2009

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.