[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn rm $URI mangles the commit-message-template

From: Mark Eichin <eichin_at_gmail.com>
Date: Fri, 4 Jul 2014 17:15:03 -0400

On Thu, Jul 3, 2014 at 6:24 PM, Daniel Shahaf <d.s_at_daniel.shahaf.name>
wrote:

> Mark Eichin wrote on Thu, Jul 03, 2014 at 14:59:30 -0400:
> > 1.6 on ubuntu precise gets this right, 1.8 on trusty gets it wrong, the
> > following repro script is from trusty:
> >
> > $ mkdir svnrmbug
> > $ cd svnrmbug
> > /home/eichin/tmp/svnrmbug
> > $ svnadmin create repo
> > $ U=file://$(pwd)/repo
> > $ svn mkdir --parents -m 'base'
> $U/{trunk,branches,tags,branches/me,tags/me}
> >
> > Committed revision 1.
> > $ svn cp -m 'branch' $U/trunk $U/branches/me/bugbranch
> >
> > Committed revision 2.
> > $ EDITOR=false VISUAL=false svn rm $U/branches/me/bugbranch
> > svn: E200012: system('false svn-commit.tmp') returned 256
> > svn: E200012: Your commit message was left in a temporary file:
> > svn: E200012: 'svn-commit.tmp'
> > $ cat svn-commit.tmp
> >
> > --This line, and those below, will be ignored--
> >
> > D file:///home/eichin/tmp/svnrmbug/repo/bugbranch
> >
>
> Allow me to clarify: it says
>
> D file:///home/eichin/tmp/svnrmbug/repo/bugbranch
>
> but it should have said
>
> D file:///home/eichin/tmp/svnrmbug/repo/branches/me/bugbranch
>
>
Correct. My reference to (basename branchpath) below was extrapolating
from the original case and some tests against real repos that I couldn't
include.

> > Note specifically the URI to be deleted. (Ironically, I think it really
> > *does* get ignored, a coworker pointed it out just recently and noone
> else
> > had seen it before...)
> >
>
> Yes, it does get ignored, just like it says.
>
>
The irony I was trying to point out is that if it's been wrong this long,
the material beneath that line gets ignored by *human readers* as well as
by the code.

> Daniel
>
> > In a few experiments, it looks like the deletion line is consistently
> > (svnroot)/(basename branchpath) and that it doesn't happen with svn
> mkdir.
> > Some quality time with trunk/CHANGES turned up only issue #1199 as being
> > even vaguely related (which mostly confirmed that it wasn't an
> intentional
> > change...)
> >
> > --
> > _Mark_ <eichin_at_thok.org> <eichin_at_gmail.com>
>

-- 
_Mark_ <eichin_at_thok.org> <eichin_at_gmail.com>
Received on 2014-07-04 23:15:35 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.