Bug #71745 for namespace-sweep: Doesn't work with Class::Std

Mon Oct 17 13:27:42 2011 len.budney [...] gmail.com - Ticket created

Subject:

Doesn't work with Class::Std

If I use namespace::sweep in a package based on Class::Std, the :ATTR() trait breaks. I tested this by using namespace::sweep in CLI::Startup and then doing a "make test." The problem manifests itself immediately.

Mon Oct 17 14:14:50 2011 FRIEDO [...] cpan.org - Taken

Wed Aug 29 07:01:29 2012 chip [...] pobox.com - Correspondence added

From:

chip [...] pobox.com

On Mon Oct 17 13:27:42 2011, BUDNEY wrote: Show quoted text

> If I use namespace::sweep in a package based on Class::Std, the :ATTR() > trait breaks. > > I tested this by using namespace::sweep in CLI::Startup and then doing a > "make test." The problem manifests itself immediately.

What's the right thing to do in that case? Leave &ATTR alone?

Wed Aug 29 07:01:31 2012 The RT System itself - Status changed from 'new' to 'open'

Mon Oct 01 12:39:11 2012 len.budney [...] gmail.com - Correspondence added

On Wed Aug 29 07:01:29 2012, CHIPS wrote: Show quoted text

> > What's the right thing to do in that case? Leave &ATTR alone?

I'm not sure what's going wrong. The error message is, Error: Invalid HASH attribute: ATTR() at ./CLI-Startup/blib/lib/CLI/Startup.pm line 238 I'm thinking that namespace::sweep is sweeping the sub MODIFY_HASH_ATTRIBUTES(), which is causing attributes not to be processed correctly. It gets shoved into my class's namespace by Class::Std::import() as a monkey patch. When I add MODIFY_HASH_ATTRIBUTES and MODIFY_CODE_ATTRIBUTES to %keep in a local copy of sweep.pm, my tests all pass. That's obviously not the right fix, but it suggests what the problem is.