# from John J. Trammell via RT # on Monday 01 October 2007 13:24: > Queue: Module-Build > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=29225 > > >On Mon Oct 01 02:17:51 2007, EWILHELM wrote: >> So, options for the fakeinstall target in ~/.modulebuildrc should be >> an error? The options are per-target. We can certainly >> special-case some of them, but that tends to lead to subtle bugs. > >According to the documentation: > > fakeinstall > [version 0.02] > This is just like the "install" action, but it won't actually do > anything, it will just report what it would have done if you had > actually run the "install" action.
Yes, but .modulebuildrc is documented to set switches "per action". Thus, the documentation is correct, albeit somewhat ambiguous (and the design or .modulebuildrc is somewhat non-optimal due to the currently simplistic "per action" treatment (which does what is it told -- not necessarily what is intended.) If you didn't set the switches on the "fakeinstall" target (or "*"), it won't look for them. >A few non-exclusive options make sense to me: > (1) warn if, for some reason, fakeinstall and install would differ > (2) don't permit fakeinstall options in .modulebuildrc > (3) update the documentation to clarify the behavior #2 would be a fine feature if you would like to send tests and a patch. At the moment, I'm just trying to tame the bug queue. Ultimately, it would be nice if the handling of .modulebuildrc never allowed strange things to happen, but that requires special treatment for certain keys (which closes some cases while opening others.) --Eric
