Have heard no complaints; resolving ticket.
I had some off-list conversations with ptc about a month ago about these
issues. Running the step with --verbose-step indicated that attributes
that the compiler does not recognize do not get added to the 'ccflags'
attribute. Cf. patch applied to trunk:
http://perlcabal.org/svn/parrot/revision/?
Either config/auto/attributes.pm is now DWIMming or I don't understand
it correctly. Please help me determine which.
Consider the first of two files attached (neither of which is yet in
trunk): t/configure/114-auto_attributes-02.t. It's set up like many of
the other t/configure/*.t tests, no no
Patch applied in r22436 for $/ problem. Am still considering other
refinements per chromatic's suggestion.
On 10/22/07, James Keenan via RT <[EMAIL PROTECTED]> wrote:
> On Mon Oct 22 18:16:51 2007, [EMAIL PROTECTED] wrote:
>
> > > i.e., Why we do we need this here when "\n" suffices everywhere else
> > in
> > > the Parrot distribution?
> >
> > I can't think of a reason. While you're at it, how about ge
On 23/10/2007, James Keenan via RT <[EMAIL PROTECTED]> wrote:
> While examining this file for the purpose of writing tests, I noticed
> that it uses Perl's special variable $/ in 6 instances where a newline
> would suffice.
>
> [parrot] 504 $ grep -n '$/' config/auto/attributes.pm
> 54:print $/
On Mon Oct 22 18:16:51 2007, [EMAIL PROTECTED] wrote:
> > i.e., Why we do we need this here when "\n" suffices everywhere else
> in
> > the Parrot distribution?
>
> I can't think of a reason. While you're at it, how about getting rid
> of the
> use of 'and' for flow control here by turning them
On Monday 22 October 2007 18:01:27 James Keenan via RT wrote:
> While examining this file for the purpose of writing tests, I noticed
> that it uses Perl's special variable $/ in 6 instances where a newline
> would suffice.
>
> [parrot] 504 $ grep -n '$/' config/auto/attributes.pm
> 54:print $
No reason I can see, other than golf.
On Oct 22, 2007, at 9:01 PM, James Keenan via RT wrote:
While examining this file for the purpose of writing tests, I noticed
that it uses Perl's special variable $/ in 6 instances where a newline
would suffice.
[parrot] 504 $ grep -n '$/' config/auto/attr
While examining this file for the purpose of writing tests, I noticed
that it uses Perl's special variable $/ in 6 instances where a newline
would suffice.
[parrot] 504 $ grep -n '$/' config/auto/attributes.pm
54:print $/ if $verbose;
67:$verbose and print "trying attribute '$attr'$/";
83
File holding framework for writing tests for this configuration step was
committed to trunk Oct 21 2007 in r22362. Tests do not yet provide high
coverage due to obstacles in testing runstep(). You are encouraged to
poke around and add tests.
11 matches
Mail list logo
