Patch applied in r22436; ticket is once again resolved.
Reopening this ticket to pose same question as in 44171: Why $/ where a
newline would suffice?
./config//inter/progs.pm:39:print $/ if $verbose;
Patches committed to trunk in r21127.
On Tue Sep 04 05:20:44 2007, [EMAIL PROTECTED] wrote:
>
> Thanks for your feedback. Assuming no one else speaks up for it, I
> will delete it in 1-2 days.
>
This patch deletes the antiquated code -- but also does a bit more
refactoring to eliminate the last patches of code uncovered by test
On Sep 4, 2007, at 8:13 AM, Andy Dougherty via RT wrote:
On Mon, 3 Sep 2007, James Keenan via RT wrote:
I'd say just delete it.
Thanks for your feedback. Assuming no one else speaks up for it, I
will delete it in 1-2 days.
On Mon, 3 Sep 2007, James Keenan via RT wrote:
> Here's an issue I hadn't previously spotted in config/inter/progs.pm --
> one that was revealed in the coverage analysis
> (http://thenceforward.net/parrot/coverage/configure-build/config-inter-progs-pm.html).
>
> In this package's sub runstep(), t
Here's an issue I hadn't previously spotted in config/inter/progs.pm --
one that was revealed in the coverage analysis
(http://thenceforward.net/parrot/coverage/configure-build/config-inter-progs-pm.html).
In this package's sub runstep(), there is the following code:
# Find a working version
Reopened due to bugs; dependency on 45025 added.
Hearing no objection, applied to trunk in r20878.
On Fri Jun 22 18:45:22 2007, [EMAIL PROTECTED] wrote:
> Write unit tests for config/inter/progs.pm,
> the module whose functionality executes Parrot
> configuration step inter::progs.
>
See patch attached, which includes two testfiles plus a small package,
stuffed under t/configure/testlib, whi
# New Ticket Created by James Keenan
# Please include the string: [perl #43341]
# in the subject line of all future correspondence about this issue.
# http://rt.perl.org/rt3/Ticket/Display.html?id=43341 >
Write unit tests for config/inter/progs.pm,
the module whose functionality executes Par
11 matches
Mail list logo