logo       
Google Custom Search
    AddThis Social Bookmark Button
-->

Re: [Module::Build] Re: [Cpanplus-devel] CP+ and M::B cooperation: msg#00257

Subject: Re: [Module::Build] Re: [Cpanplus-devel] CP+ and M::B cooperation
On Nov 29, 2004, at 11:11, Jos I. Boumans wrote:
If you limit that to objects, then it should be possible to use cleverly use Data::Dumper here ...
Perhaps, but we're using coderefs right now, so that wouldn't work i'm afraid.

Fair enough - just looking for some middle ground. But you mentioned you've since worked around this, so maybe it's not worth the effort to change CPAN++ and M::B to agree here..
yes we have worked around here, but as stated below, if the guts of M::B change in a way that effects our workaround, things will break again. so we have to agree on *something* here to make sure things keep working.

At risk of sounding silly, why not agree on using objects in @INC? AFAIK, you'd still be able to accomplish the same things you would with coderefs. Ok, so it's a bit more work to write the classes than a simple sub (albeit, not much). And the existing coderefs CPAN++ uses in @INC would have to be refactored, as would M::B's code above. But at least that would solve one compatibility problem.

Would that make sense / be a lot of work from the CPAN++ side?
but what's the point?

the current behaviour of M::B is wrong in either case -- the stringification of refs.

why allow one and disallow the other? what's the benefit to either party? what's the difference between allowing objects and disallowing coderefs?

Hmm, I thought the point was to allow CPAN+ to use complex behaviour in @INC, and for M::B to preserve that when it serializes @INC. Allowing objects and not coderefs was a 'halfway house' suggestion -- it's easy to serialize objects, and hard to serialize coderefs -- that would let us do that.

But if that's not a requirement, then yeah, not much point in doing the extra work. M::B should just ignore refs. The last bit of code I sent does that. Now we just need some tests & some docs to back that up.

-Steve



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now. http://productguide.itmanagersjournal.com/


<Prev in Thread] Current Thread [Next in Thread>