Skip Menu |

This queue is for tickets about the namespace-clean CPAN distribution.

Report information
The Basics
Id: 90372
Status: resolved
Priority: 0/
Queue: namespace-clean
People
Owner: Nobody in particular
Requestors: yoshida [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.24
Fixed in: 0.25

History Show all quoted text
  Thu Nov 14 21:18:24 2013 yoshida [...] cpan.org - Ticket created
Subject: fatal on configure if no EU::CBuilder
I think the usage of $@ is wrong. What do you think? Show quoted text
>Can't locate object method "new" via package "ExtUtils::CBuilder" at Makefile.PL line 109.
I have attached the patch. Regards.
Subject: namespace-clean-0.24-Makefile.PL.patch
--- Makefile.PL.orig 2012-11-27 03:47:12.000000000 +0900 +++ Makefile.PL 2013-11-15 11:12:20.522694817 +0900 @@ -99,7 +99,7 @@ # Do we have the configure_requires checker? local $@; eval "require ExtUtils::CBuilder;"; - if (! $@ ) { + if ( $@ ) { # They don't obey configure_requires, so it is # someone old and delicate. Try to avoid hurting # them by falling back to an older simpler test.
  Thu Jan 16 23:10:47 2014 haarg [...] haarg.org - Correspondence added
On Thu Nov 14 21:18:24 2013, YOSHIDA wrote: Show quoted text
> I think the usage of $@ is wrong. What do you think? >
> > Can't locate object method "new" via package "ExtUtils::CBuilder" at > > Makefile.PL line 109.
> > I have attached the patch. > Regards.
Fixed in the repository.
  Thu Jan 16 23:10:47 2014 The RT System itself - Status changed from 'new' to 'open'
  Tue Mar 25 00:40:06 2014 yoshida [...] cpan.org - Status changed from 'open' to 'resolved'
  Tue Mar 25 00:40:06 2014 yoshida [...] cpan.org - Fixed in 0.25 added