# from John J. Trammell via RT
# on Monday 01 October 2007 13:24:
Show quoted text> 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.
Show quoted text>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
