Augie Fackler wrote:
> On Nov 19, 2007, at 3:42 PM, Karl Fogel wrote:
>> Augie Fackler <email@example.com> writes:
>> One more thing: the code in adm_files.c predates the invention of
>> subversion/include/private/*.h. It's possible that some of the adm
>> area handling could be simplified now, by using svn_wc_private.h. So
>> don't take the current architecture as gospel -- if things look
>> twisted, it's because they are, and you're free to untwist them!
> Excellent. The actual code responsible for silently ignoring .svn and
> _svn is in subversion/libsvn_subr/opt.c:1011. If nobody beats me to it,
> I'll add some code there to print a warning to the user. My inclination
> is that the client should bail out if it sees an argument like this
> (rather than continue) - does anyone vehemently object to that?
We must continue to allow folks to do:
$ svn propget svn:mergeinfo .*
(where that * will be expanded to include .svn) without error. I don't have
any problem with printing a "hey we skipped .svn on your behalf" message,
but doing so must not interrupt the operation.
C. Michael Pilato <firstname.lastname@example.org>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on Mon Nov 19 21:53:58 2007