Re: Fix git-svn for SVN 1.7

2012-08-02 Thread Eric Wong
Junio C Hamano wrote: > Eric Wong writes: > > Thanks for reminding me, I went back to an old chroot 1.4.2 indeed > > does fail canonicalization. > > > > Will bisect and squash a fix in. > > Oops; should I eject this out of next and wait for a reroll, then? Y

Re: [PATCH 6/7] Switch path canonicalization to use the SVN API.

2012-08-02 Thread Eric Wong
Eric Wong wrote: > Michael G Schwern wrote: > > On 2012.7.28 6:55 AM, Jonathan Nieder wrote: > > > Michael G. Schwern wrote: > > >> --- a/perl/Git/SVN/Utils.pm > > >> +++ b/perl/Git/SVN/Utils.pm > > >> @@ -86,6 +86,27 @@ sub _collapse_do

Re: Fix git-svn for SVN 1.7

2012-08-02 Thread Eric Wong
Eric Wong wrote: > Junio C Hamano wrote: > > Eric Wong writes: > > > Thanks for reminding me, I went back to an old chroot 1.4.2 indeed > > > does fail canonicalization. > > > > > > Will bisect and squash a fix in. > > > > Oops; sh

Re: Fix git-svn for SVN 1.7

2012-08-02 Thread Eric Wong
"Robin H. Johnson" wrote: > On Thu, Aug 02, 2012 at 11:58:08AM -0700, Junio C Hamano wrote: > > > Thanks from me as well. I'm still worried about whether the increased > > > use of canonicalize_url will introduce regressions for the existing > > > SVN 1.6 support, and I should have time to look

Re: [PATCH v2 1/2] git-svn.perl: consider all ranges for a given merge, instead of only tip-by-tip

2012-08-06 Thread Eric Wong
Steven Walter wrote: > Consider the case where you have trunk, branchA of trunk, and branchB of > branchA. trunk is merged back into branchB, and then branchB is > reintegrated into trunk. The merge of branchB into trunk will have > svn:mergeinfo property references to both branchA and branchB.

Re: [PATCH v2 2/2] git-svn.perl: keep processing all commits in parents_exclude

2012-08-06 Thread Eric Wong
Steven Walter wrote: > This fixes a bug where git finds the incorrect merge parent. Consider a > repository with trunk, branch1 of trunk, and branch2 of branch1. > Without this change, git interprets a merge of branch2 into trunk as a > merge of branch1 into trunk. Sam: your eyes would be much a

Re: [PATCH] git svn: reset invalidates the memoized mergeinfo caches

2012-08-07 Thread Eric Wong
Peter Baumann wrote: > Therefore the easiest solution to clear the cache is to delete the files > on disk in 'git svn reset'. Normally, deleting the files behind the back > of the memoization module would be problematic, because the in-memory > representation would still exist and contain wrong da

Re: [PATCH] git svn: reset invalidates the memoized mergeinfo caches

