From: Andreas Koenig <k@anna.in-berlin.de>
Date: Fri, 10 Jan 1997 12:43:39 +0000 (+0100)
Subject: Re: MakeMaker and 'make uninstall'
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=54bc88081d9355e4567ce06a1f7b14ac5a7a5334;p=p5sagit%2Fp5-mst-13.2.git

Re: MakeMaker and 'make uninstall'

>>>>> Hugo van der Sanden writes:

  > I tried uninstalling a package today, the first time I've done so. The
  > package was MailFolder0.03, and as far as I can see the uninstall:
  > target in the Makefile was provided by MakeMaker. This does:

[...]

  > and FULLEXT is defined as 'Mail'. The result was that most of the
  > MailTools package was also removed.

Thanks for the report and _Sorry_.

Folks, I think, I have to deprecate the uninstall target. It was never
documented, it was never really clever, it just caused me a lot of
grief. I'd rather design a new solution from scratch.

If nobody objects, I'd recommend the following patch to
lib/ExtUtils/MM_Unix.pm. It makes use of the "nonono" switch of the
uninstall routine and will not uninstall anything.

Chip, can you sneak that in, please? Thanks!

p5p-msgid: <199701101243.NAA26400@anna.in-berlin.de>
---

diff --git a/lib/ExtUtils/MM_Unix.pm b/lib/ExtUtils/MM_Unix.pm
index f4ba1d7..e64423f 100644
--- a/lib/ExtUtils/MM_Unix.pm
+++ b/lib/ExtUtils/MM_Unix.pm
@@ -2956,8 +2956,9 @@ DOC_INSTALL = $(PERL) -e '$$\="\n\n";' \
 -e 'print "=back";'
 
 UNINSTALL =   $(PERL) -MExtUtils::Install \
--e 'uninstall($$ARGV[0],1);'
-
+-e 'uninstall($$ARGV[0],1,1); print "\nUninstall is deprecated. Please check the";' \
+-e 'print " packlist above carefully.\n  There may be errors. Remove the";' \
+-e 'print " appropriate files manually.\n  Sorry for the inconveniences.\n"'
 };
 
     return join "", @m;