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

Re: [PATCH v1] git-p4: Add option to ignore empty commits

From
Luke Diamand <luke@diamand.org>
Date
Oct 24, 2015, 19:07 UTC
Message-ID
<562BD709.1020307@diamand.org>
In-Reply-To
<C5352225-D67D-4ECD-83DB-D5E5A8ED13A7@gmail.com>
On 24/10/15 17:28, Lars Schneider wrote:
>
Show 10 quoted lines
>> Also I have this suspicion that those who do want to use client spec
>> to get a narrowed view into the history would almost always want
>> this "ignore empty" behaviour (I'd even say the current behaviour to
>> leave empty commits by default is a bug).  What are the advantages
>> of keeping empty commits?  If there aren't many, perhaps git-p4
>> should by the default skip empties and require p4.keepEmpty
>> configuration to keep them?
>
> I agree.
> @Luke: What option do you prefer? "git-p4.keepEmptyCommits" or "git-p4.ignoreEmptyCommits" ?
keepEmptyCommits.
Previous: Lars SchneiderNext: Luke Diamand
Message 4 of 8 in “git-p4: Add option to ignore empty commits”
  1. git-p4: Add option to ignore empty commitslarsxschneider@gmail.com, Oct 19, 2015
  2. Junio C HamanoOct 20, 2015
  3. Lars SchneiderOct 24, 2015
  4. Luke DiamandOct 24, 2015
  5. Luke DiamandOct 21, 2015
  6. Lars SchneiderOct 24, 2015
  7. Luke DiamandOct 26, 2015
  8. Lars SchneiderOct 28, 2015

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.