2012-08-08 Thread Eric Wong
Peter Baumann wrote: > On Tue, Aug 07, 2012 at 08:45:10PM +, Eric Wong wrote: > > Peter Baumann wrote: > > > + for my $suffix (qw(yaml db)) { > > > + unlink("$cache_file.$suffix"); > > > > Nee

Re: [PATCH/RFC] git svn: handle errors and concurrent commits in dcommit

2012-08-08 Thread Eric Wong
Robert Luberda wrote: > Eric Wong wrote: > >> + echo "PATH=\"$PATH\"; export PATH" >> $hook > >> + echo "svnconf=\"$svnconf\"" >> $hook > >> + cat >> "$hook" <&

Re: [PATCH/RFC] git svn: handle errors and concurrent commits in dcommit

2012-08-10 Thread Eric Wong
Junio C Hamano wrote: > I should have asked this yesterday, but do you mean you want to have > your "maint" in the upcoming 1.7.12? This does look like a useful > thing to do, but does not seem like a regression fix to me. Yeah, I wasn't sure what to name it since my master is still carrying Mic

Re: [PATCH v2] git svn: reset invalidates the memoized mergeinfo caches

2012-08-10 Thread Eric Wong
Peter Baumann wrote: > First, let me thank you for your review and your detailed explanation. > I really appreciate it. You're welcome, Peter. Thanks again for the patch. I've signed-off and pushed for Junio. The following changes since commit 034161a94e827ef05790b1c7ce5a6e3e740c864e: Merge

Re: [PATCH v2] git svn: reset invalidates the memoized mergeinfo caches

2012-08-10 Thread Eric Wong
Junio C Hamano wrote: > Eric Wong writes: > > Peter Baumann (1): > > git svn: reset invalidates the memoized mergeinfo caches > > > > Robert Luberda (1): > > git svn: handle errors and concurrent commits in dcommit > > OK, so these two are fi

Re: [PATCH v3 1/2] git-svn.perl: consider all ranges for a given merge, instead of only tip-by-tip

2012-08-13 Thread Eric Wong
Steven Walter wrote: > Consider the case where you have trunk, branchA of trunk, and branchB of > branchA. trunk is merged back into branchB, and then branchB is > reintegrated into trunk. The merge of branchB into trunk will have > svn:mergeinfo property references to both branchA and branchB.

Re: [PATCH v3 2/2] git-svn.perl: keep processing all commits in parents_exclude

2012-08-19 Thread Eric Wong
Steven Walter wrote: > On Sat, Aug 18, 2012 at 3:51 PM, Sam Vilain wrote: > > On 08/11/2012 10:14 AM, Steven Walter wrote: > >> --- > >> git-svn.perl |1 - > >> t/t9164-git-svn-fetch-merge-branch-of-branch2.sh | 53 > >> ++ Thanks a

Re: [PATCH v3 0/2] Make git-svn work with gitdir links

2013-01-22 Thread Eric Wong
Barry Wardell wrote: > These patches fix a bug which prevented git-svn from working with repositories > which use gitdir links. > > Changes since v2: > - Rebased onto latest master. > - Added test case which verifies that the problem has been fixed. > - Fixed problems with git svn (init|clone|

Re: [PATCH v3 0/2] Make git-svn work with gitdir links

2013-01-23 Thread Eric Wong
Barry Wardell wrote: > On Wed, Jan 23, 2013 at 2:32 AM, Eric Wong wrote: > > Does squashing this on top of your changes fix all your failures? > > I plan on squashing both your changes together with the below: > > Yes, I can confirm that applying this patch on top of min

[PATCH] git-svn: cleanup sprintf usage for uppercasing hex

2013-01-23 Thread Eric Wong
We do not need to call uc() separately for sprintf("%x") as sprintf("%X") is available. Signed-off-by: Eric Wong --- perl/Git/SVN.pm| 4 ++-- perl/Git/SVN/Editor.pm | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/perl/Git/SVN.pm b/perl/Git/

[PULL] git-svn updates for master

2013-01-23 Thread Eric Wong
: git-svn: cleanup sprintf usage for uppercasing hex (2013-01-24 01:30:04 +) Barry Wardell (1): git-svn: Simplify calculation of GIT_DIR Eric Wong (1): git-svn: cleanup sprintf usage for uppercasing hex git-svn.perl

Re: [PATCH v3 0/2] Make git-svn work with gitdir links

2013-01-24 Thread Eric Wong
Junio C Hamano wrote: > Eric Wong writes: > > > diff --git a/git-svn.perl b/git-svn.perl > > index c232798..e5bd292 100755 > > --- a/git-svn.perl > > +++ b/git-svn.perl > > @@ -332,11 +332,13 @@ if ($cmd && $cmd =~ /(?:clone|init|multi-init)$/) { >

[PULL (updated)] git-svn updates for master

2013-01-24 Thread Eric Wong
er is unchanged.) Barry Wardell (1): git-svn: Simplify calculation of GIT_DIR Eric Wong (1): git-svn: cleanup sprintf usage for uppercasing hex git-svn.perl | 37 ++--- perl/Git/SVN.pm | 4

[PATCH] Documentation/githooks: Fix linkgit

2013-03-01 Thread Andrew Wong
Signed-off-by: Andrew Wong --- Documentation/githooks.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/githooks.txt b/Documentation/githooks.txt index 8181e4e..eab9b35 100644 --- a/Documentation/githooks.txt +++ b/Documentation/githooks.txt @@ -365,7 +365,7

Re: [PATCH] In partial SVN merges, the ranges contains additional character "*"

2013-03-07 Thread Eric Wong
(adding Sam to the Cc:, I rely on him for SVN merge knowledge) Jan Pešta wrote: > See http://www.open.collab.net/community/subversion/articles/merge-info.html > Extract: > The range r30430:30435 that was added to 1.5.x in this merge has a '*' > suffix for 1.5.x\www. > This '*' is the marker for a

[PATCH] setup.c: Fix prefix_pathspec from looping pass end of string

2013-03-07 Thread Andrew Wong
The previous code was assuming length ends at either `)` or `,`, and was not handling the case where strcspn returns length due to end of string. So specifying ":(top" as pathspec will cause the loop to go pass the end of string. Signed-off-by: Andrew Wong --- setup.c | 6 -- 1 fi

Re: [PATCH] setup.c: Fix prefix_pathspec from looping pass end of string

2013-03-07 Thread Andrew Wong
On 3/7/13, Junio C Hamano wrote: > The parser that goes past the end of the string may be a bug worth > fixing, but is this patch sufficient to diagnose such an input as an > error? Yea, the patch should fix the passing end of string too. The parser was going past end of string because the nextat

Re: [PATCH] setup.c: Fix prefix_pathspec from looping pass end of string

2013-03-07 Thread Andrew Wong
On 3/7/13, Junio C Hamano wrote: > This did not error out for me, though. > > $ cd t && git ls-files ":(top" No error message at all? Hm, maybe in your case, the byte after the end of string happens to be '\0' and the loop ended by chance? git doesn't crash for me, but it generates this erro

Re: [PATCH] setup.c: Fix prefix_pathspec from looping pass end of string

2013-03-07 Thread Andrew Wong
On 03/07/13 20:51, Junio C Hamano wrote: > But it is equally broken to behave as if there is nothing wrong in > the incomplete magic ":(top" that is not closed, isn't it? Ah, yea, I did notice that, but then I saw a few lines below: if (*copyfrom == ')') copyfrom++; which is exp

Re: [PATCH] git svn: ignore partial svn:mergeinfo

2013-03-08 Thread Eric Wong
, I've pushed this with a minor formatting change (added space after "W:") will push another change for formatting existing warnings more consistently. Signed-off-by: Eric Wong -- 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 svn: consistent spacing after "W:" in warnings

2013-03-08 Thread Eric Wong
Eric Wong wrote: > will push another change for formatting existing warnings more > consistently. Just pushed. My master is sitting at git://git.bogomips.org/git-svn.git commit eae6cf5aa8ae2d8a90a99bbe4aeb01c29e01fd02 Eric Wong (1): git svn: consistent spacing after &

Re: rebase: strange failures to apply patc 3-way

2013-03-08 Thread Andrew Wong
On 3/8/13, Max Horn wrote: > All in all, I suspect that Mac OS X and/or the filesystem (HFS+ with > journaling, not case sensitive (the default)) might be at fault. Still, this > is quite puzzling and annoying, and so I still wonder if anybody has any > insights on this. When "rebase" errors out

Re: rebase: strange failures to apply patc 3-way

2013-03-08 Thread Andrew Wong
On 3/8/13, Max Horn wrote: > I tried this a dozen times, but 'git apply' failed to fail even once. No > surprise there, given that the patch that throws off rebase every time is > clean and simple. I am flabbergasted :-( Hm, what if you add in the "--index" flag? i.e. git apply --index .git/r

Re: rebase: strange failures to apply patc 3-way

2013-03-08 Thread Andrew Wong
On 3/8/13, Max Horn wrote: > Same result, it works fine. Just shooting in the dark here... I wonder if there's some background process running in OS X that's messing with the files/directories while rebase is working... backup, virus scan, etc? Or maybe some programs that you're using at the same

Re: rebase: strange failures to apply patc 3-way

2013-03-09 Thread Andrew Wong
On 03/09/13 06:26, Max Horn wrote: > It tends to fail in separate places, but eventually "stabilizes". E.g. I just > did a couple test rebases, and it failed twice in commit 14, then the third > time in commit 15 (which underlines once more that the failures are > inappropriate). > > The fourth

Re: rebase: strange failures to apply patc 3-way

2013-03-09 Thread Andrew Wong
On 03/09/13 13:32, Andrew Wong wrote: > Yea, that's really suspicious. This could mean there's an issue with > when git is checking the index. Try running these a couple times in a > clean work tree: > $ git update-index --refresh > $ git diff-files > > In a

[PATCH 1/2] setup.c: Fix prefix_pathspec from looping pass end of string

2013-03-09 Thread Andrew Wong
The previous code was assuming length ends at either ")" or ",", and was not handling the case where strcspn returns length due to end of string. So specifying ":(top" as pathspec will cause the loop to go pass the end of string. Signed-off-by: Andrew Wong --- setu

[PATCH 2/2] setup.c: Check that the pathspec magic ends with ")"

2013-03-09 Thread Andrew Wong
The previous code allowed the ")" to be optional. Signed-off-by: Andrew Wong --- setup.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/setup.c b/setup.c index f4c4e73..5ed2b93 100644 --- a/setup.c +++ b/setup.c @@ -225,8 +225,9 @@ static const char *prefi

Re: rebase: strange failures to apply patc 3-way

2013-03-11 Thread Andrew Wong
On 3/10/13, Max Horn wrote: > I did run > > touch lib/*.* src/*.* && git update-index --refresh && git diff-files > > a couple dozen times (the "problematic" files where in src/ and lib), but > nothing happened. I just re-checked, and the rebase still fails in the same > why... > > Perhaps I sho

Re: rebase: strange failures to apply patc 3-way

2013-03-11 Thread Andrew Wong
On 3/11/13, Max Horn wrote: > So I tried this: > > git rebase branches/stable-4.6 # ... which leads to the error > git ls-files -m > > but got nothing. Perhaps you had something else in mind, though, but I am > not quite sure what... sorry for being dense, but if you let me know what > exactl

Re: rebase: strange failures to apply patc 3-way

2013-03-11 Thread Andrew Wong
On 3/11/13, Max Horn wrote: > PS: Just as a side note, I should mention that I have done tons of rebases > on various repositories on this very machine: same hard drive, same file > system; the git version of course has changed over time, but as I already > described, I can reproduce the same issu

Re: rebase: strange failures to apply patc 3-way

2013-03-11 Thread Andrew Wong
On 3/11/13, Max Horn wrote: > Looking at the git config man page to check what each of my config settings > does, I discovered "trustctime". And adding > trustctime = false > to .git/config made the rebase work, every single time. Huh. > > > Adding this to the fact that a clone works fine, I

Re: rebase: strange failures to apply patc 3-way

2013-03-11 Thread Andrew Wong
On 03/11/13 20:29, Max Horn wrote: > sudo launchctl unload /System/Library/LaunchDaemons/com.apple.revisiond.plist > > -> this exits revisiond, and prevents launchd from respawning it. After that, > the problem is gone, on several attempts. And once I reactivate revisions, > the problem reoccurs.

Re: rebase: strange failures to apply patc 3-way

2013-03-11 Thread Andrew Wong
On 03/11/13 21:01, Jeff King wrote: > From "git help config": > > core.trustctime > If false, the ctime differences between the index and the working > tree are ignored; useful when the inode change time is regularly > modified by something outside Git (file system crawlers and some >

Re: git svn error "Not a valid object name"

2013-03-15 Thread Eric Wong
Adam Retter wrote: > Our public SourceForge Subversion repository is here: > http://svn.code.sf.net/p/exist/code/trunk/eXist It's asking me for a username/password... > We cloned that to the local server using rsync and are attempting to > migrate to git using the following commands: > > $ git

Re: git svn error "Not a valid object name"

2013-03-16 Thread Eric Wong
Adam Retter wrote: > If your able, any idea of when you might be able to take a look at the > bug? Our svn repo is publicly available for all. svn ls https://svn.code.sf.net/p/exist/code/trunk ...Is asking me for username -- To unsubscribe from this list: send the line "unsubscribe git" in the bo

Re: [BUG?] rebase -i: edit versus unmerged changes

2013-03-19 Thread Andrew Wong
On 3/19/13, Ramkumar Ramachandra wrote: > I know that this is expected behavior, but is there an easy way to get > rid of this inconsistency? You can actually rely on "rebase" to run the appropriate command. In the first edit commit (the_ no conflict one), I usually run only "git add" to add my c

Re: Raise your hand to Ack jk/code-of-conduct if your Ack fell thru cracks

2019-10-09 Thread Eric Wong
Junio C Hamano wrote: > For reference, here is the CoC the patch wants to add (there is no > such topic yet locally, nor a single patch that can be made into > such a topic, so there isn't exactly something people can Ack on > yet. So here is a "preview" of what we would see once such a series

Re: RFE: use patchwork to submit a patch

2019-10-10 Thread Eric Wong
Jonathan Nieder wrote: > Eric Wong wrote: > > Konstantin Ryabitsev wrote: > > >> This is actually really fast if you already have a local copy of the > >> repository with most objects. Try this yourself if you have > >> torvalds/linux.git locally:

Re: ds/sparse-cone, was Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)

2019-10-14 Thread Eric Wong
Johannes Schindelin wrote: > On Fri, 11 Oct 2019, Junio C Hamano wrote: > > * ds/sparse-cone (2019-10-08) 17 commits > > - sparse-checkout: cone mode should not interact with .gitignore > > - sparse-checkout: write using lockfile > > - sparse-checkout: update working directory in-process > > -

Re: ds/sparse-cone, was Re: What's cooking in git.git (Oct 2019, #03; Fri, 11)

2019-10-15 Thread Eric Wong
Junio C Hamano wrote: > Eric Wong writes: > > > I just took a brief look, but that appears to leak memory. > > > > "hashmap_free(var, 1)" should be replaced with > > "hashmap_free_entries(var, struct foo, member)" > > > > Onl

[RFC/WIP] range-diff: show old/new blob OIDs in comments

2019-10-17 Thread Eric Wong
ld. I'm also still learning my way around git's C internals, but using patch.{old,new}_oid_prefix seems alright... FIXME: tests, t3206 needs updating Not-signed-off-by: Eric Wong --- range-diff.c | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/ran

Re: email as a bona fide git transport

2019-10-17 Thread Eric Wong
ers brought up around exactness and working on top of cherry-picks. > PS: Eric Wong described something that comes quite close to this idea, but > AFAICT without actually recreating commits exactly. I've included the link > for completeness. [4] > [4]: https://lore.kernel.org/work

Re: email as a bona fide git transport

2019-10-22 Thread Eric Wong
Vegard Nossum wrote: > I sent v2 of the patches (with metadata _after_ the diff) to the git > list here: > > https://public-inbox.org/git/20191022114518.32055-1-vegard.nos...@oracle.com/T/#u > > As I wrote in there, we could already today start using > >git am --message-id > > when applyin

Re: [RFC/WIP] range-diff: show old/new blob OIDs in comments

2019-10-22 Thread Eric Wong
Johannes Schindelin wrote: > Hi Eric, > > > On Thu, 17 Oct 2019, Eric Wong wrote: > > > (WIP, mostly stream-of-concious notes + reasoning) > > > > When using "git format-patch --range-diff", the pre and > > post-image blob OIDs are in each

Re: [RFC/WIP] range-diff: show old/new blob OIDs in comments

2019-10-23 Thread Eric Wong
Eric Wong wrote: > Johannes Schindelin wrote: > > I guess your patch won't hurt. > > Cool, will update tests and resend. Turns out the t3206 "trivial reordering" case is way too noisy. I might need to change diff.c to support this case; will update in a few days or week.

[no subject]

2015-08-15 Thread Mr. Peter Wong
Tervehdys ja kohteliaisuuksia. Olen Peter Wong Työskentelen BANK OF CHINA Minulla Business ehdotus vireessä US $ 22.500.000 Million siirretään offshore tilin apua, jos haluaa. Jos kiinnostaa minä lähetän teille täyden tapahtuman tiedot saatuaan vastaustasi. Voit ottaa yhteyttä minuun minun

For your Interest

2014-02-06 Thread Wong and Partners
-- 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

<    5   6   7   8   9   10