Hi,
Some quick details.
Florian Achleitner wrote:
>Anyways, specifying cat-blob-fd is not
> allowed via the 'option' command (see Documentation and 85c62395).
> It wouldn't make too much sense, because the file descriptor must be set up
> by
> the paren
Martin von Zweigbergk writes:
> Do you want a reroll with updated commit messages (the missing "avoid"
> above, the dropped "seems like" about the prefix in 1/2)?
Nah, I've already queued them with log message tweaks. Thanks for
asking, though.
--
To unsubscribe from this list: send the line "
On Fri, Jul 27, 2012 at 3:30 PM, Junio C Hamano wrote:
> Martin von Zweigbergk writes:
>> Also re-initializing rev_info fields to the same values already set in
>> init_revisions().
Oops, that should have been " _avoid_ re-initializing".
> I suspect that
> explicit initialization to revs.ignore
Eric Wong writes:
> Junio C Hamano wrote:
>> Eric Wong writes:
>>
>> > "Michael G. Schwern" wrote:
>> >> This series of patches extracts the remaining classes from git-svn.
>> >> They're
>> >> all simple extractions and functionally have no change.
>> >
>> > I've also pushed this to the "ex
Junio C Hamano wrote:
> Eric Wong writes:
>
> > "Michael G. Schwern" wrote:
> >> This series of patches extracts the remaining classes from git-svn.
> >> They're
> >> all simple extractions and functionally have no change.
> >
> > I've also pushed this to the "extract-remaining" series which
Eric Wong writes:
> "Michael G. Schwern" wrote:
>> This series of patches extracts the remaining classes from git-svn. They're
>> all simple extractions and functionally have no change.
>
> I've also pushed this to the "extract-remaining" series which
> also includes everything that's currently
Michael G. Schwern wrote:
> This is the last refactoring patch series. After this bugs, start
> getting fixed.
I just wanted to say thanks for your thoughtful presentation of this
code. I was worried before, but these have been pleasantly submitted.
If you have a chance at some point to offer
"Michael G. Schwern" wrote:
We try to keep Subject: lines short (~50 char soft limit) as documented
in SubmittingPatches.
How about:
Subject: [PATCH 4/5] use Git::SVN->path accessor globally
Subject: [PATCH 5/5] use Git::SVN{,::RA}->url accessor globally
?
I can make the changes to the commit
Our e-mail content detector has just been triggered by a message you sent:
To: sp-man...@sp.interorient.com
Subject: Delivery reports about your e-mail
Date: Fri Jul 27 22:52:37 2012
One or more of the attachments (interorient.co.scr, interorient.co.zip) are on
the list of unacceptable attac
On Fri, Jul 27, 2012 at 3:44 PM, Junio C Hamano wrote:
> Martin von Zweigbergk writes:
>
> s/it seems the prefix doesn't change/the prefix never changes/;
:-) Thanks for confirming.
> In that sense it probably does not even matter if you did not pass
> rev->prefix down to check_rev, and instead
"Michael G. Schwern" wrote:
> This series of patches extracts the remaining classes from git-svn. They're
> all simple extractions and functionally have no change.
I've also pushed this to the "extract-remaining" series which
also includes everything that's currently in my master.
I squashed Mi
On Friday, July 27, 2012 03:58:49 PM you wrote:
> Sascha Cunz writes:
> > Ok, so repository and working directory are simply not meant to be on
> > different file systems. Thanks for the clarification.
>
> I did not mean "and that is a rule we need to enforce and keep
> forever".
I did not parse
Stefan Zager writes:
> On Fri, Jul 27, 2012 at 2:38 PM, Junio C Hamano wrote:
>
>> Stefan Zager writes:
>>
>> > + module_list "$@" | awk '{print $4}' | xargs -L 1 -P
>> "$jobs" git submodule update $orig_args
>>
>> Capital-P option to xargs is not even in POSIX, no?
>
> I wasn't awa
Junio C Hamano wrote:
> Ok, please don't forget to add necessary .gitignore rule for the new
> stamp file.
I noticed/remembered that, but I forgot to mention I squashed that in,
too :)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.ker
Eric Wong writes:
> Junio C Hamano wrote:
>> Eric Wong writes:
>> > The redundant dependencies are biting us :< I agree there presence in
>> > the top-level Makefile needs to be reviewed.
>>
>> Do you feel confident enough that we can leave that question hanging
>> around and still merge this
Sascha Cunz writes:
> Ok, so repository and working directory are simply not meant to be on
> different file systems. Thanks for the clarification.
I did not mean "and that is a rule we need to enforce and keep
forever". I was just answering your (implied) question "why does
code comment, behav
Eric Wong writes:
> Eric Wong wrote:
>> So I'll hold off until we can fix the build regressions (working on it
>> now)
>
> OK, all fixed, all I needed was this (squashed in):
>
> --- a/perl/Makefile
> +++ b/perl/Makefile
> @@ -22,6 +22,8 @@ clean:
> $(RM) $(makfile).old
> $(RM) PM.st
On Friday, July 27, 2012 02:55:49 PM you wrote:
> Sascha Cunz writes:
> > From 3f449e719b924929f1f8ca9b5eff83f17bc64c60 Mon Sep 17 00:00:00 2001
> > From: Sascha Cunz
> > Date: Fri, 27 Jul 2012 22:54:56 +0200
> > Subject: [PATCH] Use work tree to determine if it supports symlinks
> >
> > When cr
Junio C Hamano wrote:
> Eric Wong writes:
> > The redundant dependencies are biting us :< I agree there presence in
> > the top-level Makefile needs to be reviewed.
>
> Do you feel confident enough that we can leave that question hanging
> around and still merge this before 1.7.12 safely?
Yes.
Martin von Zweigbergk writes:
> get_patch_ids() takes an already initialized rev_info and a
> prefix. The prefix is used when initalizing a second rev_info. Since
> the initialized rev_info already has a prefix and it seems the prefix
> doesn't change, we can used the prefix from the initialized
Eric Wong writes:
> OK, all fixed, all I needed was this (squashed in):
>
> --- a/perl/Makefile
> +++ b/perl/Makefile
> @@ -22,6 +22,8 @@ clean:
> $(RM) $(makfile).old
> $(RM) PM.stamp
>
> +$(makfile): PM.stamp
> +
> ifdef NO_PERL_MAKEMAKER
> instdir_SQ = $(subst ','\'',$(prefix)/
Martin von Zweigbergk writes:
> While walking the revision list in get_patch_ids and cmd_cherry, we
> ignore merges by checking if there is more than one parent. However,
> since the revision walk was initialized with revs.ignore_merges = 1,
> this would never happen. Remove the unnecessary check
Eric Wong wrote:
> Junio C Hamano wrote:
> > Eric Wong writes:
> >
> > > I'll put the following after ms/makefile-pl but before ms/git-svn-pm:
> >
> > OK, it seems that you haven't pushed out the result yet (which is
> > fine); how do we want to proceed?
> So I'll hold off until we can fix th
Junio C Hamano wrote:
> Eric Wong writes:
>
> > I'll put the following after ms/makefile-pl but before ms/git-svn-pm:
>
> OK, it seems that you haven't pushed out the result yet (which is
> fine); how do we want to proceed?
Oops, got sidetracked into something else. Before I got sidetracked,
Sascha Cunz writes:
> From 3f449e719b924929f1f8ca9b5eff83f17bc64c60 Mon Sep 17 00:00:00 2001
> From: Sascha Cunz
> Date: Fri, 27 Jul 2012 22:54:56 +0200
> Subject: [PATCH] Use work tree to determine if it supports symlinks
>
> When creating a new repository, we check some capabilities of the
> u
Eric Wong writes:
> I'll put the following after ms/makefile-pl but before ms/git-svn-pm:
OK, it seems that you haven't pushed out the result yet (which is
fine); how do we want to proceed?
I generally prefer pulling from maintainer trees directly to my
'master' without staging them in 'next',
>From 3f449e719b924929f1f8ca9b5eff83f17bc64c60 Mon Sep 17 00:00:00 2001
From: Sascha Cunz
Date: Fri, 27 Jul 2012 22:54:56 +0200
Subject: [PATCH] Use work tree to determine if it supports symlinks
When creating a new repository, we check some capabilities of the
underlying file system(s). We check
Stefan Zager writes:
> + module_list "$@" | awk '{print $4}' | xargs -L 1 -P "$jobs" git
> submodule update $orig_args
Capital-P option to xargs is not even in POSIX, no?
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.ker
Michael G Schwern writes:
> On 2012.7.27 1:07 PM, Eric Wong wrote:
>> While Makefile.PL now finds .pm files on its own, it does not
>> detect new files after it generates perl/perl.mak.
>
> Are you saying this doesn't work?
>
> perl Makefile.PL
> make -f perl.mak
> touch Git/Foo.pm
> perl Makefil
Michael G Schwern wrote:
> On 2012.7.27 1:07 PM, Eric Wong wrote:
> > While Makefile.PL now finds .pm files on its own, it does not
> > detect new files after it generates perl/perl.mak.
>
> Are you saying this doesn't work?
>
> perl Makefile.PL
> make -f perl.mak
> touch Git/Foo.pm
> perl Makef
On 2012.7.27 1:07 PM, Eric Wong wrote:
> While Makefile.PL now finds .pm files on its own, it does not
> detect new files after it generates perl/perl.mak.
Are you saying this doesn't work?
perl Makefile.PL
make -f perl.mak
touch Git/Foo.pm
perl Makefile.PL
make -f perl.mak
or this?
perl Makefi
Junio C Hamano wrote:
> What was happening was that originally, pu had ms/makefile-pl but
> not ms/git-svn-pm. Hence, perl/Git/SVN.pm did not exist. I ran
> "make" and it created perl/perl.mak that does not know about
> Git/SVN.pm;
>
> Then ms/git-svn-pm is merged to pu and now we have perl/Git
From: "Michael G. Schwern"
Note: The structure returned from Git::SVN->read_all_remotes() does not appear
to
contain objects, so I'm leaving them alone.
That's everything converted over to the url and path accessors.
No functional change.
---
git-svn.perl | 11 ++-
perl/G
From: "Michael G. Schwern"
No functional change.
---
git-svn.perl| 12 +++-
perl/Git/SVN.pm | 4 ++--
perl/Git/SVN/Fetcher.pm | 2 +-
perl/Git/SVN/Ra.pm | 6 +++---
4 files changed, 13 insertions(+), 11 deletions(-)
diff --git a/git-svn.perl b/git-svn.perl
in
From: "Michael G. Schwern"
Later it can canonicalize automatically.
A later change will make other things use the accessor.
No functional change.
---
perl/Git/SVN/Ra.pm | 40 +---
1 file changed, 29 insertions(+), 11 deletions(-)
diff --git a/perl/Git/SVN/R
From: "Michael G. Schwern"
So later it can do automatic canonicalization.
A later patch will make other things use the accessor.
No functional change here.
---
perl/Git/SVN.pm | 44 ++--
1 file changed, 30 insertions(+), 14 deletions(-)
diff --git a/per
From: "Michael G. Schwern"
Then later it can be canonicalized automatically rather than everywhere
its used.
Later patch will make other things use it.
---
perl/Git/SVN.pm | 87 +
1 file changed, 56 insertions(+), 31 deletions(-)
diff --g
This patch series gives Git::SVN and Git::SVN::Ra accessors for
path and url and then makes the rest of the code use them, rather
than grab at $obj->{path} and $obj->{url}. This then will give
us the control necessary to canonicalize them as early as
possible (done in the next patch series).
Ther
On 2012.7.27 4:29 AM, Erik Faye-Lund wrote:
> On Fri, Jul 27, 2012 at 2:27 AM, Michael G Schwern wrote:
>> On 2012.7.26 5:25 PM, Michael G. Schwern wrote:
>>> This series of patches extracts the remaining classes from git-svn. They're
>>> all simple extractions and functionally have no change.
>>
The --jobs parameter may be used to set the degree of per-submodule
parallel execution.
Signed-off-by: Stefan Zager
---
Documentation/git-submodule.txt | 8 +++-
git-submodule.sh| 23 ++-
2 files changed, 29 insertions(+), 2 deletions(-)
diff --git a/Doc
On Fri, Jul 27, 2012 at 7:10 PM, Erik Faye-Lund wrote:
> Since the Windows port of Git expects binary pipes, we need to make
> sure the helper-end also sets up binary pipes.
>
> Side-step CRLF-issue in test to make it pass.
>
> Signed-off-by: Erik Faye-Lund
> ---
>
> Sorry for the delay, but here
Michael J Gruber writes:
> 5/5 needs a fix in the subject line, sorry. It should be:
>
> t3910: use the UTF8_NFD_TO_NFC test prereq
>
> (5/5 hasn't hit next)
That is because I thought that you would like the lazy-probe and
that you would be capable of rerolling it on your own ;-)
--
To unsubscri
These are just some things I noticed while looking at the revision
walking code. I'm not very familiar with the code at all, so the
patches may very well be completely wrong; view them as newbie
questions in patch-form if you like.
Martin von Zweigbergk (2):
remove unnecessary parameter from ge
get_patch_ids() takes an already initialized rev_info and a
prefix. The prefix is used when initalizing a second rev_info. Since
the initialized rev_info already has a prefix and it seems the prefix
doesn't change, we can used the prefix from the initialized rev_info
to initialize the second rev_in
Jeff King writes:
> On Fri, Jul 27, 2012 at 11:45:00AM -0400, Jeff King wrote:
>
>> And of course with non-broken probes, "-v" would also help to see exactly
>> when the probe is run, using which commands, and what output they
>> produced. E.g., for SYMLINKS, we should probably drop the stderr
>>
Since the Windows port of Git expects binary pipes, we need to make
sure the helper-end also sets up binary pipes.
Side-step CRLF-issue in test to make it pass.
Signed-off-by: Erik Faye-Lund
---
Sorry for the delay, but here's the third iteration of my credential
helper for Windows' credential
On Thu, Jul 26, 2012 at 11:09 PM, Junio C Hamano wrote:
> * mz/rebase-range (2012-07-18) 7 commits
> - rebase (without -p): correctly calculate patches to rebase
> - rebase -p: don't request --left-right only to ignore left side
> - rebase -p: use --cherry-mark for todo file
> - git-rebase--in
On Fri, Jul 27, 2012 at 11:45:00AM -0400, Jeff King wrote:
> And of course with non-broken probes, "-v" would also help to see exactly
> when the probe is run, using which commands, and what output they
> produced. E.g., for SYMLINKS, we should probably drop the stderr
> redirection from the probe
On Thu, Jul 26, 2012 at 04:11:00PM -0700, Junio C Hamano wrote:
> Introduce a mechanism to probe the prerequiste lazily. Changes are:
>
> - test_lazy_prereq () function, which takes the name of the
>prerequisite it probes and the script to probe for it, is
>added. This only registers t
Thanks Jeff for the comment. You are right. I oversimplified the use-case (
forget to fill the directory )
Sent from my iPad
On Jul 27, 2012, at 5:29 PM, Jeff King wrote:
> On Fri, Jul 27, 2012 at 07:06:08AM +0400, Aleksandr Pryimak wrote:
>
>> I also recreated it
>>
>> aleksandr@beast:/tmp/
On Fri, Jul 27, 2012 at 09:29:53AM -0400, Jeff King wrote:
> If we put actual files inside "x", which git does track, then they would
> be part of the stash, and should be properly retained. But they're not:
>
> $ rm x && mkdir x && echo foo >x/file
>
> Now we have some precious contents in th
On Fri, Jul 27, 2012 at 07:06:08AM +0400, Aleksandr Pryimak wrote:
> I also recreated it
>
> aleksandr@beast:/tmp/test$ git init
> Initialized empty Git repository in /tmp/test/.git/
> aleksandr@beast:/tmp/test$ touch x
> aleksandr@beast:/tmp/test$ git add x
> aleksandr@beast:/tmp/test$ git commi
Junio C Hamano wrote:
> The result will be queued tentatively near the tip of 'pu', but as
> this is primarily about git-svn, I would prefer a copy that is
> vetted by Eric to be fed from him.
OK. I've signed-off on the 7 patches you have in pu. Will look at the
other series in a few hours.
Th
Michael G Schwern wrote:
> On 2012.7.26 10:18 PM, Junio C Hamano wrote:
> > Again, I agree with you that passing $prefix as one of the arguments
> > to ->new is the right thing to do in the final state after applying
> > the whole series. I don't know if later steps in your patch series
> > will
Scott Chacon writes:
> GitHub would like to volunteer to organize and pay for these events
> this year. I would like to hold the developer-centric one in Berlin
> in early October
Yay, Berlin! I would be glad to join there; I would probably not have
the time and resources to travel to SF this
Junio C Hamano wrote:
> "Michael G. Schwern" writes:
> > +# Note: not certain why this is in use instead of die. Probably because
> > +# the exit code of die is 255? Doesn't appear to be used consistently.
Yes, 255 caused problems for the test suite:
commit d25c26e771fdf771f264dc85be348719886
On Fri, Jul 27, 2012 at 2:27 AM, Michael G Schwern wrote:
> On 2012.7.26 5:25 PM, Michael G. Schwern wrote:
>> This series of patches extracts the remaining classes from git-svn. They're
>> all simple extractions and functionally have no change.
>
> PS This is on top of the previous Git::SVN ext
Junio C Hamano venit, vidit, dixit 26.07.2012 22:10:
> Jeff King writes:
>
>> That looks like a maintenance annoyance. Can't we just have the
>> prerequisite-checker lazily perform the test on demand and cache the
>> result? It should be OK as long as:
>>
>> 1. The prereq is careful about its
On 2012.7.26 10:18 PM, Junio C Hamano wrote:
> If you swap the order of steps 3/4 and 4/4 by creating Git/SVN.pm
> that only has these variable definitions (i.e. "our $X" and "use
> vars $X") and make git-svn.perl use them from Git::SVN in the first
> step, and then do the bulk-moving (equivalent o
On 2012.7.26 10:18 PM, Junio C Hamano wrote:
>> +# Note: not certain why this is in use instead of die. Probably because
>> +# the exit code of die is 255? Doesn't appear to be used consistently.
>> +sub fatal (@) { print STDERR "@_\n"; exit 1 }
>
> Very true. Also I do not think the line-noise
On 2012.7.26 10:18 PM, Junio C Hamano wrote:
> Forgot to sign-off, or are you still unsure about this step?
I just never think to do it. It's just a line in the commit message, right?
There's no crypto involved like tag -s. Is it a blocker? I guess I can write
a msg-filter if it's important.
On Thursday 26 July 2012 09:54:26 Jonathan Nieder wrote:
>
> Since the svn remote helper relies on this, it seems worth working on,
> yeah. As for how to spend your time (and whether to beg someone else
> to work on it instead :)): I'm not sure what's on your plate or where
> you are with respect
Junio C Hamano writes:
> Junio C Hamano writes:
>
>> t91XX series seem to fail in 'pu' with "Can't locate Git/SVN.pm in
>> @INC" for me. I see perl/blib/lib/Git/SVN/ directory and files
>> under it, but there is no perl/blib/lib/Git/SVN.pm installed. I see
>> Git/I18N.pm and Git/SVN/Ra.pm (and
63 matches
Mail list logo