Bug #30374 for Mac-Carbon: Mac::Carbon fails tests on Leopard

This queue is for tickets about the Mac-Carbon CPAN distribution.

Report information

The Basics

Id:	30374
Status:	resolved
Priority:	0/
Queue:	Mac-Carbon

People

Owner:	Nobody in particular
Requestors:	markemer [...] cpan.org
Cc:
AdminCc:

Bug Information

Severity:	Critical
Broken in:	0.77
Fixed in:	(no value)

Attachments

Mac-Carbon-AppleEvents-t-event.t.diff

Thu Feb 21 16:33:25 2008 (1.3k) by jeff [...] zeroclue.com

History Show all quoted text — Show full headers

# Tue Oct 30 17:57:21 2007 markemer [...] cpan.org - Ticket created

Subject:

Mac::Carbon fails tests on Leopard

Download (untitled) / with headers
text/plain 322b

Mac Carbon fails to test on Leopard, Perl 5.8.8 with this message: AppleEvents/t/event....ok 1/305 Too many arguments to AEBuildParameters() at AppleEvents/t/event.t line 162, <DATA> line 1. # Looks like you planned 305 tests but only ran 145. # Looks like your test died just after 145. All other tests seem to work ok

# Tue Oct 30 18:12:24 2007 cnandor [...] cpan.org - Correspondence added

CC:	undisclosed-recipients: ;
Subject:	Re: [rt.cpan.org #30374] Mac::Carbon fails tests on Leopard
Date:	Tue, 30 Oct 2007 15:09:51 -0700
To:	bug-Mac-Carbon [...] rt.cpan.org
From:	Chris Nandor <cnandor [...] cpan.org>

Download (untitled) / with headers
text/plain 1000b

That's weird, since I think Mac::Carbon is included with Leopard, and is built for it. I'll look into it. At 17:57 -0400 2007.10.30, Mark Anderson via RT wrote: Show quoted text

>Tue Oct 30 17:57:21 2007: Request 30374 was acted upon. >Transaction: Ticket created by MARKEMER > Queue: Mac-Carbon > Subject: Mac::Carbon fails tests on Leopard > Broken in: 0.77 > Severity: Critical > Owner: Nobody > Requestors: MARKEMER@cpan.org > Status: new > Ticket <URL: rt.cpan.org/Ticket/Display.html?id=30374 > > > >Mac Carbon fails to test on Leopard, Perl 5.8.8 with this message: >AppleEvents/t/event....ok 1/305 >Too many arguments to AEBuildParameters() at AppleEvents/t/event.t line >162, <DATA> line 1. ># Looks like you planned 305 tests but only ran 145. ># Looks like your test died just after 145. > >All other tests seem to work ok

-- Chris Nandor pudge@pobox.com pudge.net/ Slashdot / SourceForge pudge@slashdot.org slashdot.org/

# Tue Oct 30 18:12:25 2007 The RT System itself - Status changed from 'new' to 'open'

# Wed Dec 19 13:12:41 2007 essuu [...] ourshack.com - Correspondence added

From:

essuu [...] ourshack.com

Download (untitled) / with headers
text/plain 423b

Hi there, On Tue Oct 30 18:12:24 2007, CNANDOR wrote: Show quoted text

> That's weird, since I think Mac::Carbon is included with Leopard, and is > built for it. > > I'll look into it.

Just as an update to this ticket, I'm seeing the same problem trying to install Mac::Carbon under the shiny new perl5.10.0. It would be great to see this fixed so that 5.10.0 is fully usable on Leopard. Just sorry I have no idea how to fix it. Simon.

# Thu Feb 21 16:33:23 2008 jeff [...] zeroclue.com - Correspondence added

From:

jeff [...] zeroclue.com

Download (untitled) / with headers
text/plain 448b

For what it's worth, the commented out $fmt on line 159 of AppleEvents/t/event.t works fine. The diff is attached. $ prove -b AppleEvents/t/event.t AppleEvents/t/event....ok 33/305 skipped: various reasons All tests successful, 33 subtests skipped. Files=1, Tests=305, 0 wallclock secs ( 0.14 cusr + 0.02 csys = 0.16 CPU) $ perl -v This is perl, v5.10.0 built for darwin-2level

Download Mac-Carbon-AppleEvents-t-event.t.diff
text/x-diff 1.3k

--- ../Mac-Carbon-0.77/AppleEvents/t/event.t 2006-06-21 20:23:25.000000000 -0700 +++ AppleEvents/t/event.t 2008-02-21 13:26:33.000000000 -0800 @@ -155,8 +155,8 @@ } #diag($filedesc_print); - my $fmt = "'----':'obj '{ 'form':'prop', 'want':type(prop), 'from':$filedesc_print, 'seld':type(pnam) }"; - # $fmt = q"'----':'obj '{ 'form':prop, 'want':type(prop), 'from':'obj '{ 'want':type(docf), 'from':'obj '{ 'want':type(cfol), 'from':'obj '{ 'want':type(cobj), 'from':'obj '{ 'want':type(prop), 'from':'null'(), 'form':prop, 'seld':type(sdsk) }, 'form':name, 'seld':'utxt'($0070007200690076006100740065$) }, 'form':name, 'seld':'utxt'($0074006D0070$) }, 'form':name, 'seld':'TEXT'(@) }, 'seld':type(pnam) }"; + #my $fmt = "'----':'obj '{ 'form':'prop', 'want':type(prop), 'from':$filedesc_print, 'seld':type(pnam) }"; + my $fmt = q"'----':'obj '{ 'form':prop, 'want':type(prop), 'from':'obj '{ 'want':type(docf), 'from':'obj '{ 'want':type(cfol), 'from':'obj '{ 'want':type(cobj), 'from':'obj '{ 'want':type(prop), 'from':'null'(), 'form':prop, 'seld':type(sdsk) }, 'form':name, 'seld':'utxt'($0070007200690076006100740065$) }, 'form':name, 'seld':'utxt'($0074006D0070$) }, 'form':name, 'seld':'TEXT'(@) }, 'seld':type(pnam) }"; #diag($fmt); ok(AEBuildParameters($event, $fmt, 'mac-carbon-aeevent-test'), 'AEBuildParameters');

