[PATCH modular] Remove addconfigh script.

[Dan Nicholson]

On Tue, Sep 21, 2010 at 7:44 AM, Alan Coopersmith
<alan.coopersmith at oracle.com> wrote:
> Dan Nicholson wrote:
>> On Sun, Sep 19, 2010 at 9:49 AM, Jesse Adkins <jesserayadkins at gmail.com> wrote:
>>> This was added in 2005 as part of a big-ish commit. Since then, it hasn't been updated.
>>>
>>> This script isn't used by anything else in modular, and no real reason was given for it in
>>> the commit (the commit only mentions it being added).
>>>
>>> Signed-off-by: Jesse Adkins <jesserayadkins at gmail.com>
>>
>> Clearly it's for adding '#include <config.h>' to packages to support
>> autoheader. It seems like a useful script to throw away even if it's
>> unlikely to be used much. In fact, I really wish I'd known this
>> existed when I was autotooling XTS. I ended up writing my own script
>> with sed to do the same thing.
>
> There's still a large body of code out there (like much of the old
> ftp.x.org/contrib) written to use imake that may someday want to
> convert to autotools that may find such a script useful - seems like
> it's harmless to leave it there for them, though perhaps more useful
> if we had a "imake -> autotools" conversion guide that pointed it out.

It might be useful to organize the scripts in modular more. In my
view, there are 3 types:

packaging - converting a component to the modular, autotooled world
release - preparing a component or katamari for release
build - building one or all the components

I'd rather see the scripts get moved to subdirectories than deleted.

--
Dan