On Mon, Aug 22, 2011 at 7:18 PM, Greg Stein <gst...@gmail.com> wrote: > On Mon, Aug 22, 2011 at 19:19, Neels J Hofmeyr <ne...@elego.de> wrote: >>... >> So if 'svn:hold' *only* affects commit; Would I eventually be allowed to >> code this into trunk?
I think we don't need to be talking about a one-size-fits-all approach here. While I don't think an explosion of special properties is good either, a well chosen set which handle the various behaviors people want may be a better solution than an 'svn:hold' attempting to be all things to all people. Another consideration is that properties live forever, possibly even past some eventual 2.0, and when implementing new special props, conservative is definitely the way to go. We can always add more; we can't ever take them away. > I'd give that a +1 if two things are implemented: > > 1) adjust the log message to note the hold-back Are you talking about the suggested log message that appears in $EDITOR, or munging the log message in flight? I'm fine with the former, but the latter feels it's a huge minefield. > 2) status shows 'H' for held files that also have local mods > >>... >> And I can stay on the branch for a while to come, too. Would just be more >> convenient on trunk, cuts out all that merging. > > I prefer trunk development. But only if you're talking about 'svn > commit', the log message changes, and 'svn status'. Affecting other > commands... then we get back into discussion :-) By limiting the scope as I mention above, it's much easier to implement on trunk. > btw, regarding 'svn diff' showing changes to held files: I think it > should. There are two types of changes to those held files: > local-only, and changes that you *want* to propagate to your > teammates. If you hide local mods, then a developer could miss the > need to commit the important work. +1 'svn diff' should still show mods to held files. Feel free to introduce some option to diff which could potentially screen files with this property, but as the default behavior, 'svn diff' should show all mods to a working copy. -Hyrum -- uberSVN: Apache Subversion Made Easy http://www.uberSVN.com/
