GNU bug report logs - #8896
[PATCH] doc: replace obsolete @vindex entry with a useful one

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 8896 in the body.
You can then email your comments to 8896 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: bug-automake <at> gnu.org
Subject: [PATCH] doc: replace obsolete @vindex entry with a useful one
Date: Sun, 19 Jun 2011 12:32:59 +0200

Here's a small fix for master:

From 5115c011c2f6689508020612ecee97775da7687f Mon Sep 17 00:00:00 2001
From: Jim Meyering <meyering <at> redhat.com>
Date: Sun, 19 Jun 2011 12:29:18 +0200
Subject: [PATCH] doc: replace obsolete @vindex entry with a useful one

* doc/automake.texi (Program Sources): Do not index obsolete
pkglib_PROGRAMS here.  Do index pkglibexec_PROGRAMS.
---
 ChangeLog         |    6 ++++++
 doc/automake.texi |    2 +-
 2 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index e09b24a..d702976 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2011-06-19  Jim Meyering  <meyering <at> redhat.com>
+
+	doc: replace obsolete @vindex entry with a useful one
+	* doc/automake.texi (Program Sources): Do not index obsolete
+	pkglib_PROGRAMS here.  Do index pkglibexec_PROGRAMS.
+
 2011-06-18  Stefano Lattarini  <stefano.lattarini <at> gmail.com>

 	tests: more checks on portable fd redirection in TESTS_ENVIRONMENT
diff --git a/doc/automake.texi b/doc/automake.texi
index 7e84fdd..2c91ec5 100644
--- a/doc/automake.texi
+++ b/doc/automake.texi
@@ -4659,7 +4659,7 @@ Program Sources
 @vindex bin_PROGRAMS
 @vindex sbin_PROGRAMS
 @vindex libexec_PROGRAMS
-@vindex pkglib_PROGRAMS
+@vindex pkglibexec_PROGRAMS
 @vindex noinst_PROGRAMS
 @vindex check_PROGRAMS

--
1.7.6.rc2.4.g36bfb.dirty

Message #10 received at 8896-done <at> debbugs.gnu.org (full text, mbox):

From: Stefano Lattarini <stefano.lattarini <at> gmail.com>
To: Jim Meyering <jim <at> meyering.net>
Cc: 8896-done <at> debbugs.gnu.org
Subject: Re: bug#8896: [PATCH] doc: replace obsolete @vindex entry with a
	useful one
Date: Sun, 19 Jun 2011 13:19:59 +0200

[Message part 1 (text/plain, inline)]

Hi Jim, thanks for the patch.

However, note that patches for automake should be posted to the
'automake-patches' list; 'bug-automake' is only for bug reporting
(as any message posted here open a new entry in the debbugs tracker).

On Sunday 19 June 2011, Jim Meyering wrote:
> Here's a small fix for master:
>
Indeed it's good for maint too.  I've applied it to maint, and merged
to master.  See more below.

