Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Christopher Faylor <me@cgf.cx>
To: binutils@sources.redhat.com, gdb-patches@sources.redhat.com
Subject: Re: top-level removal of dejagnu, expect
Date: Thu, 10 Jun 2004 00:38:00 -0000	[thread overview]
Message-ID: <20040610003839.GB13254@coe.casa.cgf.cx> (raw)
In-Reply-To: <20040610101446.B3128@mailhub.air.net.au>

On Thu, Jun 10, 2004 at 10:14:46AM +1000, Ben Elliston wrote:
>> 1. Have you asked EVERY project that uses the src repository?
>>    binutils and gdb aren't the only projects.  Why not just remove
>>    those directories from the modules file instead?  cgf has already
>>    mentioned that src/expect has cygwin-specific code in it.
>
>I have asked the major players and got overwhelming support. Removing
>the directories from the modules file is broken, as you know.  It
>should be possible to check those files out if they were included in a
>past tag (which they were).
>
>As for Expect, the version in src/expect has not been updated since
>sourceware was instated last century :-) There are much newer versions
>which carry fixes for some of the problems that are being reported.  I
>am pushing all patches made since the import to Don Libes and do not
>intend to drop any.  I found a couple of patches related to Cygwin.
>Many of them were removed by Chris Faylor in 2001, however.

I think DJ removed a few, too.  The basic problem was that expect was
trying to invent ptys on Windows.  Since cygwin already has ptys, a lot
of code could go away.

>> 2. Our internal repository has a customized dejagnu; it would be
>>    convenient for us if support for an in-tree dejagnu were
>>    maintained.  Likewise for other developers who may want to (or need
>>    to) customize dejagnu for new ports in conjunction with porting the
>>    tools.
>
>I stated at the GCC Summit that, as a DejaGnu maintainer, I will be
>responsive to urgent bug fixes.  I appreciate that.  Those fixes can
>be made available via the Savannah CVS server, or I can push out
>maintenance releases as required.  No one felt this was a problem.

If it was anyone else but you Ben, I might have reservations.  I know
that you will be enthusiastically responsive to any problems.

>> So, I would prefer that support for an in-tree dejagnu were
>> retained.  Even if dejagnu and expect were totally wiped from the
>> src repository, the toplevel changes would not be needed to support
>
>An in-tree dejagnu is a pain to maintain.  The last six months have
>demonstrated this.  I merged the two dejagnu trees at the end of
>January and since then, have struggled to keep them in sync.

That's why I can't object too strongly to this.  I don't like mirroring
other people's repositories on sourceware since it just guarantees
drift.  Readline suffers similarly.

So, even if it causes a little pain now, the net gain should be worth
it.

cgf


  reply	other threads:[~2004-06-10  0:38 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-06-09 23:23 Ben Elliston
2004-06-09 23:26 ` Daniel Jacobowitz
2004-06-09 23:29   ` Ben Elliston
2004-06-10 14:08     ` Andrew Cagney
2004-06-10 20:10       ` Ben Elliston
2004-06-11  3:44         ` Andrew Cagney
2004-06-11  3:51           ` Ben Elliston
2004-06-11  4:15             ` Andrew Cagney
2004-06-11  4:18               ` Daniel Jacobowitz
2004-06-09 23:40 ` DJ Delorie
2004-06-10  0:15   ` Ben Elliston
2004-06-10  0:38     ` Christopher Faylor [this message]
2004-06-10  2:15     ` DJ Delorie
2004-06-10  2:19       ` Ben Elliston
2004-06-10  2:30         ` DJ Delorie
2004-06-10  2:34           ` Ben Elliston
2004-06-10  2:41             ` DJ Delorie
2004-06-10  2:50               ` Ben Elliston
2004-06-10 14:03             ` Andrew Cagney
2004-06-14 20:31               ` Alexandre Oliva
2004-06-10  2:22       ` Ian Lance Taylor
2004-06-10  2:27       ` Ben Elliston
2004-06-10  2:31         ` DJ Delorie
2004-06-14 20:25 ` Alexandre Oliva

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20040610003839.GB13254@coe.casa.cgf.cx \
    --to=me@cgf.cx \
    --cc=binutils@sources.redhat.com \
    --cc=gdb-patches@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox