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

Re: [PATCH] Fix git p4 sync errors

From
Christian Couder <christian.couder@gmail.com>
Date
Oct 26, 2012, 15:44 UTC
Message-ID
<CAP8UFD0+FPDERfkTeJWuq=T6RQ65yjHwbhTP_SYv6aA+1xq32Q@mail.gmail.com>
In-Reply-To
<F0F92B0A-D37F-40D4-A0DF-43EEDA2818B9@gmail.com>
Hi,
On Thu, Oct 25, 2012 at 4:41 AM, Matt Arsenault <arsenm2@gmail.com> wrote:
Show 9 quoted lines
>
> On Oct 21, 2012, at 12:06 , Junio C Hamano <gitster@pobox.com> wrote:
>>
>> - Why is it a bug not to pass "-s"?  How does the bug happen?
>
> I encountered this one time after using it for months. One day I couldn't git p4 rebase
> with the key error.  I searched for the error and found some version of git-p4 that fixed
> a similar error by adding the -s to describe. Adding the -s fixed the error and
> everything seemed to be working correctly.
The perforce documentation says the following about this flag:
"The -s flag omits the diffs of files that were updated."

So if the diffs are not used, there is no downside to use -s. Maybe the patch should just state this.

Best, Christian.

Previous: Matt ArsenaultNext: Pete Wyckoff
Message 4 of 5 in “Fix git p4 sync errors”
  1. Fix git p4 sync errorsMatt Arsenault, Oct 21, 2012
  2. Junio C HamanoOct 21, 2012
  3. Matt ArsenaultOct 25, 2012
  4. Christian CouderOct 26, 2012
  5. Pete WyckoffOct 28, 2012

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.