From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32109 invoked by alias); 9 Nov 2013 03:35:40 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 32099 invoked by uid 89); 9 Nov 2013 03:35:39 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.1 required=5.0 tests=BAYES_00,FREEMAIL_FROM,RDNS_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no version=3.3.2 X-HELO: mail-qa0-f48.google.com Received: from Unknown (HELO mail-qa0-f48.google.com) (209.85.216.48) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Sat, 09 Nov 2013 03:35:37 +0000 Received: by mail-qa0-f48.google.com with SMTP id w8so263053qac.0 for ; Fri, 08 Nov 2013 19:35:29 -0800 (PST) X-Received: by 10.49.94.226 with SMTP id df2mr27996132qeb.76.1383968129694; Fri, 08 Nov 2013 19:35:29 -0800 (PST) Received: from hydrogen (naesten-pt.tunnel.tserv4.nyc4.ipv6.he.net. [2001:470:1f06:57::2]) by mx.google.com with ESMTPSA id h2sm29897812qaf.10.2013.11.08.19.35.28 for (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Fri, 08 Nov 2013 19:35:28 -0800 (PST) Received: from naesten by hydrogen with local (Exim 4.80) (envelope-from ) id 1VezKY-00019z-LM; Fri, 08 Nov 2013 22:35:26 -0500 From: Samuel Bronson To: gdb-patches@sourceware.org Cc: Tom Tromey , Jan Kratochvil , Doug Evans , Samuel Bronson Subject: [PATCH v2] Resurrect gdb-add-index as a contrib script Date: Sat, 09 Nov 2013 06:23:00 -0000 Message-Id: <1383968099-4420-1-git-send-email-naesten@gmail.com> In-Reply-To: <1383519137-26388-1-git-send-email-naesten@gmail.com> References: <1383519137-26388-1-git-send-email-naesten@gmail.com> X-IsSubscribed: yes X-SW-Source: 2013-11/txt/msg00255.txt.bz2 This includes some changes made in Fedora's gdb packaging[1], some slight layout changes to make it look nicer in 80 columns, and a line from Doug's patch[2] to deal with file arguments with no "/". I'm still thinking about whether to include the rest of Doug's gdb-add-index changes: they'd make the script harder to read, but more robust. (I've also recast it under the license from (info "(maintain)License Notices for Other Files") so people aren't scared to use snippets in their Makefiles or whatever.) [1]: http://pkgs.fedoraproject.org/cgit/gdb.git/log/gdb-gdb-add-index-script.patch?id=fe74423b0812bae6d7bb027584e401a2ac37d24d [2]: https://sourceware.org/ml/gdb-patches/2010-09/msg00130.html I'm including a log of relevant previous activity. (I would just include committishes, but I'm not quite confident that history won't be rewritten again, invalidating the hashes, plus there are no relevant tags so the first two parts of git-describe(1) output wouldn't be any help either.) commit 97924a976b7d6e1669cf22c5fdf292a53bae6bd0 Author: Tom Tromey Date: Tue Nov 16 17:38:40 2010 +0000 * gdb-add-index.sh: Really remove. gdb/ChangeLog | 4 ++++ gdb/gdb-add-index.sh | 29 ----------------------------- 2 files changed, 4 insertions(+), 29 deletions(-) commit c29c521f43a1f470355cc76391bb10ec2b73284a Author: Tom Tromey Date: Tue Aug 10 18:35:53 2010 +0000 gdb Revert gdb-add-index addition: * Makefile.in (install-only): Don't install gdb-add-index. * gdb-add-index.sh: Remove. gdb/doc Revert gdb-add-index addition: * gdb.texinfo (Index Files): Don't document gdb-add-index. gdb/ChangeLog | 6 ++++++ gdb/Makefile.in | 11 +---------- gdb/doc/ChangeLog | 5 +++++ gdb/doc/gdb.texinfo | 10 ---------- 4 files changed, 12 insertions(+), 20 deletions(-) commit c2bbed2a5ab3c5c5950ef192f9be610a11aaa367 Author: Tom Tromey Date: Fri Jul 30 20:46:34 2010 +0000 gdb * Makefile.in (install-only): Install gdb-add-index. * gdb-add-index.sh: New file. gdb/doc * gdb.texinfo (Index Files): Mention gdb-add-index. gdb/ChangeLog | 5 +++++ gdb/Makefile.in | 11 ++++++++++- gdb/doc/ChangeLog | 4 ++++ gdb/doc/gdb.texinfo | 9 +++++++++ gdb/gdb-add-index.sh | 29 +++++++++++++++++++++++++++++ 5 files changed, 57 insertions(+), 1 deletion(-) --- gdb/ChangeLog | 8 ++++++++ gdb/contrib/gdb-add-index.sh | 27 +++++++++++++++++++++++++++ 2 files changed, 35 insertions(+) create mode 100755 gdb/contrib/gdb-add-index.sh diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 79abe3d..def36a3 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,11 @@ +2013-11-08 Tom Tromey + Jan Kratochvil + Doug Evans + Samuel Bronson + + Bring back gdb-add-index as a contrib script. + * contrib/gdb-add-index.sh: New file. + 2013-11-08 Doug Evans * dwarf2read.c (dwarf2_read_debug): Change to unsigned int. diff --git a/gdb/contrib/gdb-add-index.sh b/gdb/contrib/gdb-add-index.sh new file mode 100755 index 0000000..81b7377 --- /dev/null +++ b/gdb/contrib/gdb-add-index.sh @@ -0,0 +1,27 @@ +#! /bin/sh + +# Add a .gdb_index section to a file. + +# Copyright (C) 2010, 2012-2013 Free Software Foundation, Inc. +# +# Copying and distribution of this file, with or without modification, +# are permitted in any medium without royalty provided the copyright +# notice and this notice are preserved. This file is offered as-is, +# without any warranty. + +file="$1" +dir="${file%/*}" +test "$dir" = "$file" && dir="." + +# We don't care if gdb gives an error. +gdb -nx -iex 'set auto-load no' --batch-silent \ + -ex "file $file" \ + -ex "save gdb-index $dir" + +if test -f "${file}.gdb-index"; then + objcopy --add-section .gdb_index="${file}.gdb-index" \ + --set-section-flags .gdb_index=readonly "$file" "$file" + rm -f "${file}.gdb-index" +fi + +exit 0 -- 1.8.4.rc3