# Tue Dec 23 15:29:46 2008 tmb [...] nanite.net - Correspondence added

From:

tmb [...] nanite.net

Download (untitled) / with headers
text/plain 1k

On Thu Feb 21 16:33:23 2008, jeff@zeroclue.com wrote: Show quoted text

> > > For what it's worth, the commented out $fmt on line 159 of > AppleEvents/t/event.t works fine. The diff is attached. > > $ prove -b AppleEvents/t/event.t > AppleEvents/t/event....ok > > 33/305 skipped: various reasons > All tests successful, 33 subtests skipped. > Files=1, Tests=305, 0 wallclock secs ( 0.14 cusr + 0.02 csys = 0.16 CPU) > > $ perl -v > > This is perl, v5.10.0 built for darwin-2level

Just confirming that Jeff's patch / workaround worked for me as well. Uncommenting line 159 makes all the tests successful and makes it build properly on MacOS X 10.5.6, Intel, perl 5.10.0. I hit this issue while trying to install an updated Bundle::CPAN, which required File::HomeDir (more specifically File::HomeDir::Darwin), which required Mac::Files which required Mac::Carbon. It would be great to see this patched and released to CPAN so others won't have any issues building this module on MacOS X 10.5.

# Tue Dec 23 15:33:44 2008 cnandor [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #30374] Mac::Carbon fails tests on Leopard
Date:	Tue, 23 Dec 2008 12:33:29 -0800
To:	bug-Mac-Carbon [...] rt.cpan.org
From:	Chris Nandor <cnandor [...] cpan.org>

Download (untitled) / with headers
text/plain 413b

Yeah, I need to do that, and hope to do it soon. Thanks for the ping. On Dec 23, 2008, at 12:29, Mason Brown via RT wrote: Show quoted text

> It would be great to see this patched and released to CPAN so others > won't have any issues > building this module on MacOS X 10.5.

-- Chris Nandor pudge@pobox.com pudge.net/ Slashdot / SourceForge pudge@slashdot.org slashdot.org/

# Fri Jan 09 04:02:26 2009 polettix [...] cpan.org - Correspondence added

Download (untitled) / with headers
text/plain 70b

Jeff's workaround makes the test happy in Leopard/Intel/5.8.8 as well.

# Sun Jan 11 18:26:52 2009 wyant [...] cpan.org - Correspondence added

Download (untitled) / with headers
text/plain 511b

Just in case you were wondering what would happen under a *ahem* more mature version of Mac OS: Under Mac OS 10.3.9 (Panther) PPC, AppleEvents/t/events.t passes both _with_ and _without_ the suggested modification, under Perls 5.10.0 and 5.8.1 RC3 (which Apple shipped with this OS). Is there enough interest in results under 10.2 (Jaguar) that I should load a machine and try? I can't go farther back than that, and if you want 10.4 (Tiger) say so, because that's what I would wipe to try Jaguar. Tom Wyant

# Sat Jul 25 07:48:33 2009 RUZ [...] cpan.org - Correspondence added

Download (untitled) / with headers
text/plain 88b

Can you please release a new version with this patch applied? -- Best regards, Ruslan.

# Sat Aug 22 08:18:03 2009 casey [...] shobe.info - Correspondence added

From:

casey [...] shobe.info

Download (untitled) / with headers
text/plain 352b

On Sat Jul 25 07:48:33 2009, RUZ wrote: Show quoted text

> Can you please release a new version with this patch applied?

This being broken with default cpan settings results in a cpan that won't even start without manually removing stuff that requires it from the library directory. Why has this not been fixed in such a long time even though the fix is so simple?

# Tue Sep 08 02:04:17 2009 cnandor [...] cpan.org - Status changed from 'open' to 'resolved'

This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.