On Dec 3, 2007 12:26 PM, Paul Burba <pburba@collab.net> wrote:
> In r28213, I put some restrictions in place as to what values svn ps
> svn:mergeinfo will permit (previously we allowed just about anything).
> Most of these restrictions were already checked for in
> svn_mergeinfo_parse(), except for one: mergeinfo with paths that map to
> empty revisions. As
> http://subversion.tigris.org/issues/show_bug.cgi?id=3029* discusses
> these are redundant and best avoided. With r28213 The checks are done
> by leveraging svn_mergeinfo_parse() and then performing a second check
> for empty ranges.
>
> I'd like to move the empty range check into svn_mergeinfo_parse(), but I
> worry about users who have been working with trunk builds the last year
> and have mergeinfo in their repositories with paths mapped to empty
> ranges. Since svn_mergeinfo_parse() would error out when trying even
> list their svn:mergeinfo it would be pretty hard for them to fix. *So
> now for my real question*: How much deference should we give to those
> users who have been using trunk builds?
>
> Paul
>
> * Yes, yes, I know as we battle with issue #2897 and all the "Big" (tm)
> questions, that the small details like this don't get much attention,
> but a fantastic new A-class office building with the three L's ain't
> much if the toilets don't work.
It's important to give notification when incompatible changes hit
trunk (we could even do this in a file or issue or something instead
of relying on constant mailing list reading), and if it's easy enough
posting a migration script might help. But we shouldn't leave code in
the actual codebase to migrate from unreleased versions.
--dave
--
David Glasser | glasser_at_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
Received on Mon Dec 3 21:34:40 2007