Subject: Re: bug#8896: [PATCH] doc: replace obsolete @vindex entry with a useful one
From: Stefano Lattarini <[email-masked]>
To: Jim Meyering <[email-masked]>
Cc: [email-masked]
Date: Sun, 19 Jun 2011 13:19:59 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

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 <[email-masked]>
> 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  <[email-masked]>
> +
> +	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  <[email-masked]>
> 
>  	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

From f541f9a4c8a5016ed4d77e54f4a3d707a863feaa Mon Sep 17 00:00:00 2001
Message-Id: <f541f9a4c[email-masked]>
From: Stefano Lattarini <[email-masked]>
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  <[email-masked]>
+
+	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  <[email-masked]>
 
 	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