Mailing-List: contact lucy-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: lucy-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: local policy)
Date: Sat, 22 Oct 2011 12:43:54 -0700
From: Marvin Humphrey <marvin@rectangular.com>
To: lucy-dev@incubator.apache.org
Message-ID: <20111022194354.GA6712@rectangular.com>
References: <4E9F98AC.7050602@peknet.com> <4EA30AC7.1020909@faui2k3.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4EA30AC7.1020909@faui2k3.org>
User-Agent: Mutt/1.5.18 (2008-05-17)
Subject: Re: [lucy-dev] [VOTE] Apache Lucy (incubating) 0.2.2 RC 1 (cleanup)

Hi Moritz, thanks for reviewing the release candidate and reporting your
findings!

On Sat, Oct 22, 2011 at 08:26:15PM +0200, Moritz Lenz wrote:
> FWIW all tests pass on Debian GNU/Linux stable with a custom perl
> 5.14.1, same with the system perl (5.10.1).

:)

> However I had to wipe out the directory between the two tests, because
> the build and configuration left some files behind which ./Build clean
> in the perl/ subdirectory did not clean, and which caused some @INC
> paths to be set up wrongly.

I believe that this patch to the 0.2 branch would produce your desired
behavior:

--- perl/buildlib/Lucy/Build.pm (revision 1187752)
+++ perl/buildlib/Lucy/Build.pm (working copy)
@@ -716,8 +716,8 @@
 sub ACTION_clean {
     my $self = shift;
     if ( -e $CLOWNFISH_BUILD ) {
-        system("$^X $CLOWNFISH_BUILD clean")
-            and die "Clownfish clean failed";
+        system("$^X $CLOWNFISH_BUILD realclean")
+            and die "Clownfish realclean failed";
     }
     $self->_run_make( dir => $CHARMONIZER_ORIG_DIR, args => ['clean'] );
     $self->SUPER::ACTION_clean;

I'm not sure that's technically "correct", because I think you are supposed to
supposed to invoke './Build realclean' rather than './Build clean' when
switching Perl executables.  However, it seems that './Build clean' works
under ordinary circumstances whether or not it ought to, and so +1 to make it
work for Lucy.

Can you please open a ticket?

    http://issues.apache.org/jira/browse/LUCY

Marvin Humphrey