From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6240 invoked by alias); 23 Feb 2007 03:15:13 -0000 Received: (qmail 6230 invoked by uid 22791); 23 Feb 2007 03:15:10 -0000 X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (65.74.133.4) by sourceware.org (qpsmtpd/0.31) with ESMTP; Fri, 23 Feb 2007 03:15:00 +0000 Received: (qmail 32734 invoked from network); 23 Feb 2007 03:14:57 -0000 Received: from unknown (HELO digraph.polyomino.org.uk) (joseph@127.0.0.2) by mail.codesourcery.com with ESMTPA; 23 Feb 2007 03:14:57 -0000 Received: from jsm28 (helo=localhost) by digraph.polyomino.org.uk with local-esmtp (Exim 4.63) (envelope-from ) id 1HKQtc-0004Id-MB; Fri, 23 Feb 2007 03:14:56 +0000 Date: Fri, 23 Feb 2007 07:44:00 -0000 From: "Joseph S. Myers" To: Andreas Schwab cc: binutils@sourceware.org Subject: Re: Configure options --with-bugurl, --with-pkgversion In-Reply-To: Message-ID: References: <20070220140554.GA5035@caradoc.them.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Mailing-List: contact binutils-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: binutils-owner@sourceware.org X-SW-Source: 2007-02/txt/msg00344.txt.bz2 On Tue, 20 Feb 2007, Andreas Schwab wrote: > Daniel Jacobowitz writes: > > > On Tue, Feb 20, 2007 at 03:00:01PM +0100, Andreas Schwab wrote: > >> But that is not what pkgversion is supposed to do, if I understand > >> correctly. It should rather > >> specify an additional version tag (a vendor identifier) _extending_ the > >> existing one. > > > > Perhaps binutils should use the 2.17.50.20070220 notation by default > > for dated snapshots, as GDB does. > > Yes, I agree that this would definitely be a good thing. This patch implements the suggested change of using "." between version number and date so that the --version output conforms to the GNU Coding Standards requirement that the version proper be after the last space. It makes other fixes for conformance to those standards: "(GNU Binutils)" is used by default to indicate the common package, unless overridden by --with-pkgversion specifying another text to go there; the --version copyright dates are updated (recall that those dates should just be the most recent year, and don't need to include any older years), the stray word "version" is removed from the ld --version output and gprof now uses BFD_VERSION_STRING so it gets the full version including the date (and meaning the gprof directory no longer needs a copy of the PKGVERSION configure logic). OK to commit? bfd: 2007-02-23 Joseph Myers * Makefile.am (bfdver.h): Use "." not " " between version number and date. * Makefile.in: Regenerate. * configure.in (PKGVERSION): Default to "(GNU Binutils) ". * configure: Regenerate. binutils: 2007-02-23 Joseph Myers * version.c (print_version): Update copyright date. gas: 2007-02-23 Joseph Myers * as.c (parse_args): Update copyright date. gprof: 2007-02-23 Joseph Myers * gprof.c: Include bfdver.h (main): Use BFD_VERSION_STRING for version number and package name. * Makefile.am (PKGVERSION): Remove. (INCLUDES): Update. Regenerate dependencies. * Makefile.in: Regenerate. * configure.in (PKGVERSION): Remove. * configure: Regenerate. ld: 2007-02-23 Joseph Myers * ldver.c (ldversion): Remove word "version" from output. Update copyright date. Index: bfd/Makefile.am =================================================================== RCS file: /cvs/src/src/bfd/Makefile.am,v retrieving revision 1.187 diff -u -r1.187 Makefile.am --- bfd/Makefile.am 17 Feb 2007 13:33:52 -0000 1.187 +++ bfd/Makefile.am 23 Feb 2007 03:00:26 -0000 @@ -968,7 +968,7 @@ bfd_version_package="\"$(PKGVERSION)\"" ;\ if test "x$(RELEASE)" = x ; then \ bfd_version_date=`sed -n -e 's/.*DATE //p' < $(srcdir)/version.h` ;\ - bfd_version_string="\"$(VERSION) $${bfd_version_date}\"" ;\ + bfd_version_string="\"$(VERSION).$${bfd_version_date}\"" ;\ fi ;\ sed -e "s/@bfd_version@/$$bfd_version/" -e "s/@bfd_version_string@/$$bfd_version_string/" -e "s/@bfd_version_package@/$$bfd_version_package/" < $(srcdir)/version.h > $@ Index: bfd/configure.in =================================================================== RCS file: /cvs/src/src/bfd/configure.in,v retrieving revision 1.221 diff -u -r1.221 configure.in --- bfd/configure.in 17 Feb 2007 13:33:53 -0000 1.221 +++ bfd/configure.in 23 Feb 2007 03:00:28 -0000 @@ -72,7 +72,7 @@ no) PKGVERSION= ;; *) PKGVERSION="($withval) " ;; esac], - PKGVERSION= + PKGVERSION="(GNU Binutils) " ) AC_SUBST(PKGVERSION) Index: binutils/version.c =================================================================== RCS file: /cvs/src/src/binutils/version.c,v retrieving revision 1.11 diff -u -r1.11 version.c --- binutils/version.c 8 May 2005 14:17:39 -0000 1.11 +++ binutils/version.c 23 Feb 2007 03:00:28 -0000 @@ -1,6 +1,6 @@ /* version.c -- binutils version information - Copyright 1991, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005 - Free Software Foundation, Inc. + Copyright 1991, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, + 2007 Free Software Foundation, Inc. This file is part of GNU Binutils. @@ -32,7 +32,7 @@ /* This output is intended to follow the GNU standards document. */ /* xgettext:c-format */ printf ("GNU %s %s\n", name, BFD_VERSION_STRING); - printf (_("Copyright 2005 Free Software Foundation, Inc.\n")); + printf (_("Copyright 2007 Free Software Foundation, Inc.\n")); printf (_("\ This program is free software; you may redistribute it under the terms of\n\ the GNU General Public License. This program has absolutely no warranty.\n")); Index: gas/as.c =================================================================== RCS file: /cvs/src/src/gas/as.c,v retrieving revision 1.71 diff -u -r1.71 as.c --- gas/as.c 17 Feb 2007 13:33:55 -0000 1.71 +++ gas/as.c 23 Feb 2007 03:00:28 -0000 @@ -590,7 +590,7 @@ case OPTION_VERSION: /* This output is intended to follow the GNU standards document. */ printf (_("GNU assembler %s\n"), BFD_VERSION_STRING); - printf (_("Copyright 2005 Free Software Foundation, Inc.\n")); + printf (_("Copyright 2007 Free Software Foundation, Inc.\n")); printf (_("\ This program is free software; you may redistribute it under the terms of\n\ the GNU General Public License. This program has absolutely no warranty.\n")); Index: gprof/Makefile.am =================================================================== RCS file: /cvs/src/src/gprof/Makefile.am,v retrieving revision 1.31 diff -u -r1.31 Makefile.am --- gprof/Makefile.am 17 Feb 2007 13:33:57 -0000 1.31 +++ gprof/Makefile.am 23 Feb 2007 03:00:29 -0000 @@ -16,12 +16,10 @@ MKDEP = gcc -MM -PKGVERSION = "\"@PKGVERSION@\"" REPORT_BUGS_TO = "\"@REPORT_BUGS_TO@\"" INCLUDES = -D_GNU_SOURCE -DDEBUG -I../bfd -I$(srcdir)/../include \ -I$(srcdir)/../bfd @INCINTL@ -I. \ - -DPKGVERSION=$(PKGVERSION) \ -DREPORT_BUGS_TO=$(REPORT_BUGS_TO) \ -DLOCALEDIR="\"$(datadir)/locale\"" @@ -198,116 +196,115 @@ $(INCDIR)/ansidecl.h gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h basic_blocks.h corefile.h gmon_io.h gmon_out.h \ + gconfig.h basic_blocks.h corefile.h gmon_io.h gmon_out.h \ search_list.h source.h symtab.h sym_ids.h call_graph.o: call_graph.c gprof.h $(BFDDIR)/sysdep.h \ $(INCDIR)/ansidecl.h ../bfd/config.h $(INCDIR)/fopen-same.h \ $(INCDIR)/filenames.h ../bfd/bfd.h $(INCDIR)/ansidecl.h \ - $(INCDIR)/symcat.h ./gconfig.h search_list.h source.h \ + $(INCDIR)/symcat.h gconfig.h search_list.h source.h \ symtab.h cg_arcs.h call_graph.h corefile.h gmon_io.h \ gmon_out.h sym_ids.h cg_arcs.o: cg_arcs.c $(INCDIR)/libiberty.h $(INCDIR)/ansidecl.h \ gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h ../bfd/config.h \ $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h ../bfd/bfd.h \ - $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h ./gconfig.h \ - search_list.h source.h symtab.h call_graph.h cg_arcs.h \ - cg_dfn.h cg_print.h utils.h sym_ids.h + $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h gconfig.h search_list.h \ + source.h symtab.h call_graph.h cg_arcs.h cg_dfn.h cg_print.h \ + utils.h sym_ids.h cg_dfn.o: cg_dfn.c $(INCDIR)/libiberty.h $(INCDIR)/ansidecl.h \ gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h ../bfd/config.h \ $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h ../bfd/bfd.h \ - $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h ./gconfig.h \ - search_list.h source.h symtab.h cg_arcs.h cg_dfn.h \ - utils.h + $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h gconfig.h search_list.h \ + source.h symtab.h cg_arcs.h cg_dfn.h utils.h cg_print.o: cg_print.c $(INCDIR)/libiberty.h $(INCDIR)/ansidecl.h \ gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h ../bfd/config.h \ $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h ../bfd/bfd.h \ - $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h ./gconfig.h \ - search_list.h source.h symtab.h cg_arcs.h cg_print.h \ - hist.h utils.h corefile.h + $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h gconfig.h search_list.h \ + source.h symtab.h cg_arcs.h cg_print.h hist.h utils.h \ + corefile.h corefile.o: corefile.c $(INCDIR)/libiberty.h $(INCDIR)/ansidecl.h \ gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h ../bfd/config.h \ $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h ../bfd/bfd.h \ - $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h ./gconfig.h \ - search_list.h source.h symtab.h corefile.h + $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h gconfig.h search_list.h \ + source.h symtab.h corefile.h gmon_io.o: gmon_io.c gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h search_list.h source.h symtab.h cg_arcs.h \ + gconfig.h search_list.h source.h symtab.h cg_arcs.h \ basic_blocks.h corefile.h call_graph.h gmon_io.h gmon_out.h \ gmon.h hertz.h hist.h $(INCDIR)/libiberty.h $(INCDIR)/ansidecl.h gprof.o: gprof.c $(INCDIR)/libiberty.h $(INCDIR)/ansidecl.h \ gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h ../bfd/config.h \ $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h ../bfd/bfd.h \ - $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h ./gconfig.h \ + $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h gconfig.h ../bfd/bfdver.h \ search_list.h source.h symtab.h basic_blocks.h call_graph.h \ cg_arcs.h cg_print.h corefile.h gmon_io.h hertz.h hist.h \ sym_ids.h $(INCDIR)/demangle.h $(INCDIR)/libiberty.h hertz.o: hertz.c gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h hertz.h + gconfig.h hertz.h hist.o: hist.c $(INCDIR)/libiberty.h $(INCDIR)/ansidecl.h \ gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h ../bfd/config.h \ $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h ../bfd/bfd.h \ - $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h ./gconfig.h \ - search_list.h source.h symtab.h corefile.h gmon_io.h \ - gmon_out.h hist.h sym_ids.h utils.h + $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h gconfig.h search_list.h \ + source.h symtab.h corefile.h gmon_io.h gmon_out.h hist.h \ + sym_ids.h utils.h source.o: source.c gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h $(INCDIR)/libiberty.h $(INCDIR)/ansidecl.h \ + gconfig.h $(INCDIR)/libiberty.h $(INCDIR)/ansidecl.h \ $(INCDIR)/filenames.h search_list.h source.h search_list.o: search_list.c $(INCDIR)/libiberty.h \ $(INCDIR)/ansidecl.h gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h search_list.h + gconfig.h search_list.h symtab.o: symtab.c gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h search_list.h source.h symtab.h cg_arcs.h \ + gconfig.h search_list.h source.h symtab.h cg_arcs.h \ corefile.h sym_ids.o: sym_ids.c $(INCDIR)/libiberty.h $(INCDIR)/ansidecl.h \ $(INCDIR)/safe-ctype.h gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h search_list.h source.h symtab.h cg_arcs.h \ + gconfig.h search_list.h source.h symtab.h cg_arcs.h \ sym_ids.h utils.o: utils.c $(INCDIR)/demangle.h $(INCDIR)/libiberty.h \ $(INCDIR)/ansidecl.h gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h search_list.h source.h symtab.h cg_arcs.h \ + gconfig.h search_list.h source.h symtab.h cg_arcs.h \ utils.h i386.o: i386.c gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h search_list.h source.h symtab.h cg_arcs.h \ + gconfig.h search_list.h source.h symtab.h cg_arcs.h \ corefile.h hist.h alpha.o: alpha.c gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h search_list.h source.h symtab.h cg_arcs.h \ + gconfig.h search_list.h source.h symtab.h cg_arcs.h \ corefile.h hist.h vax.o: vax.c gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h search_list.h source.h symtab.h cg_arcs.h \ + gconfig.h search_list.h source.h symtab.h cg_arcs.h \ corefile.h hist.h tahoe.o: tahoe.c gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h search_list.h source.h symtab.h cg_arcs.h \ + gconfig.h search_list.h source.h symtab.h cg_arcs.h \ corefile.h hist.h sparc.o: sparc.c gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h search_list.h source.h symtab.h cg_arcs.h \ + gconfig.h search_list.h source.h symtab.h cg_arcs.h \ corefile.h hist.h mips.o: mips.c gprof.h $(BFDDIR)/sysdep.h $(INCDIR)/ansidecl.h \ ../bfd/config.h $(INCDIR)/fopen-same.h $(INCDIR)/filenames.h \ ../bfd/bfd.h $(INCDIR)/ansidecl.h $(INCDIR)/symcat.h \ - ./gconfig.h search_list.h source.h symtab.h cg_arcs.h \ + gconfig.h search_list.h source.h symtab.h cg_arcs.h \ corefile.h hist.h flat_bl.o: flat_bl.c $(INCDIR)/ansidecl.h bsd_callg_bl.o: bsd_callg_bl.c $(INCDIR)/ansidecl.h Index: gprof/configure.in =================================================================== RCS file: /cvs/src/src/gprof/configure.in,v retrieving revision 1.32 diff -u -r1.32 configure.in --- gprof/configure.in 17 Feb 2007 13:33:57 -0000 1.32 +++ gprof/configure.in 23 Feb 2007 03:00:30 -0000 @@ -13,19 +13,6 @@ AM_PROG_LIBTOOL -# Package version. For an official FSF release, it is empty. -AC_ARG_WITH(pkgversion, - AS_HELP_STRING([--with-pkgversion=PKG], - [Add PKG to the version string]), - [case "$withval" in - yes) AC_MSG_ERROR([package version not specified]) ;; - no) PKGVERSION= ;; - *) PKGVERSION="($withval) " ;; - esac], - PKGVERSION= -) -AC_SUBST(PKGVERSION) - # The location to which bugs should be reported. AC_ARG_WITH(bugurl, AS_HELP_STRING([--with-bugurl=URL], Index: gprof/gprof.c =================================================================== RCS file: /cvs/src/src/gprof/gprof.c,v retrieving revision 1.29 diff -u -r1.29 gprof.c --- gprof/gprof.c 17 Feb 2007 13:33:57 -0000 1.29 +++ gprof/gprof.c 23 Feb 2007 03:00:30 -0000 @@ -29,6 +29,7 @@ #include "libiberty.h" #include "gprof.h" +#include "bfdver.h" #include "search_list.h" #include "source.h" #include "symtab.h" @@ -411,7 +412,7 @@ break; case 'v': /* This output is intended to follow the GNU standards document. */ - printf (_("GNU gprof %s\n"), PKGVERSION VERSION); + printf (_("GNU gprof %s\n"), BFD_VERSION_STRING); printf (_("Based on BSD gprof, copyright 1983 Regents of the University of California.\n")); printf (_("\ This program is free software. This program has absolutely no warranty.\n")); Index: ld/ldver.c =================================================================== RCS file: /cvs/src/src/ld/ldver.c,v retrieving revision 1.11 diff -u -r1.11 ldver.c --- ld/ldver.c 12 May 2005 07:32:03 -0000 1.11 +++ ld/ldver.c 23 Feb 2007 03:00:30 -0000 @@ -1,6 +1,6 @@ /* ldver.c -- Print linker version. Copyright 1991, 1992, 1993, 1994, 1995, 1996, 1997, 2000, 2001, 2002, - 2003 Free Software Foundation, Inc. + 2003, 2007 Free Software Foundation, Inc. This file is part of GLD, the Gnu Linker. @@ -35,11 +35,11 @@ ldversion (int noisy) { /* Output for noisy == 2 is intended to follow the GNU standards. */ - fprintf (stdout, _("GNU ld version %s\n"), BFD_VERSION_STRING); + fprintf (stdout, _("GNU ld %s\n"), BFD_VERSION_STRING); if (noisy & 2) { - printf (_("Copyright 2005 Free Software Foundation, Inc.\n")); + printf (_("Copyright 2007 Free Software Foundation, Inc.\n")); printf (_("\ This program is free software; you may redistribute it under the terms of\n\ the GNU General Public License. This program has absolutely no warranty.\n")); -- Joseph S. Myers joseph@codesourcery.com