Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Mark Kettenis <kettenis@chello.nl>
To: drow@false.org
Cc: mec.gnu@mindspring.com, gdb-patches@sources.redhat.com
Subject: Re: [patch] configure.in: revert osf5.1 no-noncurses special case
Date: Fri, 07 May 2004 16:19:00 -0000	[thread overview]
Message-ID: <200405071619.i47GJ5S4065835@elgar.kettenis.dyndns.org> (raw)
In-Reply-To: <20040507153338.GA22511@nevyn.them.org> (message from Daniel Jacobowitz on Fri, 7 May 2004 11:33:38 -0400)

   Date: Fri, 7 May 2004 11:33:38 -0400
   From: Daniel Jacobowitz <drow@false.org>

   On Fri, May 07, 2004 at 11:10:48AM -0400, Michael Chastain wrote:
   > Hi Mark,
   > 
   > Yeah, I was unhappy about reverting the patch, but in the long run,
   > the problem is not really specific to osf5.1, so it's better to
   > solve the real problem.
   > 
   > > 4. Unly use ncurses if the user passes --with-ncurses to configure.
   > 
   > I prefer this solution the best.  We've had similar requests for
   > readline from people who want to use the system readline library
   > or their own readline library rather than our bundled readline.
   > And this way a clueful user has the maximum usability, while a
   > no-customization user has a good chance of getting a working gdb
   > and even a gdbtui.

   I think this is a bad idea.  Remember, there's this huge base of
   installed systems where ncurses is the default library and/or installed
   in a system directory.  Why penalize them?

Most of the open source systems use ncurses as the native curses
libraries.  On such systems, ncurses is either installed as libncurses
(OpenBSD) or there is a link from libcurses to libcurses (FreeBSD,
Debian GNU/Linux).  The headers are treated in a similar way.  I'm
certainly not proposing to detect whether libcurses is actually
libncurses, and refuse to use libcurses in that case.  I'm just
proposing to search for libcurses and ncurses.h only if --with-ncurses
is specified.

I really doubt that there are really any systems out there where
ncurses is installed in a system directory alongside with the native
curses library where the two are not one and the same.  If there are
such systems out there, we'd only penalize them if the native curses
library is somehow broken.

   > The advantage of the current scheme (option #0) is that it might work
   > on some systems.  I'm unhappy with #0 because I know that it doesn't
   > work on my hp test drive system.  As I understand it, you are unhappy
   > with #1 for the converse reason: the configury can automatically make
   > a bad choice on some systems.

   But the reason it doesn't work on your HP test drive system is a broken
   or extremely unusual installation of ncurses, so I don't want to make
   policy decisions for GDB based on it.  Personally, I don't see the
   point in worrying about this.  If you've got a broken ncurses
   installation - one where the linker finds -lncurses but gcc doesn't, or
   vice versa, is broken in my book - it's your problem.

It's broken in my book too, but it's very likely that you'll end up in
that situation if you install GCC and ncurses using

./configure && make && make install

on almost any UNIX system.

   I believe that checking for whichever header we are going to use is the
   appropriate decision.  Then, if that fails, either error out or disable
   gdbtui.  This is what the thousands of other software packages using
   non-system libraries do.

We certainly should disable the TUI if something is amiss.  To my
amazement we currently don't do that.

Mark


  reply	other threads:[~2004-05-07 16:19 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-05-07 15:10 Michael Elizabeth Chastain
2004-05-07 15:33 ` Daniel Jacobowitz
2004-05-07 16:19   ` Mark Kettenis [this message]
2004-05-07 16:49     ` Daniel Jacobowitz
2004-05-07 15:51 ` Andrew Cagney
  -- strict thread matches above, loose matches on Subject: below --
2004-05-08  0:11 Michael Elizabeth Chastain
2004-05-08 15:58 ` Daniel Jacobowitz
2004-05-07 17:05 Michael Elizabeth Chastain
2004-05-07 15:52 Michael Elizabeth Chastain
2004-05-07 16:23 ` Mark Kettenis
2004-05-07 16:45   ` Daniel Jacobowitz
2004-05-01 22:27 Michael Elizabeth Chastain
2004-05-07 13:50 ` Mark Kettenis

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=200405071619.i47GJ5S4065835@elgar.kettenis.dyndns.org \
    --to=kettenis@chello.nl \
    --cc=drow@false.org \
    --cc=gdb-patches@sources.redhat.com \
    --cc=mec.gnu@mindspring.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