Bug #40698 for ExtUtils-MakeMaker: Rejigger basic.t

Wed Nov 05 17:50:12 2008 yves [...] cpan.org - Ticket created

Please see perl core change: http://public.activestate.com/cgi-bin/perlbrowse/p/34742 This includes updates from ExtUtils-Install 1.50_04

Fri Nov 07 02:39:18 2008 SREZIC [...] cpan.org - Correspondence added

On Wed Nov 05 17:50:12 2008, YVES wrote: Show quoted text

> Please see perl core change: > > http://public.activestate.com/cgi-bin/perlbrowse/p/34742 > > This includes updates from ExtUtils-Install 1.50_04

This probably explains the only ExtUtils-MakeMaker test failure: http://www.nntp.perl.org/group/perl.cpan.testers/2008/11/msg2559750.html Regards, Slaven

Fri Nov 07 02:39:20 2008 The RT System itself - Status changed from 'new' to 'open'

Fri Nov 07 04:31:40 2008 demerphq [...] gmail.com - Correspondence added

CC:	yves [...] cpan.org
Subject:	Re: [rt.cpan.org #40698]
Date:	Fri, 7 Nov 2008 10:31:30 +0100
To:	bug-ExtUtils-MakeMaker [...] rt.cpan.org
From:	demerphq <demerphq [...] gmail.com>

2008/11/7 Slaven_Rezic via RT <bug-ExtUtils-MakeMaker@rt.cpan.org>: Show quoted text

> <URL: http://rt.cpan.org/Ticket/Display.html?id=40698 > > > On Wed Nov 05 17:50:12 2008, YVES wrote:

>> Please see perl core change: >> >> http://public.activestate.com/cgi-bin/perlbrowse/p/34742 >> >> This includes updates from ExtUtils-Install 1.50_04

> > This probably explains the only ExtUtils-MakeMaker test failure: > http://www.nntp.perl.org/group/perl.cpan.testers/2008/11/msg2559750.html

Correct. The test needs to be changed to invoke "verbinst". Yves -- perl -Mre=debug -e "/just|another|perl|hacker/"

Thu Nov 13 13:52:51 2008 mschwern [...] cpan.org - Subject changed from (no value) to 'Rejigger basic.t'

Thu Nov 13 13:53:05 2008 mschwern [...] cpan.org - Severity Normal added

Fri Apr 03 07:33:58 2009 mschwern [...] cpan.org - Correspondence added

basic.t was fixed to handle EUI formatting changes earlier. EUI and MakeMaker's basic.t will not be merging. Instead, MakeMaker will be getting its own ext/ directory.

Fri Apr 03 07:33:59 2009 mschwern [...] cpan.org - Status changed from 'open' to 'resolved'

Sun Jul 26 16:10:13 2009 demerphq [...] gmail.com - Correspondence added

CC:	yves [...] cpan.org
Subject:	Re: [rt.cpan.org #40698] Rejigger basic.t
Date:	Sun, 26 Jul 2009 22:09:53 +0200
To:	bug-ExtUtils-MakeMaker [...] rt.cpan.org
From:	demerphq <demerphq [...] gmail.com>

2009/4/3 Michael G Schwern via RT <bug-ExtUtils-MakeMaker@rt.cpan.org>: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=40698 > > > basic.t was fixed to handle EUI formatting changes earlier. > > EUI and MakeMaker's basic.t will not be merging. Instead, MakeMaker > will be getting its own ext/ directory.

MakeMaker getting its own ext directory does not resolve the core problem here. And i really wish, especially given the extreme lag of responding to this ticket that you had been more proactive about contacting me about this. We now have totally out of sync tests, with I suspect EUMM missing a number of EUI tests, and probably vice versa. And as I said, EUMM and EUI getting their own ext directory does NOT solve this problem. cheers Yves -- perl -Mre=debug -e "/just|another|perl|hacker/"

Sun Jul 26 16:10:14 2009 The RT System itself - Status changed from 'resolved' to 'open'

Sun Jul 26 18:01:13 2009 schwern [...] pobox.com - Correspondence added

Subject:	Re: [rt.cpan.org #40698] Rejigger basic.t
Date:	Sun, 26 Jul 2009 15:00:51 -0700
To:	bug-ExtUtils-MakeMaker [...] rt.cpan.org
From:	Michael G Schwern <schwern [...] pobox.com>

demerphq via RT wrote: Show quoted text

> Queue: ExtUtils-MakeMaker > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=40698 > > > 2009/4/3 Michael G Schwern via RT <bug-ExtUtils-MakeMaker@rt.cpan.org>:

>> <URL: https://rt.cpan.org/Ticket/Display.html?id=40698 > >> >> basic.t was fixed to handle EUI formatting changes earlier. >> >> EUI and MakeMaker's basic.t will not be merging. Instead, MakeMaker >> will be getting its own ext/ directory.

> > MakeMaker getting its own ext directory does not resolve the core problem here. > > And i really wish, especially given the extreme lag of responding to > this ticket that you had been more proactive about contacting me about > this. We now have totally out of sync tests, with I suspect EUMM > missing a number of EUI tests, and probably vice versa.

Sorry, I thought we did talk about it. Show quoted text

> And as I said, EUMM and EUI getting their own ext directory does NOT > solve this problem.

That's fine. EUI is a dependency which MM relies on to work as documented and does not need to thoroughly test. Any EUI bugs which get reported to MM get pushed upstream so I don't think I've added much in the way of EUI specific tests. If you've been adding MM tests you should push them downstream. The separation will further promote the independence of EUI from MM. -- 44. I am not the atheist chaplain. -- The 213 Things Skippy Is No Longer Allowed To Do In The U.S. Army http://skippyslist.com/list/

Sun Jul 26 18:19:58 2009 demerphq [...] gmail.com - Correspondence added

CC:	yves [...] cpan.org
Subject:	Re: [rt.cpan.org #40698] Rejigger basic.t
Date:	Mon, 27 Jul 2009 00:19:40 +0200
To:	bug-ExtUtils-MakeMaker [...] rt.cpan.org
From:	demerphq <demerphq [...] gmail.com>

2009/7/27 Michael G Schwern via RT <bug-ExtUtils-MakeMaker@rt.cpan.org>: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=40698 > > > demerphq via RT wrote:

>> Queue: ExtUtils-MakeMaker >> Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=40698 > >> >> 2009/4/3 Michael G Schwern via RT <bug-ExtUtils-MakeMaker@rt.cpan.org>:

>>> <URL: https://rt.cpan.org/Ticket/Display.html?id=40698 > >>> >>> basic.t was fixed to handle EUI formatting changes earlier. >>> >>> EUI and MakeMaker's basic.t will not be merging. Instead, MakeMaker >>> will be getting its own ext/ directory.

>> >> MakeMaker getting its own ext directory does not resolve the core problem here. >> >> And i really wish, especially given the extreme lag of responding to >> this ticket that you had been more proactive about contacting me about >> this. We now have totally out of sync tests, with I suspect EUMM >> missing a number of EUI tests, and probably vice versa.

> > Sorry, I thought we did talk about it.

No i dont believe that we did. :-) Show quoted text

>

>> And as I said, EUMM and EUI getting their own ext directory does NOT >> solve this problem.

> > That's fine. EUI is a dependency which MM relies on to work as documented and > does not need to thoroughly test. Any EUI bugs which get reported to MM get > pushed upstream so I don't think I've added much in the way of EUI specific > tests. If you've been adding MM tests you should push them downstream. > > The separation will further promote the independence of EUI from MM.

Sure, but /it doesnt address/ the point of this thread or the patch you rejected. Can we please arrange to be in the same IRC channel soon, and please read my comment in the p5p thread about the core aspect of this so we can get it sorted? I need to keep the EUI tests somewhat synced with the EUMM versions, and you need to keep some of your tests synced with the EUI version. We can do this most easily be separating the pieces and isolated the shared bits, or we can do it by integrating the pieces and I can selectively not run parts that cause problems. Thanks, yves -- perl -Mre=debug -e "/just|another|perl|hacker/"

Tue Jul 28 13:30:31 2009 mschwern [...] cpan.org - Correspondence added

Yves and I talked it out on IRC. I chopped all but the EUI specific bits out of EUI's basic.t and called it MakeMaker.t to provide a minimal "I didn't break MakeMaker" test. It should be simple enough that we're unlikely to have to sync changes back and forth.

Tue Jul 28 13:30:34 2009 mschwern [...] cpan.org - Status changed from 'open' to 'rejected'