Julian Foad wrote on Mon, Nov 19, 2012 at 14:08:34 +0000: > C. Michael Pilato wrote: > > > On 11/18/2012 10:56 PM, Daniel Shahaf wrote: > >> Julian Foad wrote on Mon, Nov 19, 2012 at 02:50:46 +0000: > >>> Thoughts, objections? > >> > >> Another related trap is setting a revprop as a nodeprop, or vice-versa: > >> svn commit --with-revprop=svn:eol-style=native -mm foo.c > >> svn propset svn:log x foo.c > >> > >> You might want to require --force for these too. > > > > +1 to both suggestions. I can never remember if it's > > "svn:ignore[s]" or > > "svn:keyword[s]". Don't mind getting smacked for choosing > > wrongly, but it's > > always irritating for the propset to succeed yet the behavior not change. > > I agree with Daniel's suggestion too. > > I have filed enhancement issue #4261 "Setting unknown svn: propnames should > require 'force'", which covers both suggestions together. >
Another thing we could do is warn when unrecognised options are found in the config file (~/.subversion/*) or in the --config-option command-line option. This would be an independent improvement, i.e., it neither blocks nor is blocked by the propset improvements in #4261. > Anybody want to work on it? Please do. I can get to it some time, but I > don't know when, if nobody else picks it up. > > - Julian
