From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <archer-return-2005-listarch-archer=sourceware.org@sourceware.org>
Received: (qmail 14561 invoked by alias); 29 Jun 2010 23:21:55 -0000
Mailing-List: contact archer-help@sourceware.org; run by ezmlm
Sender: <archer@sourceware.org>
Precedence: bulk
List-Post: <mailto:archer@sourceware.org>
List-Help: <mailto:archer-help@sourceware.org>
List-Subscribe: <mailto:archer-subscribe@sourceware.org>
List-Id: <archer.sourceware.org>
Received: (qmail 14550 invoked by uid 22791); 29 Jun 2010 23:21:55 -0000
X-SWARE-Spam-Status: No, hits=-4.2 required=5.0
	tests=AWL,BAYES_00,RCVD_IN_DNSWL_HI,SPF_HELO_PASS,TW_YM,T_RP_MATCHES_RCVD
X-Spam-Check-By: sourceware.org
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
From: Roland McGrath <roland@redhat.com>
To: Tom Tromey <tromey@redhat.com>
Cc: Project Archer <archer@sourceware.org>, pmatilai@redhat.com
Subject: Re: find-debuginfo.sh change for gdb index
In-Reply-To: Tom Tromey's message of  Tuesday, 29 June 2010 16:32:38 -0600 <m3vd91tqpl.fsf@fleche.redhat.com>
References: <m3vd91tqpl.fsf@fleche.redhat.com>
Message-Id: <20100629232147.C019548255@magilla.sf.frob.com>
Date: Tue, 29 Jun 2010 23:21:00 -0000
X-SW-Source: 2010-q2/txt/msg00061.txt.bz2

> I don't know what other changes may be needed to ensure that the proper
> gdb is in the buildroots when this script is run.  Also, the proper gdb
> is not actually available yet; cleaning up that patch series is my next
> task.

The rpm-build subpackage will need "Requires: gdb >= V-R".  Obviously, the
patch can't go in anywhere until that gdb is built in dist-rawhide.

> I think this should probably be local to Fedora, but if you think it
> should go into upstream RPM, I am happy to try that.

It's certainly Fedora-specific and only for Fedora 14.

> +  gdb --batch-silent -ex "file $f" -ex "maintenance save-gnu-index $d" > /dev/null 2>&1

I don't quite understand what file this writes to.
Is it implicitly "<symfile name>.index" in the argument directory?

IMHO, the file name should have "gdb" in the name.  
This is really not any very generic sort of index for the information.

>    if [ -n "$id" ]; then
>      make_id_link "$id" "$dn/$(basename $f)"
>      make_id_link "$id" "/usr/lib/debug$dn/$bn" .debug
> +
> +    if [ -f "${debugfn}.index" ]; then
> +      make_id_link "$id" "/usr/lib/debug$dn/$bn" .debug
> +    fi

What's this for?  It just repeats the work of making and recording the
build-id symlink to the .debug file.  Unless you're being quite subtle
somehow I've missed, this doesn't do anything with the index file.
Do you mean something like:

	make_id_link "$id" "/usr/lib/debug$dn/$bn" .index

?  That gets you a /usr/lib/debug/.build-id/xx/yyy.index symlink
to ../../usr/bin/foobar.index for example.


Thanks,
Roland