% iblechbot has joined #parrot r26853 | fperrad++ | trunk: : [docs] : since r26756, list_unjitted.pl moved in tools/dev diff: http://www.parrotvm.org/svn/parrot/revision?rev=26853 % IllvilJa has left IllvilJa!~jilves@emea-netcache1.oracle.co.uk % IllvilJa has joined #parrot % Senaka has joined #parrot purl: seen infinoid infinoid was last seen on #parrot 4 hours and 44 minutes ago, saying: which one was failing? hi all... when is the #parrot-sketch meeting? in GMT? % cosimo_ has joined #parrot % cosimo_ has left cosimo_!~cosimo@pat-tdc.opera.com % cosimo has joined #parrot ok got it purl: parrotsketch i think parrotsketch is 18:30 UTC, Tuesdays or a weekly status meeting for parrot design team % ruoso has joined #parrot % Debolaz_ has joined #parrot % particle has left particle!~particle@c-24-19-3-148.hsd1.wa.comcast.net % Maddingu1 has joined #parrot % skids has left skids!~bri@c-71-233-204-100.hsd1.ma.comcast.net % Khisanth has left Khisanth!~Khisanth@pool-68-237-111-126.ny325.east.verizon.net % Maddingue has left Maddingue!~Maddingue@profane.mongueurs.net % Debolaz has left Debolaz!~root@nat.andersberle.com % skids has joined #parrot % Maddingu1 is now known as Maddingue % particle has joined #parrot % ruz has left ruz!~cubic@85.112.113.222 % ruz has joined #parrot % Khisanth has joined #parrot % AndyA has joined #parrot % tetragon has joined #parrot % skids has left skids!~bri@c-71-233-204-100.hsd1.ma.comcast.net % skids has joined #parrot % Senaka has left Senaka!~senaka@124.43.208.245 % ambs has joined #parrot how do the lib/Parrot/Test/ files relate to the test harness? % AndyA_ has joined #parrot % AndyA has left AndyA!~andy@mail.detnorsketeatret.no % iblechbot has left iblechbot!~iblechbot@ppp-62-216-197-227.dynamic.mnet-online.de % tetragon has left tetragon!~seneca@CPE0040d001f62f-CM000a736592a8.cpe.net.cable.rogers.com % ambs has left ambs!~ambs@255.92.54.77.rev.vodafone.pt % AndyA has joined #parrot % AndyA_ has left AndyA_!~andy@mail.detnorsketeatret.no % paco has joined #parrot % AndyA_ has joined #parrot % AndyA has left AndyA!~andy@mail.detnorsketeatret.no % Daveman has joined #parrot % Ademan_ has joined #parrot % iblechbot has joined #parrot % Dave has left Dave!~dave@pool-141-150-77-248.mad.east.verizon.net % Ademan has left Ademan!~dan@h-68-164-168-66.snfccasy.dynamic.covad.net % skids has left skids!~bri@c-71-233-204-100.hsd1.ma.comcast.net % ambs has joined #parrot seen barney? barney was last seen on #parrot 15 hours and 22 minutes ago, saying: calling it a day, looking forward to rotty's patches seen merlyn? merlyn was last seen on #moose 18 days and 14 hours ago, saying: ... http://methodsandmessages.vox.com/library/post/the-year-of-smalltalk.html [Mar 20 15:09:20 2008] :-S % iblechbot has left iblechbot!~iblechbot@241.18-dial.augustakom.net % Daveman has left Daveman!~dave@pool-141-153-233-27.mad.east.verizon.net % AndyA_ has left AndyA_!~andy@mail.detnorsketeatret.no % Daveman has joined #parrot % kj has joined #parrot r26854 | fperrad++ | trunk: : [Lua] : - fix table.sort() diff: http://www.parrotvm.org/svn/parrot/revision?rev=26854 % wknight8111 has joined #parrot % skids has joined #parrot anybody else try to build this morning on r26854? I was just about to do so :-) takes about 2 min * ambs svnups seems to work fine here at least, I was able to build okay making builds ok I just smoked r26843 on several platforms oops r26853 this is not smoking yet, but with the heat it should smoke soon O:-) I ran it just now on the latest, and make is complaining about a cyclic dependency wknight8111: what platform? have you recently done a 'make realclean'? i did nmake realclean, configure, nmake. #52596 did it twice, actually. same result both times nopaste? i think nopaste is at http://nopaste.snit.ch/ (ask TonyC for new channels) or http://rafb.net/paste or http://paste.husk.org/ or http://poundperl.pastebin.com/ or http://paste.scsys.co.uk/ or don't bother me while I'm eating "ambs" at 77.54.92.255 pasted "Test Summary Report - Mac OS 10.5.2" (10 lines) at http://nopaste.snit.ch/12647 1 test failing on linux make smoke ok "wknight8111" at 71.230.33.251 pasted "Build error on r26854" (56 lines) at http://nopaste.snit.ch/12648 wknight8111: bah. that's my fault: gmake apparently says 'oh, that's a circular dep, I'll drop it.' hang on. i guess hang on. is this from the command prompt, or from within the ide? okay, thanks why does the test harness execute this: [perl -le '... some code...'] ? % Ademan_ has left Ademan_!~dan@h-68-164-168-66.snfccasy.dynamic.covad.net % Theory has joined #parrot % AndyA has joined #parrot wknight8111: is it possible to get nmake to *tell* us what the circular dep is? % rdice has joined #parrot % iblechbot has joined #parrot coke: http://nopaste.snit.ch/12648 says it's src\pmc\pmc_fixedintegeraray.h ... that tells us one node in the cycle. =-) ah wknight8111: can you try removing the dep of src/pmc/pmc_fixedintegerarray.h from src/pmc/pmc_fixedintegerarray.dump and rebuild? ah, I can at least get the error message with 'make src/pmc/fixedintegerarray.o' me too Coke: in languages-smoke on linux, it was hanging up on tcl/t/cmd_expr.t for a very long time % Ademan_ has joined #parrot diakopter: yes, it's slow. I dont think nmake can be more verbose, I'll check the options no, it doesnt have a more verbose error reporting mode wknight8111: fixed, I think. sadly, I had a patch very similar to that which i threw out yesterday because it didn't seem to be needed on my box. Thanks for the report. r26855 | coke++ | trunk: : [build] : Avoid circular dependency for fixedintegerarray: while it's used for METHODs, : we can't have it require it to be built before it's built. diff: http://www.parrotvm.org/svn/parrot/revision?rev=26855 let me try updating to that and see if it fixes my problem seems to build ok * ambs makes real clean and tries from scratch you'd need a reconfig at least. % AndyA_ has joined #parrot % AndyA has left AndyA!~andy@mail.detnorsketeatret.no ccache++ ... make -j ++ ++ ++ Coke: does parrot compile with -j already? yes. if it doesn't, tell me. nice I can compile with -j can we config with -j ;) nice particle: we could if we switched our config to a makefile. config dep management. P::FM could be used make -f Configure.mk -j make -j but, really, || test is where we need to go make test -j "particle" at 24.19.3.148 pasted "infinoid: that stack trace i promised you" (37 lines) at http://nopaste.snit.ch/12650 * particle has a mexican jumping interp libparrot.dll!runops_slow_core(parrot_interp_t * interp=0x0000000000000000, ... whee! scary stack smash? this is life with parrot on x86_64-msvc that almost looks like a buggy stack dumper. thanks for the paste if i step through a program, i can watch the interp move around it's not the dumper that's buggy % AndyA_ has left AndyA_!~andy@mail.detnorsketeatret.no % AndyA has joined #parrot % ambs has left ambs!~ambs@255.92.54.77.rev.vodafone.pt % uniejo has left uniejo!~uniejo@langebro.adapt.dk yeah, build worked this time! Thanks Coke++ % AndyA has left AndyA!~andy@mail.detnorsketeatret.no % AndyA has joined #parrot % kj has left kj!~IceChat7@ip565fd420.direct-adsl.nl % contingencyplan has left contingencyplan!~contingen@cpe-76-186-27-146.tx.res.rr.com % contingencyplan has joined #parrot % Debolaz_ is now known as Debolaz % ask_ has left ask_!~ask@pat-tdc.opera.com % ask_ has joined #parrot % cognominal has left cognominal!~cognomina@82.67.232.89 % peeps[work] has joined #parrot % cognominal has joined #parrot % contingencyplan has left contingencyplan!~contingen@cpe-76-186-27-146.tx.res.rr.com % grim_fandango has joined #parrot % contingencyplan has joined #parrot % TonyC has left TonyC!~tony@202-154-105-237.people.net.au % nopaste has left nopaste!~opaste@202-154-105-237.people.net.au % grim_fandango has left grim_fandango!~matt@bas2-kingston08-1167929677.dsl.bell.ca % Theory has left Theory!~Theory@c-24-21-175-208.hsd1.mn.comcast.net % Theory has joined #parrot wknight8111: heh. break the build, fix teh build, get karma. =-) karma rotty? rotty has neutral karma rotty++ ...and it begins. karma rotty? rotty has karma of 1 :) thanks karma particle particle has karma of 1277 % nopaste has joined #parrot * particle waits for 1337 karma coke coke has karma of 1837 karma must be age-based :P must be age-based :p has neutral karma purl is so stupid. purl, purl? i heard i was going alone. or almost an anagram of Donaudampfschiffahrtskapitaensmuetzenkordel or a perv or an auto-triage bot or a she or so stupid or a smartass or the sixth beatle. or http://www.infobot.org or dumb or a butt sniffer % sjansen has joined #parrot purl is definitely a butt sniffer purl, purl? i am going alone. or almost an anagram of Donaudampfschiffahrtskapitaensmuetzenkordel or a perv or an auto-triage bot or a she or so stupid or a smartass or the sixth beatle. or http://www.infobot.org or dumb or a butt sniffer it's the small things in life % contingencyplan has left contingencyplan!~contingen@cpe-76-186-27-146.tx.res.rr.com % contingencyplan has joined #parrot % kj has joined #parrot % kj has left #parrot Tests=11110 yay! parrot has thirty tests now. hehe how often is the documentation on parrotcode.org updated to reflect the current file documentation? every hour or so iirc or maybe it's daily. coke would know oh, i didn't realize it was so frequent. I was under the impression that some of the files were a little out-of-date % particl1 has joined #parrot % particle has left particle!~particle@c-24-19-3-148.hsd1.mn.comcast.net % particl1 is now known as particle opbots, names % Daveman is now known as Dave % ambs has joined #parrot particle: I can't make parrotsketch, will you be there? If so, I can paste you my report in /msg paste it to #parrotsketch it's logged, after all :) oh, good idea ;-) well, mention I left it there :) sure don't want to look like I'm slacking ;-) ok, done gotta go - dinner with $DAYJOB folks ~~ * Infinoid gets back from the dentist with half his face numb ...as long as it's not the top half... I always noticed that major portions of my wallet were missing... :) docs on the website are cached for 24 hours usually. * particle welcomes our viewers who have just returned from scrollbackland * Coke fires up his UTC clock. 17:33 here hai. 18:33 here ambs, your UTC clock is an hour off :-) PS in about an hour, or your parrot is free. it's 18:33 in london. =-) Coke: and portugal portugal is .pt * Coke is guessing they just went to DST or something. yes 18:33 in london, yes. but it's still 17:33 (now 17:34) UTC for one week or two. Can't remember % ruoso has left ruoso!~ruoso@195.23.92.2 ambs;OH. you meant here==where you are, not here==utc from your standpoint. =-) pmichaud: true utc is the same everywhere english is hard. wknight8111: don't respond to the autoreply, respond to the ticket. =-) man, I'm bad at this whole system I need to be banned from all this nbd. just that no one else has your original autoreply, so it shows as two threads now. =-) why does the test harness evaluate "perl -le "print join qq[\n], @INC" ? are you asking what it does? what it does and why it's there it's grabbing perl's module search-path if it's screwing with your non-p5 harness_perl you might be interested in the hack I used in kp6 to get around that Infinoid: it does that even when the "exec" parameter to the harness is set to something different from perl, which will not work avar: kp6? somebody said kp6 was mean as an infrastrucure to implement STD rotty: where? having trouble grepping for that kindaperl6 avar: where is the kp6 code? it's neither under languages nor compilers... kp6 is in the pugs repo Infinoid: don't know, I also could not find the causing place... hmm. who (if anyone) is working on parrot's test harness at the moment? I basically did exit 0 if "@ARGV" =~ /perl -le "print ... / in the kp6 executable, there may be a better way around it rotty: oh, it looks like test harness boilerplate. its in Test::Harness::Straps (that's /usr/lib/perl5/5.10.0/Test/Harness/Straps.pm on my system) kp6 is also "kinda perl 6" okay, Coke. can someone fix the Parrot harness to not invoke that boilerplate if exec is not perl? * rotty is a Perl noob how are you running the parrot harness if your exec is not perl? its a side effect of using the normal perl Test::Harness, which was written with perl in mind. I'm not sure what's the best way to tackle that * avar suggests his way of doing it:) or write a shellscript wrapper around your program Coke: use Parrot::Test::Harness language => 'eclectus', exec => [ 'petite', '--script' ], ah. Infinoid: can't one somehow override that behaviour in Parrot::Test::Harness ? rotty: considering your command line is straight from the Parrot::Test::Harness SYNOPSIS, it would probably be a good idea to make it work *somehow* :) rotty: what problem are you looking at/trying to address? (high level) pmichaud: running tests with an interpreter that is not perl rotty: languages/perl6 does it as does languages/abc pmichaud: these use `compiler => ...', not `exec => ...' so, you need it to run tests with an interpreter that is neither perl nor parrot pmichaud: exactly and exec => doesn't work? pmichaud: it sorta does, but there is an spurious invocation of the interpreter with args only suited for Perl rotty: okay. I think I remember seeing a problem like that somewhere before. I'm of the opinion that we should re-do parrot's test harness from scratch starting... now. I was going to ask questions about it during #ps today should we stick to Test::Harness, or move to TAP::Harness? if you're going to design a new one from scratch, please *please* design it with parallel execution in mind we still need Test::TAP::HTMLMatrix? What dependencies do we have? ... isn't ewilhelm already on this? ewilhelm or AndyA can tell us a bit about how TAP::Harness fairs on the platforms we care about we only need tth for our existing smoke setup, which we've already discussed dumping. or whether they think it's ready to support our needs. i'm fairly certain the answer is "yes" Coke: perhaps, but my impression is that ewilhelm is more interested in helping someone else build the parrot harness as opposed to building it himself the folks in oslo were working on the yaml diagnostics for tap i don't know how long until we have a framework built around it that we can use for smoke well, my reason for wanting to support tth is so that we can get a public "here's what passes" display for rakudo % kj has joined #parrot can we do that on the server side? one less dependency that way :) Infinoid: can we do... what? ... on the server side? the public "here's what passes" display so, the local make smoke passes the test results to a server which then formats it into html? in other words, we just need a tap => html tool or something? maybe I'm confused. how does yaml tie into this? I don't understand that part either :-) if you guys want to support yaml, you can. it can be converted to html later if needed % kj has left kj!~IceChat7@ip565fd420.direct-adsl.nl Infinoid: the tap infrastructure is destined to support yaml so the output would be yaml already then it's a matter of writing something that merges the yaml streams and converts to html does it need to be yaml? why not just parse the tap output? "why not just use something that already exists". =-) coke: what do we have that exists and works? smolder? smolder is http://sourceforge.net/projects/smolder or web-based smoke test aggregator used by developers and testers to upload (automated or manually) and view smoke/regression tests using the Test Anything Protocol (TAP). % AndyA_ has joined #parrot % AndyA has left AndyA!~andy@mail.detnorsketeatret.no smolder looks promising % chromatic has joined #parrot % AndyA_ has left AndyA_!~andy@mail.detnorsketeatret.no smolder++ % iblechbot has left iblechbot!~iblechbot@ppp-62-216-205-194.dynamic.mnet-online.de seen merlyn merlyn was last seen on #moose 18 days and 20 hours ago, saying: ... http://methodsandmessages.vox.com/library/post/the-year-of-smalltalk.html [Mar 20 15:09:20 2008] #ps time merlyn doesn't hang out gere here Infinoid++ particle: I know, but purl stands on #perl O:-) rotty: what did I do? you told me about Test::Harness::Straps :) % Senaka has joined #parrot % Senaka has left Senaka!~senaka@124.43.227.54 % Senaka has joined #parrot % Senaka has left #parrot % rdice_ has joined #parrot % rdice has left rdice!~richard_d@CPE0014bfafbbd5-CM0011e6ecf48a.cpe.net.cable.rogers.com r26856 | fperrad++ | trunk: : [Lua] : - fix random library : - add tests diff: http://www.parrotvm.org/svn/parrot/revision?rev=26856 "particle" at 24.19.3.148 pasted "sizes" (10 lines) at http://nopaste.snit.ch/12652 smolder or my TapTinder .... http://mj41.cz/wiki/TapTinder :-) % ambs has left ambs!~ambs@255.92.54.77.rev.vodafone.pt hrmm... taptinder. "chromatic" at 63.105.17.30 pasted "Does This Fix 64-bit Pointer Size Problems in UnManagedStruct?" (282 lines) at http://nopaste.snit.ch/12653 chromatic++ # managing PS. The src/mmd.c assertion checks for four-byte pointer alignment, I think. I'm trying to decide if it has endian problems. I don't think so. I don't know a portable way to check if a pointer is properly aligned though. I suspect the compiler won't let you make one that isn't. (or it's a bad compiler) chromatic: you have a few tickets assigned to you that may be leftovers. Can you ditch anything that's assigned to you that you don't want? ditch or fix? well, fix would be best, but if they're ones you're not going to get to them soon, ditching them is probably better. Figuring out how to pack() Parrot's types...Configure.pl: Unable to find an integer type that fits a pointer. particle: how is your perl built? * particle tees his configure and make output for 64bit Infinoid: it's activestate's 64bit build i can get details. do you want -V ? no thanks, pack() should have something, we're just detecting it wrong "particle" at 24.19.3.148 pasted "perl -V" (93 lines) at http://nopaste.snit.ch/12654 well, there it is anyway twist my arm a little harder. :) nopaste.pl++ % Theory has left Theory!~Theory@c-24-21-175-208.hsd1.or.comcast.net % Theory has joined #parrot wait, what? wait, is -L "follow link" or "gimme information about the link proper"... intsize=4, longsize=4, ptrsize=8, doublesize=8, byteorder=12345678 your sizeof(long) isn't sizeof(void*) "particle" at 24.19.3.148 pasted "configure output" (86 lines) at http://nopaste.snit.ch/12655 that's correct. There's one problem then. it's how the platform's designed. don't blame me gotta love that long long isn't c89 purl, forget wait, Infinoid: I forgot wait, so there's no way to fully support c89 on this platform there's no uint64 or such? http://msdn2.microsoft.com/en-us/library/aa384264(VS.85).aspx seen barney? barney was last seen on #parrot 22 hours and 47 minutes ago, saying: calling it a day, looking forward to rotty's patches % ambs has joined #parrot weird that activestate 64bit is built with use64bitint=define, yet intsize and longsize are both 4 oh, god, no. trying to install parrot 0.4.12? * Coke is replying. btw, someone forgot to update parrothist.pod particle: perl -e 'my $var = pack("q", 1); print(length($var), "\n")' does that print 8? ... or, I'm an idiot. * Coke checks. idiot. check. t/codingstd/idiot.t..... ok (idiot detected) Infinoid: 8 % Psyche^ has joined #parrot % allison has joined #parrot ok, that's the packtype we need, now to get it detected properly t/project/architect.t...ok (allison detected) haha hi particle I was able to get my driver's license and vehicle registration changed, fingers crossed that the license will arrive in the mail before I leave the country next Wednesday one step forward.... % Patterner has left Patterner!~Psyche@e177224219.adsl.alicedsl.de % Psyche^ is now known as Patterner what's the new VM for perl6? Chuck Norris. One punch and your code will run in half the time. :) is that going to be the name of the next release? ;-) s/the next/a future/ * Infinoid suggests "and boy are its arms tired" 0.6.1 codename "Chuck Norris" shouldn't that name be saved for a more mature release? hah 62.99.99 lol Re: pa-risc, I vote for removing the lines 1974-1976 in src/mmd.c those were bad years anyway ;) lololol * ambs wasn't around yet that's why they were so bad (1974-76) I agree. I was quite pleased when the 80s finally rolled around ITYM "Morning in America" Oh hey, here's the problem. include/parrot/parrot.h:142-153 If pointers and intvals are the same size, hooray! Otherwise if pointers and longs are the same size, hooray! egad. Otherwise, pointers and unsigned ints are the same size! They must be! What else might they be? heh I think win64 needs to use hugeintval here I think something needs to use "IT'S NOT AN INT OKAY?" there. why not check to see if INT64 is defined? argggh, I've been reading perl too long. At first I thought all those lines were comments. :-| lololol let's set up a trigger to prevent pmichaud from committing c code sachmet? Coke: not here "Infinoid" at 96.238.213.50 pasted "particle: so... does "long long" work? (possible_win64_fix.diff)" (86 lines) at http://nopaste.snit.ch/12656 particle: I'm looking at the "Figuring out how to pack() Parrot's types...Configure.pl: Unable to find an integer type that fits a pointer." line from your Configure.pl output. confound: I was asking purl. oh, sorry. np. =-) "particle" at 24.19.3.148 pasted "configure output, post patch" (85 lines) at http://nopaste.snit.ch/12657 you know, I'm not sure the "ptrconst" value probed for is ever actually used but I guess that fixed it. thanks for testing, particle i'm rebuilding now lots of conversion and pointer truncation warnings still remain "ack -a" only reports the detection, testsuite, and generated config output. ptrconst isn't ever used, that I can see * Infinoid fixed something worthless! o/ priceless. hehehe if it's never used can we rip it out now? "chromatic" at 63.105.17.30 pasted "Avoid (Some) Pointer Cast Truncations" (15 lines) at http://nopaste.snit.ch/12658 Does this help, particle? Coke: might as well, and see what breaks. ... is there an easy way for me to get a list of every generated file in the repo? MANIFEST.generated ? it has been said that MANIFEST.generated is wrong. ... hurm. svn st, mebbe. barney has commit acess right? right svn st --no-ignore | cut -c 8- (that will also include local modifications.) * rotty anticipates barneys review of his patches * rotty notes that StGit rocks for managing a patch series chromatic: perhaps some, but still src\ops\experimental.ops(51) : warning C4312: 'type cast' : conversion from 'opcode_t' to 'FLOATVAL *' of greater size zillions of opcode_t and size_t warnings * ambs makes test one more time, as he does not know enough about parrot atm. opcode_t is Parrot_Opcode which is a typdef'd long for me. Hm. Maybe I need to fire up the 64-bit box in the other room and cajole it. and long is 4 bytes on win64 "particle" at 24.19.3.148 pasted "partial nmake output after patches" (1400 lines) at http://nopaste.snit.ch/12659 Infinoid: I'm taking that list and running the same dependency checker that I used on the PMC .o files. hopefully this will rule out any non-race-condition based deps. is there a way in nqp to initialize an array at once? like @array := (1, 2, 3);? wknight8111: no that's what I figured. thanks no list context that might actually change a bit in a future pct revision i.e., @array := (1, 2, 3); might work but not much else will -- i.e., hash initialization will still require individual elements I'm trying to write up a simple example of an octal-bin converter, as an intro to PGE. I'm using a lookup table for the conversion I was just hoping for an easy way to do it I tend to use strings for that sort of stuff. :-) that's what i was thinking oh you're right. I could do that, couldn't I? * wknight8111 isn't too bright you'll fit right in. Files=565, Tests=11150, 333 wallclock secs (224.10 cusr + 49.19 csys = 273.29 CPU) :D * Coke finds some missing deps. whee! coke++ % ruoso has joined #parrot * Coke will let this run and fix them later. is there a way to turn PAST off? I'm writing a little program that parses input, but doesnt form a tree or anything could just use --target=parse depends on what you mean by "turn PAST off", I guess. Try to move in after the first date, for another option. well, I dont want to make any PAST nodes, but when I run the program I get the rror "Unable to obtain PAST from program::Grammar" the program is written in nqp? So i've been including an empty "make PAST::Stmts.new()" line yes did you try it with --target=parse? I'm trying to demonstrate PGE, so I have a grammar, and a little NQP that prints out if the input is valid or not --target==parse prints out the whole parse tree, which I dont want either % ambs has left ambs!~ambs@255.92.54.77.rev.vodafone.pt languages\perl6\src\utils\perl6doc\actions.pm % AndyA has joined #parrot % iblechbot has joined #parrot this is such a stupid little program. I'm going to forget about it for a while. thanks wknight8111: the file i pointed you to will show you how to print from PAST that's what perl6doc does now I am interested in consolidating Makefiles. (do we really need a separate Makefile for compilers/*, e.g.) * particle thinks the root Makefile is way to long I find the separate makefiles easier to manage * particle used to work on projects where every directory had a makefile you can have separate *input* makefiles, that's fine. do you mean to consolidate multiple *.in files into a single Makefile? yes. I don't like it -- too much potential for action-at-a-distance unless we have a really well defined set of macros and things that we know exist in the root makefile (and that other makefiles don't touch) .include is your friend does nmake understand .include? it has an include syntax. !INCLUDE ... action at a distance?? if I'm modifying pge.in, the rules I add coule affect other .in files in the master Makefile s/coule/could/ the separate makefiles we have now prevent using -j on the various subdirectories, and prevent having an understanding of all the dependencies. I dislike having included Makefiles - wherever the included makefile is in a different directory, all the relative paths are wrong. what would be the advantage of compiling to NQP instead of PAST? however, I dislike having separate "make" invocations even more, because as Coke says, it breaks -j. Infinoid: we could fix that if we did the includes at compile time. er... at *configure* time. when we're already massaging the heck of the .in files. that works for me I'm not sure that having a single makefile means we understand all of the dependencies, but I'll pass on that personally, -j doesn't seem all that important to me but that doesn't mean it's unimportant to others, I guess rotty: (compiling to NQP versus PAST) -- I'm not sure I understand the question? || test is more important than || make, becausue test takes way longer t/codingstd/linelengths.tap: t/codingstd/linelengths.t $(PARROT) it doesn't get much uglier than that. :) * pmichaud misses the ugliness times 623 is the ugliness, i think he means yeah, what I just proposed is horrid yes, I'd accept that as being ugly :-) .t.tap seems better :-) * Infinoid looks forward to a rewritten harness of course we won't use make to manage || test pmichaud: barney said the next step for eclectus would be to compile to NQP instead of PAST, and I'm not sure that this will be an advantage at least make wouldn't have blatant perly assumptions about the language of a test script nqp instead of pir? or past? nqp instead of PAST rotty: I agree -- but I would guess (and this is just a guess) that he meant to use NQP to create PAST instead of some other transformation pmichaud: no, as the compiler is implemented in Scheme ;) yeah, that's what i think barney meant, too barney likes playing with multiple back-end implementations % Andy has left Andy!~AndyL@host3130.follett.com rotty: then perhaps he means to use nqp instead of scheme rotty: but you know his statement better than I do I'm not sure compiling to nqp would be better... but it might not be bad either pmichaud: I doubt that (switching to NQP), as (AFAIU) the eclectus language is about a compiler implemented in Scheme well, I'll ask barney when he turns up rotty: yes, we'd have to ask barney then looking at the makefile for PGE... the rules for PGE.pbc seem overly complicated. first we remove any existing PGE.pbc ... why? because if a later step in the build fails, we don't want the old copy sitting around where someone thinks it passed if you have properly spec'd dependencies, make should handle all of this for you. no? well, maybe. no. aside from "the build failed." which should be a pretty big clue. =-) except that it's all too easy to overlook I kept overlooking it, which is why I put the rm in there ok. why do you build PGE twice? and RM it twice? it's a staged library (more) the 'name' rule is written as a perl6 rule (from PGE/builtins.pg) so in order to compile it, we have to have a working copy of PGE miniPGE? so we build a copy of PGE with no builtins, then use that copy of PGE to compile the builtin rules, then recompile PGE with the builtin rules % skids has left skids!bri@charon.clarku.edu we can serialize that nicely, if we can call the two versions different filenames ja. You should port PGE to JavaScript, then use it to build itself with no rules, then use it to build itself with rules, and then use it to remove PGE.pbc. I think a Perl 6 compiler magically falls out of there around step 7. except that the program that compiles the builtins (Perl6Grammar.pir) expects to load PGE.pbc so if we have a miniPGE, we also need a miniPerl6Grammar.pir we can tweak it to take a param. and this is all important.... why? Is it worth it to take a param though? pmichaud: because we need someone other than you to be able to maintain the makefile. =-) I can add comments into the makefile that would be most helpful. but creating a separate minipge seems to me to be more maintenance hassle than what exists now pmichaud: can you write a makefile to add the comments for you? *thwap* and what exists now is fairly robust in that it doesn't leave partially-built pge.pbc files around that might get used neither would Infinoid's version. yes, it would at least it would on my system because if building PGE.pbc fails, the makefile doesn't automatically remove $(PARROT_LIBRARY)/PGE.pbc not if you spec'd the dependencies properly and didnt' reuse the same name. when all else fails, output to PGE.pbc.tmp and rename it into place Oh! We should differentiate based on case! PGE.pbe and pge.pbc. Tene: yeah! and you could have one letter per stage... so it looks like its inflating pge.pbc Pge.pbc PGe.pbc Makefiles don't automatically remove their targets if they fail to build Best idea ever. Parrot writes out unfinished PBC files if they fail to build? Or am I misunderstanding? parrot doesn't destroy existing pbc files I think pmichaud is targetting the case where you've already built once. which may have come from a previous build which is fine, unless something has changed, in which you should depend on something * pmichaud points to the dependency in the rule above Okay, I thought it might be that. I have a dependency in place for $(PARROT_LIBRARY)/PGE.pbc but if the build fails, it *fails*. but if one of the sources changes, I'm not guaranteed that PARROT_LIBRARY/PGE.pbc isn't the old version pmichaud: if the make succeeds, you should be. (if the make fails) then the make *fails*. and PGE.pbc's timestamp isn't bumped right but the old library is stilla round Coke: he said earlier that he kept failing to notice that the make failed. have all depend on clean heh by the way, if pmichaud's caution is warranted, it's probably necessary for more than just PGE.pbc. this is only weird because we have two separate places for PGE.pbc to live which we shouldn't. (which is another issue.) so, you think PGE should be in runtime/parrot/library ? fixing one thing at a time sounds easier if that's where we're running it from, yes. what does the compile in compilers do? nqp and past also? er.. the 'copy' the 'copy' is, like, named 'clone' If these are core components of parrot, they should be in a core location. library seems reasonable to me, yes. tge? i think tge is slooooow. or the tree grammar engine or how you transform a PGE Match Hierarchy into PIR code. anything in compilers, yes. imcc? rumour has it imcc is the c of parrot or the problem or the intermediate code compiler does imcc have any generated PBC files? it's obviously core so it belongs in.... src? let me stop you from enumerating: "everything in compilers/ that generates a PBC should probably generate that PBC into the library." and their makefiles should be part of root.in? that's a separate issue, but yes, I think os. "so" what if PGE simply compiled direct to $(PARROT_LIBRARY)/PGE.pbc ? and didn't create the copy in compilers/pge/ ? I'm not entirely sure why it's not already doing that. +1 1 * Coke smacks uprl. I am having definite transposition issues today. :( uprl yours! If PGE put its finished PBC in $(PARROT_LIBRARY), it could build the stage 1 PBC in the local directory and use that for the stage 2 build. ... provided we can convince Parrot to prefer that PBC over the one in $(PARROT_LIBRARY). right ... which eventually we should be able to do anyway. that's the issue I came up with is there a reason to keep stage 1 around? if stage 2 builds, remove stage 1 as it is now stage 1 isn't kept around doing that keeps it less confusing it's an intermediate build file. we keep a lot of those around. (src/pmc/string.c) no reason to keep those make might try to rebuild if it discovers the intermediate file isn't there except for debugging but .c is a text file correct, if the intermediate isn't present, make will rebuild the target only if the source is newer than the target * Infinoid isn't convinced, and goes off to test that. I need to head out. Regards. well, this would presume that the target lists the intermediate as a dependency so if the intermediate is gone, then make treats it as being out of date and rebuilds it, then uses it to rebuild the target try it with a .o file sometime -- remove the .o, and make will rebuild it and then rebuild the target just tested, GNU make rebuilds a toplevel target if an intermediate target it depends on is removed yes this is exactly how make is supposed to work (see my .o example) yeah so... keep the intermediate around. if it's too confusing to have two PGE.pbc files in two directories with different attributes, call the intermediate miniPGE.pbc that doesn't work because Perl6Grammar.pir expects to find PGE.pbc so we live with the confusion? or overwrite the original? if it's a major pain I'll rework the makefile so that there's only one PGE.pbc (in library) I doubt it matters, as long as library is where parrot looks first however, keep in mind that if PGE fails to build at some point (and it's often hard to notice because it's in the middle of the long root build), then you end up with an incomplete PGE.pbc and things start failing I'm still not sure what the point of rejigging the makefile is at this particular moment. me neither http://aegis.sourceforge.net/auug97.pdf ... but I can imagine a couple of ways where it's possible. link contains lots of good information about make % Limbic_Region has joined #parrot coke's aiming to make make -j make faster pmichaud: how would you feel about removing it upon failure, rather than removing it upon rebuild? hmmm? something along the lines of: $(CC) -o PGE.pbc || (rm PGE.pbc; exit 1) if we can be certain that they remove on failure, sure if that can be made portable heh. s/CC/PARROT/ and make sure that Parrot's exit codes always make sense for -o which it probably should anyway it should, but I'm not sure that it does do we test that yet? sounds useful. thus I kept overlooking that the PGE build was failing understandable in that case, make would overlook it too well, make was overlooking it, so I did all right. tonight I'll write a test I should also note that in the case of PGE, having -j isn't going to help much because it's a sequential process anyway even if we list the dependencies "right" there's no opportunity for parallel build in PGE the Makefile is essentially a build script that is in the form of a Makefile, with one target. increasing the number of targets isn't going to suddenly discover parallelism where it doesn't exist (in PGE's case) right, but that doesn't mean a single all-knowing make couldn't build non-PGE things at the same time ahhh, having sub-makefiles prevents -j from working? I figured it just meant those sub-makefiles couldn't participate in being likewise || having sub-makefiles makes it more difficult one moment, let me dig up a svn rev number ok, sorry, that wasn't as relevant as I had hoped also, I have trouble seeing what might build in parallel with PGE that can't already do so with the present Makefile the whole compilers/PGE directory is currently treated as a single big dependency, which isn't really that far from the truth exactly and tge depends on pge nqp depends on pct and pge json depends on pge so... once pge is done, pct and tge could be built in parallel, and then nqp and json could be built in parallel? actually, I think that pct and pge can be in parallel, then tge and json and nqp currently, the compilers.dummy rule of the Makefile will build those 5 in serial. right I'd change it to be compilers: compilers.pge compilers.pct compilers.tge compilers.json compilers.nqp then have separate $(MAKE) for each of those compilers.pge: $(MAKE) compilers/pge compilers.tge: compilers.pge \n $(MAKE) compilers/tge etc. yeah, something along those lines also, I had to fix a faulty dependency in this area, a week or two ago. the dependencies implied libparrot could be built in parallel with pge, rather than libparrot being a dependency of pge see the latter half of http://www.parrotvm.org/svn/parrot/revision?rev=26643 % iblechbot has left iblechbot!~iblechbot@ppp-62-216-205-218.dynamic.mnet-online.de oh, that looks like a total pain because GEN_LIBRARY includes components that depend on PGE.... well, maybe not to build it seems weird that we would build $(LIBRARY_DIR)/PGE/Util.pbc prior to building PGE itself :-) see, whenever I see something like that, my makefile-rewriting finger starts itching. a n y w a y if you all want to refactor the makefiles, I have no problem with it as long as everything works that's a reasonable goal :) PGE depends on libparrot though. building PGE also depends on Parrot/HLLCompiler.pbc (for now) I do have a problem with increasing the options or number of source code files that are needed to build PGE i.e., if I have to maintain separate Perl6Grammar.pir files or various option switches, that seems like a Net Lose. % rdice_ has left rdice_!~richard_d@CPE0014bfafbbd5-CM0011e6ecf48a.cpe.net.cable.rogers.com I could also eliminate the 2-stage PGE build by simply hand-coding the builtins.pg file as PIR rules (there's only one rule in there at the moment) but if we ever decide we want a library of standard rules to be part of PGE and that those are coded using regexes as source, we'd need a multi-stage build again if we have to move or copy stuff around to maintain atomicity, it should still all come from one source file... anything else is a DRY fail. I think that adding options that are only used for building is also some sort of fail Definitely agreed. * Infinoid starts adding things to t/run/exit.t afk, dinner thanks for the discussion, pm r26857 | bernhard++ | trunk: : Regenerate MANIFEST.SKIP diff: http://www.parrotvm.org/svn/parrot/revision?rev=26857 % barney has joined #parrot r26858 | bernhard++ | trunk: : #52556: [PATCH] Eclectus: simplify PAST generation a bit : Courtesy of Andreas Rottmann diff: http://www.parrotvm.org/svn/parrot/revision?rev=26858 hey! :) hello barney Hi, just coming home from Frankfurt.pm meeting barney: just as an aside: please apply my patches verbatim or reject them (I'm managing them with StGit) % kid51 has joined #parrot % skids has joined #parrot * jonathan yawns after a long day's work negotiating Four hour flight and airport boringness tomorrow. May get some more Rakudo hacking in. rotty: Yes, applying the patches is easier when they are in form as mentioned in submissions.pod jonathan: suggestion, if it's easy -- An error message of "Type blah refuses to be assigned type blah " maybe with a "due to where statement" tacked on for refinements -- would be much more useful than "Type check failed". barney: what did I do wrong? I did have a look at submissions.pod... git patches need to be applied with patch -p1, not patch -p0 the usual 'patch' does not know about the 'a' and 'b' dirs yes, I did not deem this important its a git-ism. I guess I'm used to it. (as you can just use -p1) I just tried that. perhaps I mistyped it should work... -p1 tells it to strip the a/ and b/ skids: Completely agree, the first two parts are maybe easy, though for anonymous refinement types it's a tad harder. % peeps[work] has left peeps[work]!~peepsalot@bwext.kpimdp.com But yeah, needs better error message. I'll see what I can do. Thanks for the feedback. :-) % iblechbot has joined #parrot thanks for the types ;-) jonathan: you can use "" for anonymous types until we have something better r26859 | bernhard++ | trunk: : #52588: [PATCH] Eclectus: refactor Perl test wrappers : Courtesy of Andreas Rottmann. diff: http://www.parrotvm.org/svn/parrot/revision?rev=26859 particle: Sure, good idea. rotty: yes 'patch -p1' works fine :-) that'd be a nice detail to document somewhere particle: actions.pm is getting kinda...big. What would you think to trying to refactor some of the larger actions a bit to use various subs, and putting those into a separate .pm file or files? It's just that it takes a little while to compile. By "what would you think to", I'm not asking you to do it - just asking what you think of the idea. :-) jonathan: i'm not against the idea, as long as they're well-factored Sure Well, actions.pm is close to hitting the minute mark to compile on this laptop. And yes, I know that really means I need a new laptop... r26860 | bernhard++ | trunk: : #52592: [PATCH] Eclectus: Replace brackets with parens : Courtesy of Andreas Rottmann. diff: http://www.parrotvm.org/svn/parrot/revision?rev=26860 % iblechbot has left iblechbot!~iblechbot@ppp-62-216-198-247.dynamic.mnet-online.de barney: what reason do you want to switch to generating NQP instead of PAST for? * particle heads out for a bike ride & I want to keep the PAST, just change the PAST-generation code from PIR to NQP. Just because the syntax is nicer, not so many temporary variables ah, I see The PAST generating code can be one big nested constructor btw, what do you think about using riaxpander to do the macro expansion stuff for us? riaxpander is http://mumble.net/~campbell/darcs/riaxpander/ s/is/is at/ btw, there is a good overview of Scheme macro systems at http://lists.gnu.org/archive/html/chicken-users/2008-04/msg00013.html rotty's url is at http://xrl.us/bi4y6 % schmalbe has joined #parrot rotty: I'll take a look at it tomorrow. barney: are you ok with a subdirectory for each Scheme we support for bootstrapping, or should that be layed out differently? schmalbe: ok subdirs seem sane. Have you checked with Ikarus? schmalbe: Ikarus doesn't run on my main machine (it needs SSE2 or something) (so, no) r26861 | bernhard++ | trunk: : #52600: [PATCH] Eclectus: now works also with Guile : Courtesy of Andreas Rottmann. diff: http://www.parrotvm.org/svn/parrot/revision?rev=26861 % barney has left barney!~bernhard@dslb-084-058-173-232.pools.arcor-ip.net schmalbe: we should probably select a few (maybe 3) implementations for bootstrapping, to keep the overhead low r26862 | bernhard++ | trunk: : [Eclectus] : Set SVN properties for newly added files. diff: http://www.parrotvm.org/svn/parrot/revision?rev=26862 I'm very much for dumping Petite chez, its command-line interface is *very* limited % Limbic_Region has left Limbic_Region!~Limbic_Re@c-68-49-236-220.hsd1.md.comcast.net Do you recommend Guile ? schmalbe: Guile has some problems with macros and modules; Gauche is better in that respect r26863 | bernhard++ | trunk: : Give credit to Andreas Rottmann. diff: http://www.parrotvm.org/svn/parrot/revision?rev=26863 (I'm not sure this matters for our purposes, but if we want to use riaxpander, it might) * schmalbe deprecates Petite Chez Scheme for Eclectus dalek: are you aware of the harness "-le" problem? oh, that should have been schmalbe: ... rotty: Are you on IRC tomorrow? the current eclectus codebase is not very much suited for compilers because of the way the test harness works, btw schmalbe: yes Can we do some planning this evening? (Barney realizes it's past midnight) schmalbe: sure * schmalbe calls it a day Cool :-) % IllvilJa has left IllvilJa!~jilves@emea-netcache1.oracle.co.uk % schmalbe has left schmalbe!~bernhard@dslb-084-058-136-186.pools.arcor-ip.net % TonyC has joined #parrot % nopaste has left nopaste!~opaste@202-154-105-237.people.net.au % TonyC has left TonyC!~tony@202-154-105-237.people.net.au % nopaste has joined #parrot % rdice has joined #parrot kid51: (re: Perl::Critic disappearing) you could always try the brute force method "Infinoid" at 96.238.213.50 pasted "make perlcritic.t dump @INC to stderr" (12 lines) at http://nopaste.snit.ch/12660 if there's anything weird like the search path being truncated, or another rev of perl being invoked, it'll show up there. Infinoid: What does 'brute force' mean in this context? (the problem seems to be with make test as much as Perl::Critic) On another subject: Do you have anything to add re http://rt.perl.org/rt3/Ticket/Display.html?id=52154 -- the ticket having to do with pulling 'svk' and 'git' related code out of Parrot::Revision? Coke will have to make a decision, because it's one of those things where we can't make everybody happy. % TonyC has joined #parrot kid51: by "brute force", I meant: attack the problem of "why can't it find my module" head-on, by verifying the environmental things that could affect module loading % tetragon has joined #parrot does bitcard have any "don't forget about my login" related preferences? "kid51" at 71.247.48.190 pasted "'prove' finds Perl::Critic with no problem; why not 'make test'?" (19 lines) at http://nopaste.snit.ch/12662 Infinoid: Don't know; unfamiliar with Bitcard except it simplest usage. Would that have a bearing on problem at hand? just an annoyance at being bounced to a page that doesn't have "Reply" buttons, no big deal % ruoso has left ruoso!~ruoso@81.84.156.87 hmm. I don't really feel too strongly about #52154. if the concensus is to keep svn around, I won't challenge it, as long as everything else is simplified % sjansen has left sjansen!~sjansen@hq-nat2.gurulabs.com % cognominal has left cognominal!~cognomina@82.67.232.89 % paco has left paco!~chatzilla@139.Red-80-36-122.staticIP.rima-tde.net % paco has joined #parrot you could pretty easily parse the svn rev out of "svn info". but the same argument could be made of Configure.pl doing so I think the main issue was the pain of having to support svk, git, git-svn, and that is being addressed so ... do you want me to post that to the ticket? Yes, please post whether you agree with the patch or not and what any outstanding issues may be. (Asking you 'cause I remember our conversation on IRC 2 weeks ago about this.) re: Data::Dumper, ok, that's bizarre. Now, something equally bizarre ... but a pattern is developing: "kid51" at 71.247.48.190 pasted "Where t/*.t files do an 'eval {require ...}' to locate a Perl module" (12 lines) at http://nopaste.snit.ch/12663 ok, I've cast my vote Ooh, I've got a hunch! it almost looks like the harness is doing something that breaks the *first* use, or require, or whatever, in interesting ways I got a weird warning from YAML::Syck in that patch I nopasted earlier, too. something about a barename only being used once, possible typo what's the hunch? My hunch: Note that there's no semicolon after Perl::Critic -- nor is there one after Moose in that other test. hmm. do BEGIN blocks get concatenated together before execution? But why 'make test' would gag there when 'prove' and 't/harness' don't, I cannot explain. The semicolon issue is a red herring. The final statement in a block does not require a semicolon in Perl 5. Okay, then what color is the herring? Red. * kid51 set himself up for that one. Dump @INC to STDERR from the test file and see what's different between runs. chromatic: http://nopaste.snit.ch/12662 (standalone) kid51: does "perl t/harness t/codingstd/perlcritic.t" look the same? It looks same as 'prove' i.e., it locates Perl::Critic and runs the file without complaint. so, the harness is fine, something from the Makefile maybe? for use of perl t/harness, see my post to list of 17:18:09 PDT on my box, "make test" actually runs: /usr/bin/perl5.10.0 t/harness --gc-debug --running-make-test `which perl` /usr/bin/perl (symlink to perl5.10.0) Are you seeing this craziness too? no. chromatic: When I include the semicolon, 'make test' runs t/codingstd/perlcritic.t without complaint! That's unique. $ perl eval { require CGI::Cookie }; print $@; print $INC{'CGI/Cookie.pm'} /usr/local/share/perl/5.8.8/CGI/Cookie.pm Note: I realize that I don't actually have Moose installed Well, I guess it can't hurt to put that semicolon after 'require Perl::Critic' -- correct? Correct, but I fear it's covering up some sort of weirdness elsewhere. Shall I commit, or do you want to explore further? c, can you reproduce? No, I can't. like I said, I don't see this craziness, but I do see other craziness. like, warnings from modules that normally load silently. Infinoid: You see such warnings ... when? I only get that from Text::Balanced. Name "YAML::Syck::ImplicitBinary" used only once: possible typo at /usr/lib64/perl5/vendor_perl/5.10.0/x86_64-linux-thread-multi/YAML/Syck.pm line 65. I get that under perl t/harness, but not under prove. * kid51 doesn't have YAML::Syck installed anywhere. prove -v * Infinoid can't even begin to explain this. I mean prove --version no warning under prove -v oh prove v2.64, using Test::Harness v2.64 and Perl v5.10.0 TAP::Harness v3.02 and Perl v5.8.8 Hm, I thought 5.10 included TAP::Harness 3. "kid51" at 71.247.48.190 pasted "patch to assure that 'make test' locates Perl::Critic" (27 lines) at http://nopaste.snit.ch/12665 Guys, I gotta step away. If you want to (literally) patch the problem, there's a suitable patch. % kid51 is now known as kid51_afk 5.10 does not seem to include TAP::Harness % kid51_afk has left kid51_afk!~jkeen@pool-71-247-48-190.nycmny.east.verizon.net Maybe it just missed the cutoff. Do you see the weirdness with TAP::Harness? % Ademan_ has left Ademan_!~dan@h-68-164-168-66.snfccasy.dynamic.covad.net % silug has left silug!~steve@ppp-70-225-32-179.dsl.covlil.ameritech.net % Ademan_ has joined #parrot anyone think we can close rt#51732? I'm sure it was a lacking realclean. chromatic: yes, I still see the weirdness with TAP::Harness v3.10 but I still don't know whether my weirdness is related to kid51's weirdness. The missing semicolon is bizarre. what do you think our possibilities of coming up with a oneliner that reproduces it are? Pretty low, unless it includes an -I % rdice has left rdice!~richarddi@CPE001217e365c7-CM00159a01d44c.cpe.net.cable.rogers.com % kid51 has joined #parrot (makefile) I'm aiming to get all the dependencies worked out. having -j work is a nice side effect there. kid51: are you sure you're using prove and perl from the same distro? My Perl is 5.10, compiled from source the day it came out. My prove is that associated with most recent Test::Harness (3.10) Darwin: [jimk] 503 $ prove --version TAP::Harness v3.10 and Perl v5.10.0 was that test::harness installed against the same perl? Linux: [li11-226:~] 501 $ prove --version TAP::Harness v3.09 and Perl v5.10.0 (just that would explain the library discrepancies.) Yes. Because TAP::Harness 3.10 is more recent than Perl 5.10. % Theory has left Theory!~Theory@c-24-21-175-208.hsd1.or.comcast.net But note that this error only occurred during 'make test' and only when perlcritic.t read: eval { require Perl::Critic }; But it occurred, albeit with different output, on 2 different OSES. OSES. OSes. Also, note that since perlcritic.t was only very recently included in 'make test', I wouldn't have noticed this problem, say, 3 months ago. % wknight8111 has left wknight8111!~nobody@c-71-230-33-251.hsd1.pa.comcast.net kid51: do you think we can close rt#51732? (I'm reviewing tickets in my inbox and happened across this one.) are you using perl 5.10 on both OSes? Yes. But here's some new data. "kid51" at 71.247.48.190 pasted "Using the 'vendor' Perl and prove on Darwin to run perlcritic.t" (8 lines) at http://nopaste.snit.ch/12666 That depends on where you have P::C installed and what's in @INC. That's the first case where I can use a version of 'prove' to simulate the results with make test. Err, let's see. that one's probably because P::C is installed in /usr/local/lib/perl5/site_perl/5.10.0/Perl/Critic.pm I doubt that's in 5.8.6's search path. do you have another copy installed? Yes. the @INC for 5.8.6 does not include the directory where P::C is installed. sorry I'm installing P::C right now on my Darwin install (it used to be there, but was lost in a system upgrade) tetragon: All those dependencies are fun, right? hehe I think people grumble about LedgerSMB's dependencies more I met Elliot, the current maintainer of P::C, when I was at the mini-hackathon in Chicago in December. Nice guy, but I still hate modules where the cpan shell forces me to say yes 3 times. Mainly 'cause it's sitting on PPI. % Theory has joined #parrot % Theory has left Theory!~Theory@c-24-21-175-208.hsd1.mn.comcast.net So, would my 'make test' -- on either OS -- for some reason not know to look in the @INC associated with 5.10? And why would a semicolon make a difference? Perhaps with 5.10 you need the last semicolon in a block??? Incidentally, vendor Perl on OS X 10.5 picks up on the P::C I just installed with prove And that test is swamping my system which test? t/codingstd/perlcritic.t Unless something changed the parser in a backwards-incompatible way, the semicolon isn't the problem. Really? It's not particularly speedy on my iBook G4, but there are others slower. It has finished It's pretty IO intensive. And subtest 3 failed It was consuming all CPU The question we've been discussing is that 'make test' on my boxes fails to locate Perl::Critic and therefore SKIPs that test. The failed subtest looks like it was my modifications Oooooh. I patched config/init/defaults.pm so that I wouldn't have to remove all the -arch flags from all the Makefiles Ah, yes, *that* patch :) r26864 | coke++ | trunk: : Add a TODO test for #46499: [RFE] Allow comment lines in PIR .param list diff: http://www.parrotvm.org/svn/parrot/revision?rev=26864 % Eevee has joined #parrot % kid51 has left kid51!~jkeen@pool-71-247-48-190.nycmny.east.verizon.net % silug has joined #parrot % silug has left silug!~steve@ppp-70-225-32-179.dsl.covlil.ameritech.net r26865 | infinoid++ | trunk: : [t] Make sure parrot returns non-zero on parse failure, and doesn't generate : an output file. diff: http://www.parrotvm.org/svn/parrot/revision?rev=26865 pmichaud: so far, parrot's return value looks good. but my test case is pretty simple, and might need expanding. % allison has left allison!~chatzilla@sub17-30.member.dsl-only.net Infinoid++ % tetragon has left tetragon!~seneca@CPE0040d001f62f-CM000a736592a8.cpe.net.cable.rogers.com % contingencyplan has left contingencyplan!~contingen@cpe-76-186-27-146.tx.res.rr.com % allison has joined #parrot http://www.cs.dartmouth.edu/reports/abstracts/TR2001-404/ r26866 | pmichaud++ | trunk: : [pct]: : * Allow --encoding=fixed_8 to work for HLLCompiler. diff: http://www.parrotvm.org/svn/parrot/revision?rev=26866 % silug has joined #parrot r26867 | pmichaud++ | trunk: : [rakudo]: : * Since actions.pm doesn't have any non-ascii characters in it yet, : we can tell NQP to compile using fixed_8 instead of the utf8 default. : This gives us a 28% (101 versus 141 sec) speed improvement on : my box. diff: http://www.parrotvm.org/svn/parrot/revision?rev=26867 % silug has left silug!~steve@ppp-70-225-32-179.dsl.covlil.ameritech.net Nice. I'll take temporary victories until we can fix the real things. % iblechbot has joined #parrot % uniejo has joined #parrot % iblechbot has left iblechbot!~iblechbot@ppp-62-216-200-240.dynamic.mnet-online.de % Theory has joined #parrot % Theory has left Theory!~Theory@c-24-21-175-208.hsd1.mn.comcast.net % allison has left allison!~chatzilla@sub17-30.member.dsl-only.net