From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-21147-listarch-gdb-patches=sources.redhat.com@sources.redhat.com>
Received: (qmail 25103 invoked by alias); 23 Dec 2002 10:08:59 -0000
Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm
Precedence: bulk
List-Subscribe: <mailto:gdb-patches-subscribe@sources.redhat.com>
List-Archive: <http://sources.redhat.com/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sources.redhat.com>
List-Help: <mailto:gdb-patches-help@sources.redhat.com>, <http://sources.redhat.com/ml/#faqs>
Sender: gdb-patches-owner@sources.redhat.com
Received: (qmail 25090 invoked from network); 23 Dec 2002 10:08:57 -0000
Received: from unknown (HELO is.elta.co.il) (199.203.121.2)
  by 209.249.29.67 with SMTP; 23 Dec 2002 10:08:57 -0000
Received: from is (is [199.203.121.2])
	by is.elta.co.il (8.9.3/8.8.8) with SMTP id LAA13795;
	Mon, 23 Dec 2002 11:53:15 +0200 (IST)
Date: Mon, 23 Dec 2002 06:59:00 -0000
From: Eli Zaretskii <eliz@is.elta.co.il>
X-Sender: eliz@is
To: Jim Blandy <jimb@redhat.com>
cc: gdb-patches@sources.redhat.com
Subject: Re: [repost] Re: RFC: support debug info in separate files
In-Reply-To: <vt2el89xf85.fsf@zenia.red-bean.com>
Message-ID: <Pine.SUN.3.91.1021223114735.13625A-100000@is>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
X-SW-Source: 2002-12/txt/msg00635.txt.bz2


On 23 Dec 2002, Jim Blandy wrote:

> - The documentation has been completely rewritten, and provides much
>   more detail.  It includes all the information needed to implement

Thanks.  The doco patch is approved (assuming that the code is 
accepted).  I have a few minor change requests:

> + @cindex .debug subdirectories

Please use the normal markup in @cindex entries, in this case .debug 
should be in @file...

> + @cindex .gnu_debuglink sections

...and here .gnu_debuglink should be in @code.

> + Drepper's @file{elfutils} package, starting with version 0.53,
> + contains a version of the @code{strip} command such that the command
> + @samp{strip foo -f foo.debug} removes the debugging information from

I think it's better to put commands users type in @kbd instead of @samp.  
But I don't object to @samp if you like it better.

> + @kindex gnu_debuglink_crc32
> + @example
> + unsigned long
> + gnu_debuglink_crc32 (unsigned long crc, unsigned char *buf, size_t len)

IIRC, we decided to use @smallexample throughout.  Also, the last line 
above is too long even for @smallexample (will produce overfull hbox 
warnings from TeX), so please break it into 2 lines.

Thanks again for such a clear and concise description.