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

Re: [GIT-P4] usage under Windows

From
Tor Arvid Lund <torarvid@gmail.com>
Date
Mar 1, 2008, 11:44 UTC
Message-ID
<B929D1FF-BAF7-4595-9DAF-07E62B90D767@gmail.com>
In-Reply-To
<96c268400802291048u31a2d8b6ub1726ade941afb8c@mail.gmail.com>
On 29. feb.. 2008, at 19.48, Maxim Gordienko wrote:
Show 7 quoted lines
> Thank you, Marius for detailed instructions!
> Now everything except submitting back to perforce works like a charm.
> Perforce complains about git-p4 checkout is not under it's root
>
> Synchronizing p4 checkout...
> executing p4 sync ...
> Path 'c:/tmp/p42/main\...' is not under client's root 'c:\p4'.

I have seen it too. I'm not sure, but it seems to me like even though the git-p4 script does a chdir(<perforce_dir>) before calling "p4 <command> <args>" the chdir is "not seen by" p4 on windows.

I have a patch on my machine where i simply change all the p4 calls, like so:

system("p4 sync ...") --> system("p4 sync %s..." % self.clientPath)

This seems to work in all cases, and also in Mac OS X... I can probably clean the patch up a bit, and submit it later today or tomorrow if you're interested.

      -- Tor Arvid
Previous: Marius Storm-OlsenNext: Marius Storm-Olsen
Message 7 of 9 in “[GIT-P4] usage under Windows”
  1. Maxim GordienkoFeb 22, 2008
  2. Marius Storm-OlsenFeb 27, 2008
  3. Maxim GordienkoFeb 29, 2008
  4. Marius Storm-OlsenFeb 29, 2008
  5. Maxim GordienkoMar 1, 2008
  6. Marius Storm-OlsenMar 1, 2008
  7. Tor Arvid LundMar 1, 2008
  8. Marius Storm-OlsenMar 1, 2008
  9. Maxim GordienkoMar 3, 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.