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

Re: [PATCH] Optional shrinking of RCS keywords in git-p4

From
David Brown <git@davidb.org>
Date
Sep 15, 2008, 06:35 UTC
Message-ID
<20080915063521.GA1533@linode.davidb.org>
In-Reply-To
<16219.81556.qm@web95005.mail.in2.yahoo.com>
On Mon, Sep 15, 2008 at 11:56:22AM +0530, dhruva wrote:
>8. The git diffs is now not aware of the change in RCS keyword
>9. You try to submit your local changes back to p4
>10. Applying your local changes as patch sets will fail with missing hunks tracking RCS keyword changes

It sounds like you are trying to apply these as patches to a tree which doesn't have RCS headers. As far as I can tell, P4 completely ignores whatever the $Id: ...$ headers happen to be expanded to at the time of checking. You can put garbage there, and it check in fine.

I've been checking in files for many years with stripped headers. I wrote a python script years ago to strip the P4 headers after Perforce was unwilling to implement this as an option.

I guess it isn't a problem to make this optional in git-p4, but I don't think this patch is solving the right problem.

David
Previous: dhruvaNext: Junio C Hamano
Message 2 of 9 in “Re: [PATCH] Optional shrinking of RCS keywords in git-p4”
  1. dhruvaSep 15, 2008
  2. David BrownSep 15, 2008
  3. Junio C HamanoSep 15, 2008
  4. Tor Arvid LundSep 15, 2008
  5. Daniel BarkalowSep 15, 2008
  6. David BrownSep 16, 2008
  7. Jing XueSep 16, 2008
  8. Daniel BarkalowSep 16, 2008
  9. Daniel BarkalowSep 16, 2008

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.