You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by David Glasser <gl...@davidglasser.net> on 2007/12/19 01:29:36 UTC
empty path in mergeinfo
I think we currently allow the following in mergeinfo:
: 1-3
ie, empty path strings. This is bad, right?
--dave
--
David Glasser | glasser@davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: empty path in mergeinfo
Posted by David Glasser <gl...@davidglasser.net>.
On Dec 18, 2007 6:03 PM, Paul Burba <pb...@collab.net> wrote:
> > -----Original Message-----
> > From: dglasser@gmail.com [mailto:dglasser@gmail.com] On
> > Behalf Of David Glasser
> > Sent: Tuesday, December 18, 2007 8:30 PM
> > To: dev
> > Subject: empty path in mergeinfo
> >
> > I think we currently allow the following in mergeinfo:
> >
> > : 1-3
> >
> > ie, empty path strings. This is bad, right?
>
> Yeah, that doesn't have any valid meaning that I can think of. svn
> propset allows that right now and that we can fix easily. But did you
> see this mergeinfo arise as the result of a merge or by some other
> mechanism other than a propset?
No, just concerned about a place where I'm consuming mergeinfo and Bad
Things would happen if the path was empty.
--dave
--
David Glasser | glasser@davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
RE: empty path in mergeinfo
Posted by Paul Burba <pb...@collab.net>.
> -----Original Message-----
> From: dglasser@gmail.com [mailto:dglasser@gmail.com] On
> Behalf Of David Glasser
> Sent: Tuesday, December 18, 2007 8:30 PM
> To: dev
> Subject: empty path in mergeinfo
>
> I think we currently allow the following in mergeinfo:
>
> : 1-3
>
> ie, empty path strings. This is bad, right?
Yeah, that doesn't have any valid meaning that I can think of. svn
propset allows that right now and that we can fix easily. But did you
see this mergeinfo arise as the result of a merge or by some other
mechanism other than a propset?
Paul
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org