This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PATCH: update Makefiles for expect, runtest


On Jul  3, 2005, Ben Elliston <bje+dated+1120800102.b3e4cc@air.net.au> wrote:

>> > This patch (committed) updates the EXPECT and RUNTEST make variables,
>> > as these are no longer in-tree.
>> 
>> Why are you gratuitously removing the ability for someone to keep
>> expect and dejagnu in their own tree?

> To keep the sources consistent with the tree.

They're already consistent.  There was a test in place to tell whether
you had dejagnu and expect in the tree.  Removing it doesn't make it
any more compatible if you don't have them, but breaks the in-tree
use.

> The real question is: why are you trying to use an in-tree copy of
> expect and dejagnu?

Because when you're developing a new target, it's easier to build
and test everything in a single run than tweak several small
components, build and install each of them separately, and adjust your
PATH or so for every different target.

I guess it's the same reason why people like binutils, gas, ld, gdb et
al in a single tree (and would rather have GCC thrown in as well):
convenience.

By removing the support for single-tree build&test, you're just making
the use of these custom tools less convenient.  Sure enough, people
can always patch support back into their own sources, but, really, why
should everyone who cares about such convenience have to go through
the inconvenience of carrying a custom patch in their trees, if having
it in the original tree won't hurt anyone?

-- 
Alexandre Oliva         http://www.lsd.ic.unicamp.br/~oliva/
Red Hat Compiler Engineer   aoliva@{redhat.com, gcc.gnu.org}
Free Software Evangelist  oliva@{lsd.ic.unicamp.br, gnu.org}


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]