Re: Imprecise documentation -- uninstall-hook

[Tom Tromey]

>>>>> "adl" == Alexandre Duret-Lutz <address@hidden> writes:

Tom> Is there any reason to want an uninstall-hook?
Tom> Does the functionality make sense?

adl> While the second report can be rewritten using uninstall-local,
adl> the first (removing empty directories left after uninstall)
adl> seems to be a case where uninstall-local is not enough.

Sounds good to me.  I must have intended to have uninstall-hook but
then not implemented it.  Probably few noticed since uninstall really
isn't used too much.

I'm checking this in.  It adds uninstall-hook.

Tom

Index: ChangeLog
from  Tom Tromey  <address@hidden>

        * automake.texi (Extending): Mention uninstall-hook.
        (Install): Likewise.
        * automake.in (handle_factored_dependencies): Allow
        uninstall-hook.

Index: NEWS
===================================================================
RCS file: /cvs/automake/automake/NEWS,v
retrieving revision 1.164
diff -u -r1.164 NEWS
--- NEWS 2001/12/31 00:16:37 1.164
+++ NEWS 2002/01/01 21:59:57
@@ -20,6 +20,7 @@
 * Fixed support of implicit rules leading to .lo objects.
 * Fixed late inclusion of --add-missing files (e.g. depcomp) in DIST_COMMON
 * Added new AM_AUTOMAKE_OPTIONS macro
+* Added uninstall-hook target
 
 New in 1.5:
 * Support for `configure.ac'.
Index: automake.in
===================================================================
RCS file: /cvs/automake/automake/automake.in,v
retrieving revision 1.1253
diff -u -r1.1253 automake.in
--- automake.in 2002/01/01 02:34:05 1.1253
+++ automake.in 2002/01/01 22:00:03
@@ -4265,7 +4265,7 @@
 
     # Install the -hook hooks.
     # FIXME: Why not be as liberal as we are with -local hooks?
-    foreach ('install-exec', 'install-data')
+    foreach ('install-exec', 'install-data', 'uninstall')
     {
       if (&target_defined ("$_-hook"))
        {
Index: automake.texi
===================================================================
RCS file: /cvs/automake/automake/automake.texi,v
retrieving revision 1.255
diff -u -r1.255 automake.texi
--- automake.texi 2002/01/01 02:53:02 1.255
+++ automake.texi 2002/01/01 22:00:08
@@ -3534,8 +3534,8 @@
 @trindex installdirs
 @trindex install-strip
 
-Automake supports @code{uninstall-local}.  There is no @code{uninstall-hook}
-as well as no notion of separate uninstalls for ``exec'' and ``data'', as
+Automake supports @code{uninstall-local} and @code{uninstall-hook}.
+There is no notion of separate uninstalls for ``exec'' and ``data'', as
 these features would not provide additional functionality.
 
 Note that @code{uninstall} is not meant as a replacement for a real
@@ -4410,8 +4410,8 @@
 Some targets also have a way to run another target, called a @dfn{hook},
 after their work is done.  The hook is named after the principal target,
 with @samp{-hook} appended.  The targets allowing hooks are
address@hidden, @code{install-exec}, @code{dist}, and
address@hidden
address@hidden, @code{install-exec}, @code{uninstall}, @code{dist},
+and @code{distcheck}.
 @trindex install-data-hook
 @trindex install-exec-hook
 @trindex dist-hook
Index: stamp-vti
===================================================================
RCS file: /cvs/automake/automake/stamp-vti,v
retrieving revision 1.152
diff -u -r1.152 stamp-vti
--- stamp-vti 2002/01/01 02:53:02 1.152
+++ stamp-vti 2002/01/01 22:00:08
@@ -1,4 +1,4 @@
address@hidden UPDATED 31 December 2001
address@hidden UPDATED-MONTH December 2001
address@hidden UPDATED 1 January 2002
address@hidden UPDATED-MONTH January 2002
 @set EDITION 1.5c
 @set VERSION 1.5c
Index: version.texi
===================================================================
RCS file: /cvs/automake/automake/version.texi,v
retrieving revision 1.225
diff -u -r1.225 version.texi
--- version.texi 2002/01/01 02:53:02 1.225
+++ version.texi 2002/01/01 22:00:08
@@ -1,4 +1,4 @@
address@hidden UPDATED 31 December 2001
address@hidden UPDATED-MONTH December 2001
address@hidden UPDATED 1 January 2002
address@hidden UPDATED-MONTH January 2002
 @set EDITION 1.5c
 @set VERSION 1.5c