Re: [PATCH] git-p4: support exclude in 'git p4 sync'

2015-01-18 Thread Pete Wyckoff
l...@diamand.org wrote on Sat, 17 Jan 2015 20:56 +:
 The git-p4 'clone' subcommand has long had the option to specify
 parts of the repo to be excluded, on the command line. But this has
 not been present in 'sync', which makes it less than useful: as
 soon as you do a sync, the excluded parts start being repopulated
 as those directories are changed.
 
 (You can achieve the same effect by using a client specification to
 do the exclusion, but that's then an extra step).
 
 The code for doing the exclusion is actually all present in the base
 'P4Sync' class: this change turns that on by moving the definition
 of the command-line switch.
 
 It also updates the documentation and adds a test-case.
 
 Thanks,
 Luke
 
 And yes, I'm back to using version control systems other than git :-(

So sorry. I on the other hand have been fortunate enough to
switch to using only git.

Nevertheless, I read through the patch and it looks good and
makes sense. You've got my ack on this for what it's worth.
Hopefully someone else starts picking up the git-p4 maintenance
work. Hint.

-- Pete
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] git-p4: support exclude in 'git p4 sync'

2015-01-17 Thread Luke Diamand
The git-p4 'clone' subcommand has long had the option to specify
parts of the repo to be excluded, on the command line. But this has
not been present in 'sync', which makes it less than useful: as
soon as you do a sync, the excluded parts start being repopulated
as those directories are changed.

(You can achieve the same effect by using a client specification to
do the exclusion, but that's then an extra step).

The code for doing the exclusion is actually all present in the base
'P4Sync' class: this change turns that on by moving the definition
of the command-line switch.

It also updates the documentation and adds a test-case.

Thanks,
Luke

And yes, I'm back to using version control systems other than git :-(

Luke Diamand (1):
  git-p4: support excluding paths on sync

 Documentation/git-p4.txt  |6 ++--
 git-p4.py |   18 ++--
 t/t9817-git-p4-exclude.sh |   71 +
 3 files changed, 83 insertions(+), 12 deletions(-)
 create mode 100755 t/t9817-git-p4-exclude.sh

-- 
1.7.10.4

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html