Skip Menu |
 

This queue is for tickets about the Module-Build CPAN distribution.

Report information
The Basics
Id: 56117
Status: open
Priority: 0/
Queue: Module-Build

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

Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)



Subject: Surprisingly limited support for subclassing
Download (untitled) / with headers
text/plain 430b
1.) It would be nice if the MBTest and DistGen modules could be published and better documented as they are very useful in testing subclasses of Module::Build. DistGen in particular has Module::Build hardcoded in a number of places and would need to be generalized. 2.) The ACTION_diff function seems surprisingly hardcoded. It would be better if it went through the various install_types and ran diff_${type}_files on each type.
Download (untitled) / with headers
text/plain 144b
Good points. You're welcome to just copy them into subclass distributions, of course. DistGen is much better documented than it was. -- David
Download (untitled) / with headers
text/plain 741b
On Tue Mar 30 14:44:36 2010, DAGOLDEN wrote: Show quoted text
> Good points. You're welcome to just copy them into subclass > distributions, of course. DistGen is much better documented than it was. > > -- David
Yeah I have copied them. I started off writing my own but realized I ought to use them. The hardcoding in DistGen required some manual editing however. Also I am very puzzled about the where ACTION_diff is written. It seems fragile and totally different from say ACTION_code. Thus subclassing seems very likely to break ACTION_diff. I also have another issue: 3.) If you to try in a subclass to remove the 'arch' build element you get into issues. I got round this by redefining 'install_types' to not return any type without a destination.
Download (untitled) / with headers
text/plain 487b
On Tue Mar 30 15:10:49 2010, SILASMONK wrote: Show quoted text
> Also I am very puzzled about the where ACTION_diff is > written. It seems fragile and totally different from say ACTION_code. > Thus subclassing seems very likely to break ACTION_diff.
Yeah, ACTION_diff probably hasn't seen a lot of action recently. I wrote it ages ago (it was actually one of the "killer apps" in my mind for Module::Build) but I'm not sure many people have used it. It could probably use a fresh coat of paint or two.
Download (untitled) / with headers
text/plain 850b
On Tue Mar 30 15:45:17 2010, KWILLIAMS wrote: Show quoted text
> On Tue Mar 30 15:10:49 2010, SILASMONK wrote:
> > Also I am very puzzled about the where ACTION_diff is > > written. It seems fragile and totally different from say ACTION_code. > > Thus subclassing seems very likely to break ACTION_diff.
> > Yeah, ACTION_diff probably hasn't seen a lot of action recently. I > wrote it ages ago (it was actually one of the "killer apps" in my mind > for Module::Build) but I'm not sure many people have used it. It could > probably use a fresh coat of paint or two.
Related to ACTION_diff I am also looking to write an ACTION_sync that copies from the install destinations back to the source. I am not sure if that is of general applicability, but my subclass is basically supposed to be a deployment system, and sometimes you want to capture whatever is in prod.


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

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.