On Sun, Aug 01, 2010 at 03:58:14PM +0200, Bert Huijben wrote: > I don't think we should remove the '/db' part here. > > At the repos layer the '/db' doesn't make sense, but this is an > implementation on the 'fs' layer and there the 'db' does make sense. > > (And if we decide to remove /db here, we should remove it in at least > a dozen other error messages defined in > subversion/include/private/svn_fs_util.h. (I actually wrote a patch to > do that a few days ago, but then I found the different layering).
Well, this may be a layer violation. But to guide users, I'd like to print the command users can use to fix the problem in the error message. Can you suggest a better way to achive that without violating layers? If not, I'd rather keep the error message as is. I can live with trivial layer violations like this one if it helps users. Stefan