> From 5115c011c2f6689508020612ecee97775da7687f Mon Sep 17 00:00:00 2001
> From: Jim Meyering <meyering <at> redhat.com>
> Date: Sun, 19 Jun 2011 12:29:18 +0200
> Subject: [PATCH] doc: replace obsolete @vindex entry with a useful one
> 
> * doc/automake.texi (Program Sources): Do not index obsolete
> pkglib_PROGRAMS here.  Do index pkglibexec_PROGRAMS.
> ---
>  ChangeLog         |    6 ++++++
>  doc/automake.texi |    2 +-
>  2 files changed, 7 insertions(+), 1 deletions(-)
> 
> diff --git a/ChangeLog b/ChangeLog
> index e09b24a..d702976 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,3 +1,9 @@
> +2011-06-19  Jim Meyering  <meyering <at> redhat.com>
> +
> +	doc: replace obsolete @vindex entry with a useful one
> +	* doc/automake.texi (Program Sources): Do not index obsolete
> +	pkglib_PROGRAMS here.  Do index pkglibexec_PROGRAMS.
> +
>  2011-06-18  Stefano Lattarini  <stefano.lattarini <at> gmail.com>
> 
>  	tests: more checks on portable fd redirection in TESTS_ENVIRONMENT
> diff --git a/doc/automake.texi b/doc/automake.texi
> index 7e84fdd..2c91ec5 100644
> --- a/doc/automake.texi
> +++ b/doc/automake.texi
> @@ -4659,7 +4659,7 @@ Program Sources
>  @vindex bin_PROGRAMS
>  @vindex sbin_PROGRAMS
>  @vindex libexec_PROGRAMS
> -@vindex pkglib_PROGRAMS
> +@vindex pkglibexec_PROGRAMS
>  @vindex noinst_PROGRAMS
>  @vindex check_PROGRAMS
> 
> --
> 1.7.6.rc2.4.g36bfb.dirty
> 
> 
In fact, after my recentish commit `v1.11-373-g9ca6326', "Warnings about
primary/prefix mismatch fixed and extended.", there is an error in the
documentation, that your patch has brought to my attention.  Fixed with
the attached patch (for maint).

Regards,
  Stefano

[0001-docs-primary-prefix-combination-pkglib_PROGRAMS-is-n.patch (text/x-patch, inline)]

From f541f9a4c8a5016ed4d77e54f4a3d707a863feaa Mon Sep 17 00:00:00 2001
Message-Id: <f541f9a4c8a5016ed4d77e54f4a3d707a863feaa.1308482159.git.stefano.lattarini <at> gmail.com>
From: Stefano Lattarini <stefano.lattarini <at> gmail.com>
Date: Sun, 19 Jun 2011 13:15:49 +0200
Subject: [PATCH] docs: primary/prefix combination "pkglib_PROGRAMS" is now invalid

* doc/automake.texi (Program Sources): pkglib_PROGRAMS is not a
valid combination anymore, so don't document it.  Inconsistency
introduced in commit `v1.11-373-g9ca6326'.
---
 ChangeLog         |    7 +++++++
 doc/automake.texi |    2 +-
 2 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 08814ad..917c2d4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2011-06-19  Stefano Lattarini  <stefano.lattarini <at> gmail.com>
+
+	docs: primary/prefix combination "pkglib_PROGRAMS" is now invalid
+	* doc/automake.texi (Program Sources): pkglib_PROGRAMS is not a
+	valid combination anymore, so don't document it.  Inconsistency
+	introduced in commit `v1.11-373-g9ca6326'.
+
 2011-06-19  Jim Meyering  <meyering <at> redhat.com>
 
 	docs: replace obsolete @vindex entry with a useful one
diff --git a/doc/automake.texi b/doc/automake.texi
index 1f609c7..4209c07 100644
--- a/doc/automake.texi
+++ b/doc/automake.texi
@@ -4649,7 +4649,7 @@ programs.  Most of the comments about these also apply to libraries
 In a directory containing source that gets built into a program (as
 opposed to a library or a script), the @code{PROGRAMS} primary is used.
 Programs can be installed in @code{bindir}, @code{sbindir},
-@code{libexecdir}, @code{pkglibdir}, @code{pkglibexecdir}, or not at all
+@code{libexecdir}, @code{pkglibexecdir}, or not at all
 (@code{noinst_}).  They can also be built only for @samp{make check}, in
 which case the prefix is @samp{check_}.
 
-- 
1.7.2.3

Message #11 received at 8896-done <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Stefano Lattarini <stefano.lattarini <at> gmail.com>
Cc: 8896-done <at> debbugs.gnu.org
Subject: Re: bug#8896: [PATCH] doc: replace obsolete @vindex entry with a
	useful one
Date: Sun, 19 Jun 2011 14:13:20 +0200

Stefano Lattarini wrote:
> Hi Jim, thanks for the patch.
>
> However, note that patches for automake should be posted to the
> 'automake-patches' list; 'bug-automake' is only for bug reporting
> (as any message posted here open a new entry in the debbugs tracker).

Hi Stefano,

Thanks, I knew that, and even recall thinking I'd send it to that list.
My fingers must have overruled and typed the wrong thing (bug-PKGNAME)
by force of habit.

I'll watch them more carefully next time.

Jim

P.S. I even went to my record of outgoing messages to confirm.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.