On Tue, Apr 19, 2011 at 04:46:52PM -0000, rhuijben_at_apache.org wrote:
> Author: rhuijben
> Date: Tue Apr 19 16:46:51 2011
> New Revision: 1095130
>
> URL: http://svn.apache.org/viewvc?rev=1095130&view=rev
> Log:
> Remove the 'apply svn:externals from added files' code we introduced during the
> development of Subversion 1.7.0.
>
> While nice to have in certain use cases like testing new externals
> (see issue #2267), this quick and dirty hack introduses issues like
> issue #3351, where externals are not properly removed.
>
> This patch brings us back to the original behavior of only applying the
> svn:externals as defined by the BASE tree (read: of committed nodes).
> Which fixes the original automatic (but until recently mostly hidden)
> removal of externals on update.
>
> The removal/relegation of externals was made more visible in r1095122,
> by updating the notification code.
Hmmm... was removing this feature really necessary?
Instead of removing this feature, could you not reimplement the
feature in a way that suits your requirements better, e.g. by making
these externals disappear properly when they are reverted?
I would have appreciated some discussion before this change was made.
There are people out there how need this feature for their workflow,
which is why I spent time on making this work. It would be nice to have
externals on locally added dirs working for 1.7.0. Are you planning on
getting this to work in 1.7.0? Given that you simply closed the issue
as WONTFIX I guess that's not the case.
Regarding your last comment to the issue, I don't care what the book says.
If it says that only committed externals definitions can be handled it is
probably talking about an implementation defect rather than an unsurmountable
problem.
All that said, I really appreciate your work on getting externals fixed
because it is a very important part of getting 1.7 released. However,
reverting code without prior discussion isn't nice.
Received on 2011-04-20 00:25:32 CEST