On Sun, May 15, 2011 at 03:06, Bert Huijben <b...@qqmail.nl> wrote: >... > Every loop would have to check if the repository matches the parent > directory... and/or for file externals. Review capacity for the current code > welcome.
I find this rather insulting. You check in thousands of line of differences (wc/externals.c is 1k lines all by itself), and you suggest that the burden is on US to review the code that you wrote, with no guidance towards what you were trying to accomplish. And now that it is checked in, it is "current code" ... as in somehow it has the stamp of "done" and it is behooven upon us to go over all that work? Instead... I would state that myself and Branko, possibly others, are concerned about this approach of creating "special cases" within the tree of nodes in the working copy. We may need some time to review and consider and discuss. But I take issue with "oh, well, the code is there. if you don't like it, then review it." ... spewing code into the repository faster than others can review is what most would call a "power plant". It has been done several times in the past, and has still been talked about. Your solution may be awesome. But I think you need to work with the rest of us to show why. We're still confused and don't understand how this is *good* for the codebase. Help rather than defend. -g
