Re: [PATCH RFC v3 2/2] Move libgit.a sources into separate "lib/" directory

Junio C Hamano <[email protected]>
Newsgroups org.kernel.vger.git
Message-ID <[email protected]>
Johannes Schindelin <[email protected]> writes:

>> > My own (obviously subjective and biased) take is that the tradeoff is
>> > worth it, as these issues are a one-time cost while the benefits to
>> > discoverability will be permanent.
>> 
>> It is not a one-time cost, but will be an ongoing burden.
>
> It is maybe drawn-out, but it is a one-time cost. It's not like we're
> going to mass-rename source files to move them to `lib/` every two weeks
> from now on.

Since the topic was posted, I have dealt with the fallout from it at
least twice a day (which, when we are lucky, is not a huge time
sink, as I have mostly automated it by now), and again every time a
new topic is posted that touches the moved files in substantial ways
or adds new files that ought to be moved.  The latter is the most
time-consuming to handle.  This will continue until all contemporary
topics, as well as the topic in question, graduate.

If that is not an ongoing burden, I do not know what is.

> And this statement neglects to acknowledge that the lack of clean
> organization of source code files is an ongoing burden _right now_, and
> would be at least partially addressed by the move.

At least, Gábor does not seem to think that the lack of clean
organization is so severe as to warrant a massive code churn like
this.

I value stability much more than prettiness.  If we had started out
with almost nothing at the root level and almost everything in
either 'lib' or 'builtin', I would have strongly preferred to keep
that structure.  But since we have been using a layout that has all
built-in commands in 'builtin', with subsystems like 'refs' and
'odb' in their own directories, and everything else at the root
level, I would prefer to keep that organization until a substantial
subsystem update wants to carve out a new location for itself, just
as past updates to create 'builtin', 'refs', and 'odb' did.

Compared to those past moves, the proposed change looks more like
churn for the sake of moving things around, without achieving any
real organizational improvement.

I must say that I, too, remain skeptical.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.