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

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

From
Junio C Hamano <gitster@pobox.com>
Date
Aug 9, 2012, 17:44 UTC
Message-ID
<7v1ujgot8h.fsf@alter.siamese.dyndns.org>
In-Reply-To
<20120808230754.GB24956@dcvr.yhbt.net>
Eric Wong <normalperson@yhbt.net> writes:
Show 23 quoted lines
> Robert Luberda <robert@debian.org> wrote:
>> Eric Wong wrote:
>> >> +		echo "PATH=\"$PATH\"; export PATH" >> $hook
>> >> +		echo "svnconf=\"$svnconf\"" >> $hook
>> >> +		cat >> "$hook" <<- 'EOF2'
>> >> +			cd work-auto-commits.svn
>> >> +			svn up --config-dir "$svnconf"
>> > 
>> > That doesn't seem to interact well with users who depend on
>> > svn_cmd pointing to something non-standard.  Not sure
>> > what to do about it, though....
>
>> I have no idea how to change it either. I've tried to source the
>> lib-git-svn.sh file inside the hook, but it sources test-lib.sh, and the
>> latter script doesn't work well if it is sourced by non-test script.
>> Anyway I the part of my original patch unchanged.
>
> Ah, so svn_cmd only cares about --config-dir and you already handled
> that :)   I misremembered it also allowed for non-standard SVN
> installations :x
>
> I've pushed your updated patch to my "maint" branch on
> git://bogomips.org/git-svn since "master" has larger pending changes.

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.

Previous: Eric WongNext: Eric Wong
Message 6 of 14 in “git svn: handle errors and concurrent commits in dcommit”
  1. git svn: handle errors and concurrent commits in dcommitRobert Luberda, Aug 1, 2012
  2. Eric WongAug 2, 2012
  3. Robert LuberdaAug 8, 2012
  4. git svn: handle errors and concurrent commits in dcommitRobert Luberda, Aug 8, 2012
  5. Eric WongAug 8, 2012
  6. Junio C HamanoAug 9, 2012
  7. Eric WongAug 10, 2012
  8. Robert LuberdaAug 19, 2012
  9. Junio C HamanoAug 20, 2012
  10. Eric WongAug 21, 2012
  11. Robert LuberdaAug 24, 2012
  12. Eric WongAug 26, 2012
  13. t9164: More style fixesRobert Luberda, Aug 31, 2012
  14. Junio C HamanoAug 31, 2012

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.