threads / patch / 12964

patchgit-p4: Wrong sync config filename

Subject: [PATCH] git-p4: Wrong sync config filename

## tl;dr

2 messages between Apr 2, 2008 and Apr 3, 2008. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Kevin Leung· Apr 2, 2008, 08:23 UTC · lore
The filename should be p4-git-sync.cfg.db instead of p4-git-sync.cfg.
Signed-off-by: Kevin Leung <kevinlsk@gmail.com>
---
I encounter this problem after submitting the changelist to Perforce.

I was using Mac OS X 10.5 with Python 2.5.1. I am not sure if this patch will help others.

Change 29323 submitted.
All changes applied!
Performing incremental import into refs/remotes/p4/master git branch
Depot paths: //depot/POPCAT/
Import destination: refs/remotes/p4/master
Importing revision 29323 (100%)
Do you want to rebase current HEAD from Perforce now using git-p4
rebase? [y]es/[n]o n
Traceback (most recent call last):
  File "/Users/kevin/bin/git-p4", line 1828, in <module>
    main()
  File "/Users/kevin/bin/git-p4", line 1823, in main
    if not cmd.run(args):
  File "/Users/kevin/bin/git-p4", line 835, in run
    os.remove(self.configFile)
OSError: [Errno 2] No such file or directory:
'/Users/kevin/cdc.git/popcat/.git/p4-git-sync.cfg'
 contrib/fast-import/git-p4 |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Show changes to contrib/fast-import/git-p4 +1 −1
diff --git a/contrib/fast-import/git-p4 b/contrib/fast-import/git-p4
index c80a6da..623757a 100755
--- a/contrib/fast-import/git-p4
+++ b/contrib/fast-import/git-p4
@@ -796,7 +796,7 @@ class P4Submit(Command):
                 self.logSubstitutions[tokens[0]] = tokens[1]

         self.check()
-        self.configFile = self.gitdir + "/p4-git-sync.cfg"
+        self.configFile = self.gitdir + "/p4-git-sync.cfg.db"
         self.config = shelve.open(self.configFile, writeback=True)

         if self.firstTime:
-- 
1.5.4.5
Simon Hausmann· Apr 3, 2008, 07:09 UTC · re: Kevin Leung · lore

Re: [PATCH] git-p4: Wrong sync config filename

On Wednesday 02 April 2008 10:23:00 Kevin Leung wrote:
Show 8 quoted lines
> The filename should be p4-git-sync.cfg.db instead of p4-git-sync.cfg.
>
> Signed-off-by: Kevin Leung <kevinlsk@gmail.com>
> ---
> I encounter this problem after submitting the changelist to Perforce.
>
> I was using Mac OS X 10.5 with Python 2.5.1. I am not sure if this patch
> will help others.

Thanks for the patch! Fortunately this config file is not needed anymore and I removed its use alltogether a while ago, also as a result of running into this problem on Mac OS X myself :). So the latest git has this resolved :)

Simon
Show 34 quoted lines
> Change 29323 submitted.
> All changes applied!
> Performing incremental import into refs/remotes/p4/master git branch
> Depot paths: //depot/POPCAT/
> Import destination: refs/remotes/p4/master
> Importing revision 29323 (100%)
> Do you want to rebase current HEAD from Perforce now using git-p4
> rebase? [y]es/[n]o n
> Traceback (most recent call last):
>   File "/Users/kevin/bin/git-p4", line 1828, in <module>
>     main()
>   File "/Users/kevin/bin/git-p4", line 1823, in main
>     if not cmd.run(args):
>   File "/Users/kevin/bin/git-p4", line 835, in run
>     os.remove(self.configFile)
> OSError: [Errno 2] No such file or directory:
> '/Users/kevin/cdc.git/popcat/.git/p4-git-sync.cfg'
>
>  contrib/fast-import/git-p4 |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/contrib/fast-import/git-p4 b/contrib/fast-import/git-p4
> index c80a6da..623757a 100755
> --- a/contrib/fast-import/git-p4
> +++ b/contrib/fast-import/git-p4
> @@ -796,7 +796,7 @@ class P4Submit(Command):
>                  self.logSubstitutions[tokens[0]] = tokens[1]
>
>          self.check()
> -        self.configFile = self.gitdir + "/p4-git-sync.cfg"
> +        self.configFile = self.gitdir + "/p4-git-sync.cfg.db"
>          self.config = shelve.open(self.configFile, writeback=True)
>
>          if self.firstTime:

← back to recent threads