Re: [perl #44451] [TODO] Write a SKIP block to test OS-specific branch in init::defaults

2007-08-06 Thread Joshua Isom
On Aug 6, 2007, at 5:41 PM, James Keenan via RT wrote: On Mon Aug 06 05:57:39 2007, ptc wrote: This is the block in question in config/init/defaults.pm: my $archname = $Config{archname}; if ($m) { if ( $archname =~ /x86_64/ && $m eq '32' ) { $archname =~ s/x86_64/i

Re: [perl #44379] config/auto/attributes.pm ought to use its own test_c.in

2007-08-06 Thread chromatic
On Monday 06 August 2007 22:25:06 Mark Glines wrote: > Indeed, with the attached patch I have working attributes again. Works for me too. Applied as r20531. -- c

Re: [perl #44379] config/auto/attributes.pm ought to use its own test_c.in

2007-08-06 Thread Mark Glines
On Mon, 6 Aug 2007 15:20:39 -0700 chromatic <[EMAIL PROTECTED]> wrote: > That's not exactly the same as a stringified hash. I think we can > just remove the cc_run() bits and be fine (at least, it worked for > me). Indeed, with the attached patch I have working attributes again. Mark Index: conf

Re: [perl #44379] config/auto/attributes.pm ought to use its own test_c.in

2007-08-06 Thread Mark Glines
On Mon, 6 Aug 2007 15:20:39 -0700 chromatic <[EMAIL PROTECTED]> wrote: > On Monday 06 August 2007 15:10:30 Andy Lester wrote: > > > On Mon, Aug 06, 2007 at 02:40:06PM -0700, chromatic > > ([EMAIL PROTECTED]) > wrote: > > > > I find these two lines confusing: > > > > > > my %eval = eval cc_r

Re: More on the HLL/Tcl Fix

2007-08-06 Thread jerry gay
On 8/6/07, chromatic <[EMAIL PROTECTED]> wrote: > On Monday 06 August 2007 20:43:23 Mark Glines wrote: > > > Shouldn't that be a typedef? In other words: > > > > typedef enum {...} HLL_enum_t; > > > > > > I think that'll fix Coke's build failure. > > Probably. I just hoisted it out of src/hll.c i

Re: More on the HLL/Tcl Fix

2007-08-06 Thread chromatic
On Monday 06 August 2007 20:43:23 Mark Glines wrote: > Shouldn't that be a typedef? In other words: > > typedef enum {...} HLL_enum_t; > > > I think that'll fix Coke's build failure. Probably. I just hoisted it out of src/hll.c into include/parrot/hll.h verbatim. -- c

Re: More on the HLL/Tcl Fix

2007-08-06 Thread Mark Glines
On Sun, 5 Aug 2007 23:44:16 -0700 chromatic <[EMAIL PROTECTED]> wrote: > +enum { > +e_HLL_name, > +e_HLL_lib, > +e_HLL_typemap, > +e_HLL_MAX > +} HLL_enum_t; > + Shouldn't that be a typedef? In other words: typedef enum {...} HLL_enum_t; I think that'll fix Coke's build failur

Re: More on the HLL/Tcl Fix

2007-08-06 Thread Will Coleda
I tried this patch on OSX/Intel, build failed with a ton of errors, similar to and ending with: compilers/imcc/parser_util.o private external definition of _HLL_enum_t in section (__DATA,__common) compilers/imcc/pcc.o private external definition of _HLL_enum_t in section (__DATA,__common)

[perl #42872] [PATCH] Parrot::BuildUtils: refactoring and additional tests

2007-08-06 Thread James Keenan via RT
While I thought I had written tests to cover all the code in Parrot::BuildUtil, recent coverage analysis showed this was not the case. I have written additional tests which were committed to trunk in r20527. kid51

[perl #44431] [TODO] Cleanup entries in _get_manifest_entry()

2007-08-06 Thread James Keenan via RT
This was just a left-over comment. Deleted in r20523. Closing ticket.

[perl #44453] [TODO] Peer into $ret, which is actually $conf as augmented by the operation of init::defaults::runstep()

2007-08-06 Thread James Keenan via RT
We don't need to do this. Resolved in r20522.

[perl #44451] [TODO] Write a SKIP block to test OS-specific branch in init::defaults

2007-08-06 Thread James Keenan via RT
On Mon Aug 06 15:41:43 2007, jkeen at verizon.net wrote: > > This is the block in question in config/init/defaults.pm: > > my $archname = $Config{archname}; > if ($m) { > if ( $archname =~ /x86_64/ && $m eq '32' ) { > $archname =~ s/x86_64/i386/; > > # a

[perl #44451] [TODO] Write a SKIP block to test OS-specific branch in init::defaults

2007-08-06 Thread James Keenan via RT
On Mon Aug 06 05:57:39 2007, ptc wrote: > In the file t/configure/102-init_defaults.01.t there is the todo item: > > # TODO: Write a SKIP block which will test the one OS-specific branch in > # init::defaults. This is the block in question in config/init/defaults.pm: my $archname = $Config

Re: [perl #44379] config/auto/attributes.pm ought to use its own test_c.in

2007-08-06 Thread chromatic
On Monday 06 August 2007 15:10:30 Andy Lester wrote: > On Mon, Aug 06, 2007 at 02:40:06PM -0700, chromatic ([EMAIL PROTECTED]) wrote: > > I find these two lines confusing: > > > > my %eval = eval cc_run(); > > return if !%eval; > Have you looked at the string it returns? It looks like

Re: [perl #44379] config/auto/attributes.pm ought to use its own test_c.in

2007-08-06 Thread Andy Lester
On Mon, Aug 06, 2007 at 02:40:06PM -0700, chromatic ([EMAIL PROTECTED]) wrote: > On Monday 06 August 2007 14:06:53 jerry gay wrote: > > > i think i've fixed it up as of r20521. let me know if it still behaves > > unexpectedly. > > I find these two lines confusing: > > my %eval = eval cc_run(

Re: [perl #44379] config/auto/attributes.pm ought to use its own test_c.in

2007-08-06 Thread chromatic
On Monday 06 August 2007 14:06:53 jerry gay wrote: > i think i've fixed it up as of r20521. let me know if it still behaves > unexpectedly. I find these two lines confusing: my %eval = eval cc_run(); return if !%eval; cc_run returns a string; why evaluate it? Further, why assign the re

Re: [perl #44379] config/auto/attributes.pm ought to use its own test_c.in

2007-08-06 Thread jerry gay
On 8/3/07, jerry gay <[EMAIL PROTECTED]> wrote: > On 8/3/07, via RT Andy Dougherty <[EMAIL PROTECTED]> wrote: > > # New Ticket Created by Andy Dougherty > > # Please include the string: [perl #44379] > > # in the subject line of all future correspondence about this issue. > > # http://rt.perl.org

[svn:perl6-synopsis] r14435 - doc/trunk/design/syn

2007-08-06 Thread larry
Author: larry Date: Mon Aug 6 11:32:21 2007 New Revision: 14435 Modified: doc/trunk/design/syn/S11.pod Log: avoid P5ish language ossification imposed by (lack of) library policy Modified: doc/trunk/design/syn/S11.pod ==

Implementing junctions

2007-08-06 Thread Miroslav Silovic
I've been looking at various Perl6 operators in terms of what their implementation would look at the Parrot level. Junctions struck me as possibly highly problematic. The issue is, how would one go about compiling them at the point when they get passed to a function call as 1 of the parameters.

Re: [perl #44391] r19820 breaks other JITs

2007-08-06 Thread Paul Cochrane
On 05/08/07, Joshua Isom <[EMAIL PROTECTED]> wrote: > > On Aug 4, 2007, at 5:28 AM, Paul Cochrane wrote: > > > On 03/08/07, via RT Joshua Isom <[EMAIL PROTECTED]> > > wrote: > >> # New Ticket Created by Joshua Isom > >> # Please include the string: [perl #44391] > >> # in the subject line of all

Parrot Bug Summary

2007-08-06 Thread Parrot Bug Summary
Parrot Bug Summary http://rt.perl.org/rt3/NoAuth/parrot/Overview.html Generated at Mon Aug 6 13:00:04 2007 GMT --- * Numbers * New Issues * Overview of Open Issues * Ticket Status By Version * Requestors with mo

[perl #44447] [TODO] Remove deprecated part of PGE API

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #7] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=7 > In the file t/compilers/pge/pge.t there is the todo item (with some context):

[perl #44457] [TODO] make sure files match test files for DYNPMCs and DYNOPs etc

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #44457] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=44457 > In the file t/distro/test_file_coverage.t there is the todo item: # TODO: DYNPMC, DYNO

[perl #44445] [TODO] Add more p6 regexp tests for pge

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #5] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=5 > In the file t\compilers\pge\p6regex\context.t there is the todo item: ## TODO: more T

[perl #44455] [TODO] add verbose tests with some Test::Warn like mechanism to configure step tests

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #44455] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=44455 > In the file t/configure/step.t there is the todo item: # XXX add verbose tests with so

[perl #44453] [TODO] Peer into $ret, which is actually $conf as augmented by the operation of init::defaults::runstep()

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #44453] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=44453 > In the file t/configure/102-init_defaults.01.t there is the todo item: # TODO: Peer i

[perl #44451] [TODO] Write a SKIP block to test OS-specific branch in init::defaults

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #44451] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=44451 > In the file t/configure/102-init_defaults.01.t there is the todo item: # TODO: Write

[perl #44449] [TODO] Test pod existence for subs in pge

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #9] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=9 > In t/compilers/pge/pge.t there is the todo item: ## TODO: test pod exists

[perl #44437] [TODO] Use src_dir instead of build_dir when reading MANIFEST

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #44437] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=44437 > In the file t/codingstd/linelength.t there is the todo item: # XXX this should really

[perl #44435] [TODO] support getting implementations from central superclass

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #44435] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=44435 > In the file \lib\Parrot\Pmc2c\PMC\RO.pm there is the todo item: # FIXME support gett

[perl #44443] [TODO] Increase code coverage of json

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #3] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=3 > In the file t/compilers/json/to_parrot.t there is the todo item: # XXX Need many more

[perl #44441] [TODO] Change file search to return all files in distribution?

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #1] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=1 > In the file t/codingstd/perlcritic.t there is the todo item: # XXX Change this to

[perl #44439] [TODO] Skip files copied wholesale into repository when running perlcritic.t

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #44439] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=44439 > In the file t/codingstd/perlcritic.t there is the todo item: # XXX We should skip

[perl #44433] [TODO] Explicitly clearing to be RO?

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #44433] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=44433 > In the file \lib\Parrot\Pmc2c\PMC\RO.pm there is the todo item: # autogenerate for

[perl #44431] [TODO] Cleanup entries in _get_manifest_entry()

2007-08-06 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #44431] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=44431 > In the file lib/Parrot/Manifest.pm there is the todo item within the sub _get_manifest_