Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Christopher Faylor <me@cgf.cx>
To: cgf-gdb-patches@sourceware.org, gdb-patches@sourceware.org
Subject: Re: [RFC] Add expat to the GDB sources
Date: Mon, 24 Jul 2006 15:24:00 -0000	[thread overview]
Message-ID: <20060724152438.GA17094@trixie.casa.cgf.cx> (raw)
In-Reply-To: <200607232318.k6NNIV28004376@elgar.sibelius.xs4all.nl>

On Mon, Jul 24, 2006 at 01:18:31AM +0200, Mark Kettenis wrote:
>> Date: Sun, 23 Jul 2006 18:40:32 -0400
>> From: Christopher Faylor <cgf-gdb-patches@sourceware.org>
>> 
>> On Tue, Jul 18, 2006 at 09:40:48AM -0400, Daniel Jacobowitz wrote:
>> >At the beginning of the year, I proposed adding an XML parsing library to
>> >GDB.
>> 
>> ...and, FWIW, there's already an expat directory at the top-level of src
>> which exists entirely as a branch (which you <Daniel> announced).
>> 
>> Just as a meta-issue, I have to wonder at the precedent of one of the
>> projects which shares 'src' adding directories to the top-level.
>> 
>> I just built gdb on linux and I see that it pulls in ncurses but there
>> is no ncurses directory in src.  Why can't we just say that "building
>> gdb requires a native expat library >= some version" like we do with
>> ncurses?  Any other project which uses expat would just add detection of
>> the expat library to the configure phase.
>
>Any UNIX-like system shipped within the last decade comes with a
>decent curses implementation, wo we consider it to be a part of the
>operating system.  Apart from Linux there are probably no systems that
>ship with expat.  And even on most Linux systems expat won't be usable
>because the bloody expat "development" package isn't installed.
>
>Depending on an external expat package comes with the additional
>maintenance cost of testing the detection code and handling additional
>bug reports from people who can't build gdb because of problems with
>expat.
>
>> I've really always hated the habit of duplicating (and essentially forking)
>> other project's source code in 'src' and putting expat there just seems
>> like a step backwards to me.
>
>Well, I really detest that many software packages have so many
>dependencies that I spent an hour hunting down the dependency chain
>before I get actually to building the package I want.

I hate that too but that scenario is less of an inconvenience these days
with tools like emerge, yum, or apt.

OTOH, having built hundreds of different packages for linux, one thing
that really drives me up the wall is when a package includes their own
version of a well-known library.  Did they include it because there is
an incompatibility with the shipping version?  Were they too lazy to add
a configure test?  Did they modify the library?  Will it only work with
the 0.9 version of the library?  Is it going to install the library?

Then there's the question of being polite to the expat maintainers.  Do
they want to field questions from gdb users who wonder why the version
that we have in 'src' doesn't work right?  I would imagine not.

I'm almost pathologically adverse to repeated stupid user questions but
I think in this case, I'd rather take the hit of documenting the best
way of grabbing expat for a particular OS and pointing people there
rather than adding YA duplication to 'src'.  I would really like to see
a day when 'src' will no longer include 'tcl', or 'readline', or
'expat'.

cgf


  parent reply	other threads:[~2006-07-24 15:24 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-07-18 13:40 Daniel Jacobowitz
2006-07-18 13:57 ` Daniel Jacobowitz
2006-07-21  0:35   ` Joel Brobecker
2006-07-21  0:39     ` Daniel Jacobowitz
2006-07-21  0:45       ` Joel Brobecker
2006-07-23 21:52   ` Mark Kettenis
2006-07-23 22:03     ` Daniel Jacobowitz
2006-07-23 22:40 ` Christopher Faylor
2006-07-23 22:57   ` Daniel Jacobowitz
2006-07-23 23:13     ` Christopher Faylor
2006-07-24  0:17       ` Daniel Jacobowitz
2006-07-23 23:15     ` Pedro Alves
2006-07-23 23:18   ` Mark Kettenis
2006-07-24  0:15     ` Daniel Jacobowitz
2006-07-24  6:20     ` Joel Brobecker
2006-07-24 15:30       ` Christopher Faylor
2006-07-24 15:50         ` Daniel Jacobowitz
2006-07-24 16:37           ` Christopher Faylor
2006-07-24 21:58             ` Mark Kettenis
2006-07-24 19:50           ` Eli Zaretskii
2006-07-24 19:52             ` Daniel Jacobowitz
2006-07-24 20:29               ` Eli Zaretskii
2006-07-24 20:36                 ` Daniel Jacobowitz
2006-07-24 15:24     ` Christopher Faylor [this message]
2006-07-24 19:47       ` Eli Zaretskii
2006-07-24 19:51         ` Daniel Jacobowitz
2006-07-24 20:22           ` Christopher Faylor
2006-07-24 20:29           ` Eli Zaretskii
2006-07-24 20:43             ` Daniel Jacobowitz
2006-07-24 21:42             ` Christopher Faylor
2006-07-24 22:18               ` DJ Delorie
2006-07-24 22:29               ` Mark Kettenis
2006-07-24 22:34                 ` Daniel Jacobowitz
2006-07-24 22:37                   ` Daniel Jacobowitz
2006-07-25  0:36                     ` Christopher Faylor
2006-07-24 22:49                   ` Mark Kettenis
2006-07-24 23:41                     ` Daniel Jacobowitz
2006-07-25  0:47                 ` Christopher Faylor
2006-07-31 17:33                 ` Daniel Jacobowitz
2006-07-31 20:24                   ` Daniel Jacobowitz
2006-07-31 20:39                     ` Christopher Faylor
2006-07-31 21:33                     ` Mark Kettenis
2006-08-01  0:42                       ` Daniel Jacobowitz
2006-08-01  1:01                         ` Daniel Jacobowitz
2006-07-24 22:08       ` 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=20060724152438.GA17094@trixie.casa.cgf.cx \
    --to=me@cgf.cx \
    --cc=cgf-gdb-patches@sourceware.org \
    --cc=gdb-patches@sourceware.org \
    /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