Camelia, the Perl 6 bug

IRC log for #parrot, 2011-01-06

Parrot | source cross referenced

| Channels | #parrot index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:01 mikehh whiteknight: I got a TODO pass with test 127, but test 49 fails with an optimized build (Ubuntu 10.10 i386) it passes without --optimize
00:01 bacek left #parrot
00:01 whiteknight ok
00:02 mikehh I fixed test 127, but then looking at your commit, saw you had modified test 49
00:03 mikehh but the thing is it works fine with g++, but fails with g++ --optimize
00:05 fbrito joined #parrot
00:08 davidfetter joined #parrot
00:12 mikehh whiteknight: get the same story with gcc, t/pmc/integer.t passes test 49 with gcc but fails with gcc --optimize
00:13 mikehh test 127 ok in all cases
00:15 mikehh test 49 - # Have: 8.94427190999916, # Want: 4+8i - in optimized builds, but passes without --optimize
00:18 kid51 joined #parrot
00:21 hercynium left #parrot
00:23 kid51 is now known as kid51_at_dinner
00:27 whiteknight mikehh: Looking over the old code, I think it's always been broken and the test was passing by pure, dumb luck
00:28 whiteknight the way the old code was, the result value was never being assigned back to $P0. Not explicitly anyway
00:29 whiteknight Hah! and here's the reason: That test was marked TODO
00:29 davidfetter dukeleto, ping
00:29 whiteknight it never did pass before. I untodo'd it because my changes added in the "passes by dumb luck" behavior in the non-optimized build
00:32 dalek parrot: e2af51a | Whiteknight++ | t/pmc/integer.t:
00:32 dalek parrot: remove the test for Integer.i_multiply on Complex. That test had been TODO'd, and I mistakenly untodo'd it. It has never passed as written (which is probably wrong) and the MULTI case that it was designed to test has been removed
00:32 dalek parrot: review: https://github.com/parrot/parrot/commit/e2af51a5c5
00:32 whiteknight mikehh: fixed and committed
00:32 mikehh whiteknight: yeah I saw that, I think I need to look at what is actually happening in the test, see why it was passing
00:33 whiteknight it's deleted now. I'm going to look at test #127 to see why it is passing
00:33 whiteknight mikehh: nevermind, I see now why I never saw that TODO test pass: I don'thave GMP installed on this box
00:34 whiteknight or, any of my boxen
00:34 dukeleto davidfetter: pong
00:34 davidfetter dukeleto, looking for a way to get access to my android device
00:34 davidfetter anybody had any luck with the droid2world?
00:38 dukeleto davidfetter: never heard of it
00:38 davidfetter it's one of the newer motorola ones. mine's verizon branded
00:40 NotFound_gf left #parrot
00:41 dmalcolm left #parrot
00:43 whiteknight cotto_work: ping
00:43 cotto_work whiteknight: pong
00:43 whiteknight cotto_work: I want to make finalize op no longer experimental
00:43 whiteknight we voted on it at #ps a few weeks ago with weak support and no opposition
00:44 cotto_work I remember.
00:44 whiteknight In current Parrot, it's essential for non-crashy behavior
00:44 whiteknight Users of it include NQP, Rakudo, Winxed, etc
00:44 cotto_work +1
00:44 whiteknight Okay, that's all the encouragement I need. I'm marking it not experimental
00:45 cotto_work great
00:46 dukeleto +1
00:46 gbacon left #parrot
00:47 cotto_work kid51_at_dinner: ping
00:50 Yuki`N joined #parrot
00:51 contingencyplan left #parrot
00:54 rfw left #parrot
00:55 rfw joined #parrot
00:56 kid51_at_dinner is now known as kid51
00:56 kid51 cotto_work: pong
01:03 cotto_work kid51: are you happy with the results of the make help gci task?
01:03 kid51 cotto_work: Frankly, not entirely.
01:04 kid51 I was hoping that there would be at least one comment on each target -- especially the more obscure ones -- even if that comment was only "Required external dependency."
01:04 kid51 After all, I don't know what some of those targets are about.
01:05 cotto_work kid51: could you give the student some feedback to that effect?  There's no need to be shy about saying that a task wasn't completed to your satisfaction.
01:05 kid51 I'm not sure I've seen that student on channel, nor do I have an email address for him.
01:05 kid51 In melange or in the Trac ticket?
01:06 cotto_work either place is fine
01:06 cotto_work wherever you think he's more likely to see it
01:06 cotto_work I just want to make sure he doesn't get left hanging.
01:08 dalek parrot: 02de970 | Whiteknight++ | / (3 files):
01:08 dalek parrot: the finalize op is no longer experimental
01:08 dalek parrot: review: https://github.com/parrot/parrot/commit/02de97087c
01:09 kid51 cotto_work: Different subject.  Around noon ET today I was able to build NQP and Rakudo and get them to pass their 'make test' targets.
01:09 kid51 Is it your belief that that has persisted?
01:09 kid51 If so, I can close that ticket.
01:10 cotto_work kid51: yes.  plobsing++ took care of it Tuesday evening.
01:12 cotto_work Kristaba: do you have any failing tests in your branch?  I see several.
01:13 bacek joined #parrot
01:13 bacek cotto, ping
01:13 kid51 cotto_work: Now I can't seem to spot that GCI task.  Did someone close it?
01:13 cotto_work bacek: pong
01:13 cotto_work bacek: more awesomeness?
01:14 bacek cotto, i've got strange GC failure in PIRATE+opmap_aware. Parrot VM: PANIC: custom_mark flag set but no custom mark routine defined!
01:14 kid51 Hmm, yes someone closed it.
01:15 bacek It's always during mark of PFOpMap
01:15 cotto_work I did, though I'm not sure why.
01:15 kid51 Ah, that someone was named cotto.
01:15 cotto_work must have been a misfire
01:15 Kristaba cotto: Yes, as I write in the github push comment, I have a lot of failing tests in t/compilers/imcc/opt1.t and opt2.t, but they are due to a bug in the imcc optimizer
01:16 whiteknight cotto_work: can you take a gander at TT #1515 eventually (no rush)
01:16 whiteknight it's pretty open-ended. I don't know whether it needs fixing or not
01:18 cotto_work rfw: coordinate with Matt to make sure you're not both refactoring out the same bits of fill_params.
01:19 nopaste "cotto" at 192.168.1.3 pasted "failing tests with kristaba's changes' test_status perl tools/dev/nopaste.pl -t failing" (45 lines) at http://nopaste.snit.ch/27572
01:19 Kristaba O_o
01:19 cotto_work goofy title, but the contents are correct
01:20 Kristaba I don't have these errors
01:20 bacek cotto, PIRATE exposes "bloody big memory fuckup"...
01:20 Kristaba cotto: Wait, I'll do a make fulltest again
01:22 Kristaba cotto: You're sure that you merge my branch in the zeroargs_subs_checking branch?
01:22 Kristaba Not in master?
01:23 dalek TT #1635 closed by whiteknight++: Runloop unrolling for exception handling
01:23 dalek TT #1635: http://trac.parrot.org/parrot/ticket/1635
01:23 dalek TT #833 closed by whiteknight++: Exception from :init sub in PIR compreg causes problems
01:23 cotto_work Kristaba: I checked the git log
01:23 dalek TT #833: http://trac.parrot.org/parrot/ticket/833
01:23 cotto_work apparently melange is letting students take more than one task now
01:23 cotto_work matt has two active ones
01:24 cotto_work I lie
01:24 cotto_work it's fbrito
01:24 fbrito hello :D
01:24 cotto_work Kristaba: just ignore me until I can look at it at home.  My brain is fried.
01:27 nopaste "Kristaba" at 192.168.1.3 pasted "Failing tests with "make fulltest" on my changes" (10 lines) at http://nopaste.snit.ch/27573
01:30 kid51 cotto_work: Commented on melange: http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129385113378
01:31 kid51 Kristaba: You said that was on 'make fulltest'.  Which runcore?
01:32 kid51 Hmm: I'm getting a failure on t/compilers/opsc/06-opsfile.t in 'master'
01:33 nopaste "kid51" at 192.168.1.3 pasted "t/compilers/opsc/06-opsfile.t: New failure." (25 lines) at http://nopaste.snit.ch/27574
01:33 Kristaba kid51: runcore?
01:34 Kristaba My god! With "make test" I have a huge number of test fails!
01:35 Kristaba What's the problem? fulltest isn't 'better' than test?
01:36 rfw yikes, fill_params is absolutely ginormouos
01:38 cotto_work rfw: yes.  hence the gci task
01:40 cotto_work rfw: If you can't get in contact with Matt, I recommend you drop the task until you can.  There's potential for merge conflicts if you don't coordinate.
01:40 rfw ah okay
01:40 rfw is he doing it too?
01:40 cotto_work yes.  I made two tasks and intended to publish the second once the first was done, but it got published prematurely.
01:40 rfw oh okay
01:41 rfw i don't have anything else to do so i guess i'll be fine just holding this one
01:41 cotto_work I am looking forward to seeing how that turns out.
01:41 Kristaba cotto_work: I'll try to fix this tomorrow, it's really too late in France, but if you have any idea of why fulltest (my task description says to run "full test") doesn't check all tests...
01:41 cotto_work Kristaba: that's a bug that we need to fix.
01:41 cotto_work go to sleep.  I'll make sure something happens.
01:42 Kristaba Okay, good (night/day) #parrot
01:42 cotto_work msg cotto It seems that Kristaba's branch passes under fulltest but fails under test.  Figure out what's going on.
01:42 aloha OK. I'll deliver the message.
01:43 Kristaba left #parrot
01:43 kid51 whiteknight: ping
01:44 whiteknight pong
01:45 kid51 Is it possible your last commit led to that failure in t/compilers/opsc/06-opsfile.t?  Could there be a problem in the test itself?
01:45 whiteknight kid51: possible, yes
01:45 whiteknight I'm not familiar with that test
01:46 kid51 Me neither.
01:46 cotto_work I can look when I get home.  What's explodey?
01:48 whiteknight test 1 in that file fails
01:48 whiteknight says it can't parse, but is light on details
01:49 kid51 I've got it.  It's just a counting problem.
01:49 whiteknight I moved finalize_p from experimental.ops to core.ops. I don't know if I missed a step
01:50 plobsing that test hardcodes the number of ops it expects. adding/moving/deleting ops will make it go boom.
01:51 whiteknight awesome
01:51 dalek parrot: 76466ef | jkeenan++ | t/compilers/opsc/06-opsfile.t:
01:51 dalek parrot: With certain ops no longer experimental, we have to change the expected number of ops parsed in certain files.
01:51 dalek parrot: review: https://github.com/parrot/parrot/commit/76466ef772
01:51 whiteknight (that test)--
01:51 whiteknight kid51++
01:52 dalek parrot: 0be4b23 | Whiteknight++ | / (2 files):
01:52 dalek parrot: add a new API function to register/unregister a PMC with GC
01:52 dalek parrot: review: https://github.com/parrot/parrot/commit/0be4b23460
01:52 whiteknight Parrot::Embed is hella old and has extremely limited functionality
01:52 kid51 plobsing: Thanks for taking care of the problem that was aggravating NQP yesterday.
01:52 whiteknight I'm thinking we're going to want to rewrite it
01:52 whiteknight If we move it out of the main repo, I don't know what the deprecation policy says about it
01:53 Yuki`N The problem with forking Configure.pl is I can only figure out how to get one fork running which I have a pipe to, so I can only log stdout or stderr.
01:54 Yuki`N I have a preliminary version as a pull request, but that requires redirecting STDERR to STDOUT.
01:55 dalek parrot/opmap_aware_pmcs: 1cda22b | bacek++ | src/pmc/packfilebytecodesegment.pmc:
01:55 dalek parrot/opmap_aware_pmcs: Remove unused .destroy
01:55 dalek parrot/opmap_aware_pmcs: review: https://github.com/parrot/parrot/commit/1cda22bccc
01:55 dalek parrot/opmap_aware_pmcs: 7fcad83 | bacek++ | src/pmc/packfileopmap.pmc:
01:55 dalek parrot/opmap_aware_pmcs: Call proper vtable on OpLib
01:55 dalek parrot/opmap_aware_pmcs: review: https://github.com/parrot/parrot/commit/7fcad8308a
01:55 dalek parrot/opmap_aware_pmcs: 1a9ca63 | bacek++ | src/pmc/packfileopmap.pmc:
01:55 dalek parrot/opmap_aware_pmcs: Made .mark more inheritance-friendly
01:55 dalek parrot/opmap_aware_pmcs: review: https://github.com/parrot/parrot/commit/1a9ca634b2
01:55 kid51 The GCI task corresponding to Trac TT #1925 is still unclaimed.
01:56 kid51 Yuki`N: I have to admit that I'm not really in agreement with the thrust of the RFC in the Trac ticket from which your task was created.
01:56 plobsing msg bacek bytecode->op_{info,func}_table are populated by anything that generates packfiles (imcc, pbc load, etc). op_info_table makes it easy to find out anything about the ops without linearly scanning the mapping, and op_func_table is similar but only for finding functions.
01:56 aloha OK. I'll deliver the message.
01:57 bacek plobsing, ok, thanks.
01:57 kid51 OTOH, Coke doesn't agree with all of my RFCs, either ;-)
01:57 bacek plobsing, so, I have to populate them in PackfileBytecode PMC?
01:57 dalek parrot: b1e1172 | cotto++ | src/embed/pmc.c:
01:57 dalek parrot: apostrophe fix
01:57 dalek parrot: review: https://github.com/parrot/parrot/commit/b1e117217c
01:58 whiteknight blah. I make that mistake all the time
01:58 plobsing bacek: if you want the packfile to work, then yes.
01:58 whiteknight and I know the difference between its and it's. My fingers are just stupid about it
01:59 bacek plobsing, hm. They do work now without it. I didn't try dynops yet.
02:00 plobsing bacek: they do work without what? the func_table is required for op dispatch and the info_table gets used by various tools.
02:00 bacek plobsing, may be. But my PMC-generated "hello world" do work without it.
02:01 bacek check opmap_aware_pmcs branch
02:01 bacek examples/pir/make_hello_pbc.pir
02:01 mtk left #parrot
02:03 davidfetter left #parrot
02:08 mtk joined #parrot
02:09 whiteknight left #parrot
02:10 kid51 Do we have any guidelines as to which tests need/ought to be written in parrot-nqp?
02:10 kid51 (The file I fixed earlier was the first time I ever ventured into that language.)
02:11 plobsing bacek: how does that work? the contant ids don't even agree.
02:11 bacek plobsing, no idea :)
02:12 plobsing bacek: so long as you are just dumping to pbc, you don't need either of those tables
02:12 plobsing but, if you want these to replace the structs, there are users of these tables
02:12 bacek plobsing, yes. This is main purpose of those PMCs.
02:12 bacek At least for now.
02:13 bacek Just generate workable PBC.
02:13 plobsing bacek: what about bypassing Eval?
02:13 bacek plobsing, erm. I don't get this question
02:14 plobsing bacek: if the packfile pmcs generated runnable bytecode, compilers could bypass the use of IMCC/Eval.pmc to implement runtime eval.
02:14 plobsing Eval is nasty
02:14 fbrito I am having some problems with set_*_keyed_str on OrderedHash PMC. when I try to test them, set_*_keyed is called instead
02:14 aantn joined #parrot
02:14 bacek plobsing, it will be next step. For now I just dump PBC.
02:15 plobsing fbrito: there is no PIR opcode that directly calls *_*_keyed_str
02:15 plobsing you have to find some other, indirect route of calling them
02:16 fbrito ah, thank you  :D. I have probably claimed another impossible task, ehhehe
02:17 cotto_work It's not impossible.  It just requires a different solution than the obvious one.
02:21 fbrito hm...
02:25 cotto_work either make x_x_keyed dispatch to x_x_keyed_x or drop x_x_keyed_x
02:30 shortcircuit joined #parrot
02:30 shortcircuit left #parrot
02:36 fbrito removing them don't affect the tests, but...
02:39 cotto_work but?
02:40 * cotto_work decommutes, hoping for answers when he gets home
02:40 fbrito I don't know... I just don't fell comfortable removing code from such a big project like Parrot :D
02:40 shortcircuit joined #parrot
02:41 kapace__ fbrito, would it help if it were a small project? :P
02:42 fbrito I think it is easier to understand the impact of code changes in smaller projects :P
02:44 kapace__ fbrito, just make sure that Rakudo and lua-parrot build, and tests pass
02:46 plobsing fbrito: there are instances of use of that vtable by the internals. it is entirely possible that those places never see objects of that type
02:47 plobsing use a search tool (editor, grep -r, ack) to find them and consider whether it makes sense for an object of the type under consideration to be used there
02:48 aantn left #parrot
02:55 fbrito ok, I will do some research
02:57 bacek left #parrot
02:58 dalek parrot/spaces_in_paths: 257785c | jkeenan++ | t/tools/pmc2cutils/04-dump_pmc.t:
02:58 dalek parrot/spaces_in_paths: 'glob() doesn't handle spaces in path well; use 'readdir' instead.
02:58 dalek parrot/spaces_in_paths: review: https://github.com/parrot/parrot/commit/257785c922
03:09 cotto ~~
03:18 bacek joined #parrot
03:21 contingencyplan joined #parrot
03:24 bacek left #parrot
03:32 bacek joined #parrot
03:32 bacek cotto, ping.
03:37 Yuki`N left #parrot
03:38 cotto bacek, pong
03:38 bacek cotto, there is some very-very strange GC bug with OpMap. Stuff below it just get collected.
03:38 bacek I couldn't figure out why...
03:38 cotto does it get marked?
03:42 bacek cotto, yes, afaiu
03:43 bacek #3  0xb7eab409 in Parrot_PackfileOpMap_get_integer_keyed_str (
03:43 bacek interp=0x80f8020, _self=0x9d63748, str_key=0x862a050)
03:43 bacek at src/pmc/packfileopmap.c:132
03:43 bacek 132    return VTABLE_get_integer_keyed(interp, _self, pmc_key);
03:43 bacek (gdb) p _self->flags & b_PObj_on_free_list_FLAG
03:43 bacek $3 = 0
03:43 bacek (gdb) down
03:43 bacek #2  0xb7eab048 in Parrot_PackfileOpMap_get_integer_keyed (interp=0x80f8020,
03:43 bacek _self=0x9d63748, key=0x9d6d61c) at src/pmc/packfileopmap.c:83
03:43 bacek 83    if (VTABLE_exists_keyed(interp, map_cache, key)) {
03:43 bacek (gdb) p map_cache->flags & b_PObj_on_free_list_FLAG
03:43 bacek $4 = 524288
03:44 cotto what explodes it?
03:45 bacek No idea
03:45 cotto building the branch now
03:45 bacek Just regular workflow
03:45 cotto complete_workflow.t?
03:45 bacek no
03:45 bacek just test suite of PIRATE
03:46 bacek t/03-pbc/03-sugar.t
03:47 bacek afk # flying back to Sydney
03:48 dalek parrot: a976f75 | Coke++ | lib/Parrot/Pmc2c/VTable.pm:
03:48 dalek parrot: Don't use Exporter; speed up pmc2c.pl
03:48 dalek parrot: build of src/pmc/integer.c goes from 2s to .4s
03:48 dalek parrot: (??!?)
03:48 dalek parrot: review: https://github.com/parrot/parrot/commit/a976f75fed
03:48 dalek parrot: cf51aae | Coke++ | lib/Parrot/Pmc2c/UtilFunctions.pm:
03:48 dalek parrot: Remove useless scalar.
03:48 dalek parrot: review: https://github.com/parrot/parrot/commit/cf51aae39c
03:54 bacek left #parrot
04:05 kid51 left #parrot
04:22 bacek joined #parrot
04:29 bacek left #parrot
04:32 bacek joined #parrot
04:39 bacek cotto, fixed, apparently :)
04:39 dalek parrot/opmap_aware_pmcs: 21ce7a0 | bacek++ | src/pmc/packfilebytecodesegment.pmc:
04:39 dalek parrot/opmap_aware_pmcs: set custom_mark flag early.
04:39 dalek parrot/opmap_aware_pmcs: review: https://github.com/parrot/parrot/commit/21ce7a0828
04:39 dalek parrot/opmap_aware_pmcs: a7d80c9 | bacek++ | src/pmc/packfileopmap.pmc:
04:39 dalek parrot/opmap_aware_pmcs: Fix PFOM.mark. Invoking VTABLE_mark isn't sufficient obviousely.
04:39 dalek parrot/opmap_aware_pmcs: review: https://github.com/parrot/parrot/commit/a7d80c9cb2
04:39 dalek parrot/opmap_aware_pmcs: e933bc3 | bacek++ | src/pmc/packfile (2 files):
04:39 dalek parrot/opmap_aware_pmcs: Use more idiomatic accessors.
04:39 dalek parrot/opmap_aware_pmcs: review: https://github.com/parrot/parrot/commit/e933bc3ef6
04:41 cotto ok
04:41 cotto quick flight
04:53 bacek it will depart in about an hour
05:01 dalek parrot: 97d08bb | dukeleto++ | docs/project/git_workflow.pod:
05:01 dalek parrot: [docs] How to Maintain a Fork on Github
05:01 dalek parrot: review: https://github.com/parrot/parrot/commit/97d08bb43d
05:14 bacek cotto, pingy-ping
05:14 dalek pir: a5083f5 | bacek++ | src/POST/Compiler.pm:
05:14 dalek pir: Fix selection of :main sub.
05:14 dalek pir: review: https://github.com/parrot/pir/commit/a5083f55c9
05:15 dalek pir: a5083f5 | bacek++ | src/POST/Compiler.pm:
05:15 dalek pir: Fix selection of :main sub.
05:15 dalek pir: review: https://github.com/parrot/pir/commit/a5083f55c9
05:15 bacek hmmm. Double karma?
05:15 bacek sorear, dalek reports commit twice. github problem?
05:15 bacek msg sorear, dalek reports commit twice. github problem?
05:15 aloha OK. I'll deliver the message.
05:16 bacek msg cotto Handling CLI is good "medium" GCI task :). E.g. C<./installable_pir --target=pbc -o foo.pbc foo.pir>
05:16 aloha OK. I'll deliver the message.
05:16 cotto ok
05:17 bacek msg cotto it will require parrot on branch, some tree optimizations and bloody PIRATE :)
05:17 aloha OK. I'll deliver the message.
05:17 cotto msg bacek I'm here.  You don't have to msg me.
05:17 aloha OK. I'll deliver the message.
05:17 bacek yes-yes :)
05:17 bacek I'm too lazy to remove "msg " after I already typed it.
05:18 cotto bacek, why not merge into master?
05:18 bacek I'm still hacking it a bit.
05:19 bacek Can be merged anyway.
05:19 Coke t/src/embed.t (Wstat: 0 Tests: 15 Failed: 0) TODO passed:   3
05:20 cotto ok
05:22 cotto merge whenever you hit a natural pause
05:22 bacek in progress
05:22 bacek half-an-hour before departure
05:22 bacek running make test
05:23 bacek cotto, #1038?
05:23 bacek as GCI candidate
05:24 sorear bacek: transition issue... parrot/pir wasn't removed from the poll list after pushing was configured
05:24 bacek sorear, ah, ok.
05:24 bacek I don't mind double karma :)
05:24 sorear I've fixed this for parrot/pir
05:25 sorear but somebody with admin access to the parrot organization needs to weed duplicates
05:25 bacek cotto, ^^^^
05:25 sorear too bad github has no post-receive API
05:26 cotto dukeleto has an admin bit for parrot on github
05:26 dalek parrot: e588ed8 | bacek++ | / (12 files):
05:26 dalek parrot: Merge branch 'opmap_aware_pmcs'
05:26 dalek parrot: review: https://github.com/parrot/parrot/commit/e588ed8b61
05:29 dalek pir: 398bca1 | bacek++ | src/POST/Compiler.pm:
05:29 dalek pir: Fix handling .return. Claim back one more test.
05:29 dalek pir: review: https://github.com/parrot/pir/commit/398bca1570
05:32 dalek pir: 74d2c77 | bacek++ | src/POST/Compiler.pm:
05:32 dalek pir: Fix storing Keys in ConstantTable. Claim few more tests back.
05:32 dalek pir: review: https://github.com/parrot/pir/commit/74d2c7773e
05:35 dalek parrot: 91e3883 | petdance++ | src/ (3 files):
05:35 dalek parrot: removed unused variables
05:36 dalek parrot: review: https://github.com/parrot/parrot/commit/91e3883e2c
05:36 dalek parrot: 2369288 | petdance++ | / (12 files):
05:36 dalek parrot: Merge branch 'master' of github.com:parrot/parrot
05:36 dalek parrot: review: https://github.com/parrot/parrot/commit/2369288fd5
05:38 bacek left #parrot
05:40 fbrito hm, is anyone here?
05:40 cotto hi
05:42 fbrito ah, sorry. figured out my self :)
05:43 rurban_ joined #parrot
05:43 cotto ok, but don't let it happen again
05:44 fbrito I am making some tests with OrderedHash and negative index values
05:45 dalek parrot/leto/embed_grant: 2bd6a62 | dukeleto++ | t/src/embed.t:
05:45 dalek parrot/leto/embed_grant: [t] Add extend_vtable_output_is() to prevent repeated boilerplate in embed tests
05:45 dalek parrot/leto/embed_grant: review: https://github.com/parrot/parrot/commit/2bd6a6278b
05:45 dalek parrot/leto/embed_grant: c20ee48 | dukeleto++ | t/src/embed.t:
05:45 dalek parrot/leto/embed_grant: [t] Convert many tests to use extend_vtable_output_is and delete a lot of code
05:45 dalek parrot/leto/embed_grant: review: https://github.com/parrot/parrot/commit/c20ee48bd7
05:45 rurban left #parrot
05:45 rurban_ is now known as rurban
05:49 fbrito for an n-element OrderedHash, hash[-n] = 1 and hash[-(n+1)] = 1 are really supposed to be the same? didn't know about that :o
05:51 cotto yuck
05:54 fbrito if I create an empty hash and push 7 elements to it, hash[-7] = 0 and hash[-8] = 0 are both going to change the 1st value of the hash to 0?! :o
05:55 cotto just cut that out.  It's not a sane behavior.
05:55 cotto do I even want to know what hahs[-9] = 0 does?
05:55 kapace__ lol
05:57 ppant joined #parrot
06:05 fbrito we need to test how the code handles with negative index when they are bigger than the number of elements
06:08 fbrito s/with//
06:29 chromatic left #parrot
06:32 aantn joined #parrot
06:35 fbrito I can't understand when is the line 349 be called (http://tapir2.ro.vutbr.cz/cover/cover​-results/2011-01/2011-01-05-4ff4ae3/c​_cover/src-pmc-orderedhash-pmc.html)
06:37 fbrito going to be*
06:46 dalek ohm-eta-wink-kzd: 8df082c | plobsing++ | src/o (2 files):
06:46 dalek ohm-eta-wink-kzd: proper way to check that an object has a method in winxed (emulated in JS too)
06:46 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/8df082ce7d
06:46 dalek ohm-eta-wink-kzd: ac4faa0 | plobsing++ | src/ometa-base.winxed:
06:46 dalek ohm-eta-wink-kzd: implement {g,s}et_int_keyed on OMeta to allow integer state in grammars
06:46 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/ac4faa06a2
06:46 dalek ohm-eta-wink-kzd: e92018c | plobsing++ | src/ (2 files):
06:46 dalek ohm-eta-wink-kzd: use quote-new and init_pmc for JumpTable to get initialization working
06:46 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/e92018c991
06:46 dalek ohm-eta-wink-kzd: 2cfaabc | plobsing++ | src/winxed-compiler.dual:
06:46 dalek ohm-eta-wink-kzd: add quoted-new syntax
06:46 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/2cfaabcdca
06:46 dalek ohm-eta-wink-kzd: 9483502 | plobsing++ | src/Ωη.capo.winxed:
06:46 dalek ohm-eta-wink-kzd: fix thinko in JumpTable.toTree()
06:46 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/9483502a43
06:46 dalek ohm-eta-wink-kzd: cde831c | plobsing++ | src/Ωη.coda.winxed:
06:46 dalek ohm-eta-wink-kzd: use push opcode on stringbuilder to fix jumptable code generation
06:46 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/cde831caf5
06:46 dalek ohm-eta-wink-kzd: 619f97b | plobsing++ | src/ojs.coda.js:
06:46 dalek ohm-eta-wink-kzd: semi-colon discrepancy fix
06:46 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/619f97b083
06:46 dalek ohm-eta-wink-kzd: ea27760 | plobsing++ | src/Ωη.capo.winxed:
06:46 dalek ohm-eta-wink-kzd: coerce hash iterator key to string
06:46 dalek ohm-eta-wink-kzd: review: https://github.com/plobsing/ohm​-eta-wink-kzd/commit/ea27760b31
06:48 plobsing fbrito: it gets called when there is an existing value for the key and the key is composite (example: ['composite';'key'], counter-example: ['simple-key'])
06:48 fbrito ahh
06:48 * plobsing sleeps
06:48 fbrito thank you very much
06:48 fbrito good night :)
06:50 cotto fbrito, found one
06:50 cotto oh
06:55 dukeleto cotto: did you need something?
06:56 cotto good question.  Did I ping you?
06:56 * cotto remembers
06:56 dukeleto cotto: you mentioned me having admin bits
06:56 cotto I think there's a post-commit hook on parrot/pir that needs to be removed
06:57 dukeleto cotto: why so?
06:57 cotto it was causing duplicate karma for PIRATE commits
06:57 cotto nothing serious
06:58 dukeleto cotto: actually, something needs to be deleted from dalek-conf.json from the perl6/mu repo
06:58 cotto dukeleto, why?  Isn't it better to have all configuration in one place?
07:01 dukeleto cotto: read this https://github.com/perl6/mu/bl​ob/master/misc/dalek-push.txt
07:01 dukeleto cotto: https://github.com/perl6/mu/bl​ob/master/misc/dalek-conf.json
07:01 dukeleto sorear++ fixed the issue an hour ago :)
07:02 dukeleto cotto: also, please take a look at my new addition about keeping a fork in sync in git_workflow.pod
07:02 cotto I saw.
07:02 dukeleto cotto: that should help GCI students, now that there is only 4 days left...
07:02 dukeleto but will be useful for anyone forking on github...
07:03 cotto for single-purpose forks master isn't too bad, but I agree that we should recommend that most work go in  a branch.
07:04 dukeleto cotto: single-purpose fork?
07:05 dukeleto cotto: if someone wants to ever sync up to upstream (git rebase), then committing to master leads to pain
07:05 dukeleto so i just recommend against it.
07:05 dukeleto Anyone smart enough to ignore the advice in git_workflow.pod and still get stuff done is more than welcome to.
07:05 dukeleto I just don't want to answer the same questions any more :)
07:05 cotto that's a good motivation
07:06 cotto I mean someone who forks Parrot to make a single change and never plans on changing anything again
07:06 cotto but even the user forking can't know for sure if that'll be the case
07:06 dukeleto cotto: sure, in that case, committing to master is fine.
07:06 dukeleto cotto: yep. as insurance for the future, it is best to recommend a branch, but if they ignore that and never sync, there is no harm.
07:07 dukeleto i have forked many github repos planning to work on them and then I never make a single commit.
07:08 theory left #parrot
07:08 dukeleto cotto: i have one gcc compile farm machine running "make smoke" once an hour on master. It ain't much, but it is a start.
07:09 dukeleto cotto: if there are specific config flags that we want to test (because perhaps they are uncommonly chosen), let me know
07:09 cotto we should at least have both --optimize and normal
07:09 fbrito (wow. I have typed §P0 instead of $P0 and lost 10 minutes trying to figure out why I was getting this error: error:imcc:syntax error, unexpected $endInvalid character in ASCII string)
07:10 cotto perhaps c++ too, though that gets some kind of regular testing
07:10 dukeleto fbrito: that's the way she goes, sometimes.
07:10 dukeleto fbrito++ for figuring it out :)
07:11 cotto fbrito, did you find a decent test case for that code?
07:11 dukeleto cotto: i am only doing --optimize right now
07:11 fbrito cotto which code?
07:12 dukeleto fbrito: we are talking about testing parrot with a c++ compiler
07:12 dukeleto cotto: perhaps some of the NCI Configure.pl flags? Which config flags are most likely to break?
07:12 cotto fbrito, the orderedhash code you were trying to cover a bit earlier this evening
07:14 fbrito cotto: I am trying to test the compound keys
07:15 cotto yup
07:18 fbrito cotto: And there is no PIR opcode that calls set_*_keyed_str and delete_keyed_str. Removing them (except set_pmc_keyed_str, that is called in internal code) didn't affect tests or build, but I feel that I should research a little bit more
07:21 cotto fbrito, that's a longstanding stupidity of Parrot.
07:22 KatrinaTheLamia joined #parrot
07:22 KatrinaTheLamia left #parrot
07:23 cotto My recommendation would be to either make delete_keyed a dispatcher to deleted_keyed_x (same for the others) or nuke x_keyed_x
07:23 KatrinaTheLamia joined #parrot
07:27 fbrito what I have right now is here: https://github.com/fernandobrito​/parrot/commits/gci_orderedhash
07:27 fbrito it increased the test coverage from 92.7% to 98.4%. I am trying to get it closer to 100%
07:31 cotto what's missing
07:31 cotto ?
07:32 fbrito those if clauses with "next" (plobsing mentioned the composite keys, but they are not working as expected) and something else... wait
07:34 aantn left #parrot
07:36 cotto do you have a coverage report you can nopaste or something?
07:36 fperrad joined #parrot
07:37 sorear dukeleto: You misread the scrollback.  I still need you.
07:39 cotto actually I can generate that myself
07:40 sorear dukeleto: Or rather, I still need the attention of someone with parrot org admin access.
07:42 fbrito cotto: line 349, 674, and 739 of http://tapir2.ro.vutbr.cz/cover/cover​-results/2011-01/2011-01-05-4ff4ae3/c​_cover/src-pmc-orderedhash-pmc.html
07:43 fbrito those are the lines that I haven't covered
07:45 cotto 349 is easy enough
07:45 nopaste "cotto" at 192.168.1.3 pasted "coverage for line 349" (6 lines) at http://nopaste.snit.ch/27575
07:48 dalek parrot/leto/embed_grant: 720c466 | dukeleto++ | t/src/embed.t:
07:48 dalek parrot/leto/embed_grant: [t] Parrot_PMC_(assign_pmc|add|defined)
07:48 dalek parrot/leto/embed_grant: review: https://github.com/parrot/parrot/commit/720c46602a
07:53 aantn joined #parrot
07:55 cotto the others look similar
08:00 fbrito hm, interesting
08:02 fbrito it didn't cover the line 349 for me :o
08:03 cotto odd.  I stuck a fprintf in there and it got hit
08:04 cotto wrong file
08:05 nopaste "cotto" at 192.168.1.3 pasted "for real this time" (8 lines) at http://nopaste.snit.ch/27576
08:08 fbrito I couldn't read those values that were just set
08:08 fbrito I am getting "1" instead
08:09 aantn asides from "parrot t/testfile.t" is there another way to run one test at a time?
08:09 aantn I want to see an overview of each tast (i.e. PASS or FAIL)
08:09 fbrito aantn: you can run them with "prove"
08:10 fbrito "prove t/..."
08:10 fbrito aantn: also take a look on "prove -h"
08:10 aantn fbrito: thanks
08:11 fbrito cotto: this works: http://pastie.org/1433375 but if I change from Hash to OrderedHash, it will return 11 instead of "bc"
08:14 cotto that's special.
08:15 cotto Intuitively it looks wrong, but I need to get to bed.  If you suspect that it's an incorrect behavior, please file a ticket.
08:15 fbrito I am also going to bed
08:15 fbrito thank you for your help
08:15 cotto OrderedHash used to be more hacky than it is now, so it hasn't gotten as much love as Hash.
08:15 kapace__ good night everybody
08:15 cotto 'night
08:43 cogno joined #parrot
08:59 cogno left #parrot
09:04 cogno joined #parrot
09:18 cogno left #parrot
09:24 cogno joined #parrot
09:32 dalek lua: 02251a6 | fperrad++ | lua/ (2 files):
09:32 dalek lua: fix the following error:
09:33 dalek lua:     too many named arguments: 1 passed, 0 used
09:33 dalek lua: review: https://github.com/fperrad/lua/commit/02251a6d77
09:43 ppant left #parrot
09:51 rfw left #parrot
09:54 dalek wmlscript: abca4e7 | fperrad++ | dynext/pmc/wmlsbytecode.pmc:
09:54 dalek wmlscript: string_make was removed of Parrot
09:54 dalek wmlscript: review: https://github.com/fperrad/​wmlscript/commit/abca4e7833
09:57 ligne joined #parrot
09:57 ligne left #parrot
10:14 bacek joined #parrot
10:17 cogno left #parrot
10:18 dalek nqp-rx: 412953d | bacek++ | src/setting/IO.pm:
10:18 dalek nqp-rx: Improve c<slurp> to be more consistent with Perl6 and work in utf8/binary mode
10:18 dalek nqp-rx: review: https://github.com/perl6/nqp-rx/commit/412953d1e3
10:18 dalek nqp-rx: b812ff7 | bacek++ | src/stage0/nqp-setting.nqp:
10:18 dalek nqp-rx: Reboostrap setting only.
10:18 dalek nqp-rx: review: https://github.com/perl6/nqp-rx/commit/b812ff74dd
10:19 dalek parrot: d8405fa | bacek++ | ext/nqp-rx/src/stage0/nqp-setting.nqp:
10:19 dalek parrot: Bring new nqp setting with utf8-aware slurp
10:19 dalek parrot: review: https://github.com/parrot/parrot/commit/d8405fa1c1
10:19 dalek parrot: e5b39b7 | bacek++ | src/ (3 files):
10:19 dalek parrot: Merge branch 'master' of github.com:parrot/parrot
10:19 dalek parrot: review: https://github.com/parrot/parrot/commit/e5b39b7ca6
10:20 tadzik bacek: wanna take a look at my .worry, while near nqp-rx?
10:21 bacek tadzik, nope :) It's slightly out of my responsibility.
10:21 bacek tadzik, I mostly work on nqp setting.
10:21 tadzik oh, ok :)
10:22 tadzik I'll wait for pm then
10:22 bacek tadzik, you can ask sorear also
10:23 tadzik sorear: you around?
10:24 tadzik it could use some refactoring though
10:27 cogno joined #parrot
10:29 sorear tadzik: for a few seconds longer
10:30 M_o_C joined #parrot
10:31 tadzik sorear: could you take a look at my patch in the nqp-rx GH issues?
10:32 sorear no
10:32 sorear not now
10:32 tadzik ok
10:47 fbrito left #parrot
11:10 contingencyplan left #parrot
11:45 cogno left #parrot
11:45 cogno joined #parrot
11:54 mikehh t/pmc/packfileopmap.t FAILS in make corevm/make coretest - Could not find oplib `math_ops'  (passes make test after make world)
12:03 bacek mikehh, oops. packfileopmap shouldn't be in coretest than
12:10 cogno left #parrot
12:23 Drossel left #parrot
12:31 JimmyZ joined #parrot
12:32 JimmyZ left #parrot
12:41 bluescreen left #parrot
12:46 kid51 joined #parrot
12:54 mikehh bacek: maybe you can check for 'math_ops' and skip if not there - see t/pmc/integer.t (check for 'sys_ops')
12:55 cogno joined #parrot
13:04 Kulag joined #parrot
13:10 cogno left #parrot
13:20 kid51 left #parrot
13:34 whiteknight joined #parrot
13:37 whiteknight good morning, #parrot
13:43 lucian joined #parrot
13:43 rurban_ joined #parrot
13:45 rurban left #parrot
13:45 rurban_ is now known as rurban
13:51 whiteknight msg bacek PackFileBytecodeSegment.load_lib looks like it enters into an infinite loop
13:51 aloha OK. I'll deliver the message.
13:51 M_o_C left #parrot
13:58 kapace_ joined #parrot
13:58 aantn left #parrot
13:58 whiteknight msg plobsing can you take a look at TT #749 when you get a chance? A GCI student implemented this but you reverted it. If we really don't want to go this route for now can you close the ticket? Thanks
13:58 aloha OK. I'll deliver the message.
13:59 kapace__ left #parrot
14:15 lucian left #parrot
14:27 aantn joined #parrot
14:27 bluescreen joined #parrot
14:28 DPini joined #parrot
14:28 DPini left #parrot
14:28 DPini joined #parrot
14:32 Coke left #parrot
14:33 aantn left #parrot
14:41 mtk left #parrot
14:46 mtk joined #parrot
15:07 Patterner left #parrot
15:07 Psyche^ joined #parrot
15:07 Psyche^ is now known as Patterner
15:22 Myhrlin left #parrot
15:33 DPini left #parrot
15:35 whiteknight is there anybody here with a working rakudo build?
15:36 * whiteknight can't build Rakudo on his little VM
15:36 fbrito joined #parrot
15:40 dalek TT #856 closed by whiteknight++: segfault in pbc_merge
15:40 dalek TT #856: http://trac.parrot.org/parrot/ticket/856
15:41 moritz whiteknight: last I tried rakudo built fine on newest parrot, but had about 5 spectest failures
15:42 moritz which was yesterday
15:44 lucian joined #parrot
15:47 contingencyplan joined #parrot
15:47 whiteknight moritz: I have a rakudo-related ticket that I'm trying to triage. Would you be able to take a look at TT #802?
15:48 whiteknight it's 18 months old, and I'm trying to figure out if it's still a problem or not
15:50 * moritz takes a look
15:50 donaldh joined #parrot
15:52 moritz turns out I need to build rakudo first :-)
15:57 dalek TT #1128 closed by whiteknight++: Possible memory error in compilers/imcc/parser_util.c:INS()
15:57 dalek TT #1128: http://trac.parrot.org/parrot/ticket/1128
16:04 mtk left #parrot
16:05 mtk joined #parrot
16:08 donaldh left #parrot
16:09 lucian left #parrot
16:13 moritz bah, rakudo on latest parrot is broken again :(
16:23 [hercynium] joined #parrot
16:24 chromatic joined #parrot
16:25 tadzik aloha
16:27 [hercynium] is now known as hercynium
16:28 moritz oh hai zebra
16:29 tadzik o/
16:29 dalek parrot: ea9f63e | Whiteknight++ | docs/embed.pod:
16:29 dalek parrot: start to bring docs/embed.pod up to date with current embedding API
16:29 dalek parrot: review: https://github.com/parrot/parrot/commit/ea9f63e0f1
16:29 plobsing left #parrot
16:29 tadzik did anything funny happen today? :)
16:29 dmalcolm joined #parrot
16:30 dalek TT #1591 closed by whiteknight++: PackFile_new_dummy is not in embed.h nor extend.h
16:30 dalek TT #1591: http://trac.parrot.org/parrot/ticket/1591
16:30 whiteknight stupid PackFile_new_dummy
16:31 whiteknight msg plobsing I want to get rid of PackFile_new_dummy. I think this shouldn't be too hard, but there are a few strategies we could follow to do it. I would like your input on it before I start ripping.
16:31 aloha OK. I'll deliver the message.
16:39 cotto whiteknight, that code sounds special.  What does it do?
16:43 JimmyZ joined #parrot
16:46 fbrito should I file a ticket, or is this the appropriate behavior of OrderedHash? http://pastie.org/1434417
16:46 JimmyZ left #parrot
16:50 plobsing joined #parrot
16:50 moritz whiteknight: re rakudo related segfault, the PIR example isn't valid syntax anymore (probably realted to opcode removal)
16:51 moritz tt #802 that is
16:59 whiteknight moritz: great.
17:00 moritz whiteknight: I've updated the ticket, loading io_ops helped
17:00 whiteknight cotto: If there is no packfile loaded into the interpreter, attempting to load_bytecode or similar currently fails because load_bytecode attempts to do a merge. PackFile_new_dummy creates a new empty packfile in the interp to allow new things to be merged in
17:01 whiteknight moritz++
17:01 gbacon joined #parrot
17:02 JimmyZ joined #parrot
17:23 plobsing left #parrot
17:36 gottreu joined #parrot
17:38 contingencyplan left #parrot
17:40 contingencyplan joined #parrot
17:43 contingencyplan left #parrot
17:44 DPini joined #parrot
17:45 contingencyplan joined #parrot
17:46 Kristaba joined #parrot
17:47 gottreu left #parrot
17:47 Kristaba cotto: ping
17:49 gottreu joined #parrot
17:52 theory joined #parrot
17:55 dukeleto mornin'
17:55 dukeleto whiteknight: +1 to killing PackFile_new_dummy
17:55 tadzik o/
17:58 dukeleto whiteknight: i see you made a huge change to the embed docs
17:58 cotto_work ~~
17:58 cotto_work Kristaba: pong
17:59 dukeleto whiteknight: you removed some of the docs that i just added as part of my TPF grant
18:00 Kristaba cotto: Hey, what about my task problem today?
18:00 dukeleto whiteknight: we still want to keep the old docs around while the old embed system is deprecated, for people that are still using the old embed api (like everyone)
18:00 dukeleto whiteknight: with a large DO NOT WRITE NEW CODE WITH THIS API warning
18:01 dukeleto whiteknight: what about docs/embed_old.pod ? And then when the old embed api is deleted from a supported release, we remove embed_old.pod ?
18:01 theory left #parrot
18:01 Kristaba cotto: Did you find the time to look at that? :)
18:02 dukeleto whiteknight: for instance, for people that want to hack on converted PL/Parrot to the new embed API, it is useful to compare the old docs and new
18:02 dukeleto s/converted/converting/
18:08 cotto_work Kristaba: I've verified that the branch I'm working with does have the zeroarg changes.
18:09 cotto_work and your changes
18:09 theory joined #parrot
18:10 dukeleto whiteknight: is there no way to choose runcores with the new embed API?
18:11 Kristaba cotto_work: And about the differences between make fulltest and make test?
18:11 cotto_work I will look into those now.
18:11 Kristaba ok
18:14 dalek parrot: 40e018d | Whiteknight++ | docs/pdds/pdd27_multiple_dispatch.pod:
18:15 dalek parrot: remove :invocant TODO from the PDD27
18:15 dalek parrot: review: https://github.com/parrot/parrot/commit/40e018d047
18:17 cotto_work Kristaba: I only see failures in t/compilers/imcc/imcpasm/opt1.t and t/compilers/imcc/imcpasm/opt2.t with fulltest, probably because our fulltest target runs a bunch of tests in serial and dies when one batch fails.
18:17 cotto_work with test I see many more failures.
18:18 dalek parrot/gci_orderedhash_coverage: 7e13bd7 | fbrito++ | src/pmc/orderedhash.pmc:
18:18 dalek parrot/gci_orderedhash_coverage: Remove unused set_*_keyed_str and delete_keyed_str from OrderedHash
18:18 dalek parrot/gci_orderedhash_coverage: review: https://github.com/parrot/parrot/commit/7e13bd7526
18:18 dalek parrot/gci_orderedhash_coverage: 5c61023 | fbrito++ | t/pmc/orderedhash.t:
18:18 dalek parrot/gci_orderedhash_coverage: [t] Increase OrderedHash test coverage
18:18 dalek parrot/gci_orderedhash_coverage: review: https://github.com/parrot/parrot/commit/5c610238ad
18:18 dalek parrot/gci_orderedhash_coverage: 50f36a3 | fbrito++ | t/pmc/orderedhash.t:
18:18 dalek parrot/gci_orderedhash_coverage: [t] Increase OrderedHash test coverage even more
18:18 dalek parrot/gci_orderedhash_coverage: review: https://github.com/parrot/parrot/commit/50f36a3860
18:18 dalek parrot/gci_orderedhash_coverage: b39915c | fbrito++ | t/pmc/orderedhash.t:
18:18 dalek parrot/gci_orderedhash_coverage: [t] Increase OrderedHash test coverage to 100%
18:18 dalek parrot/gci_orderedhash_coverage: review: https://github.com/parrot/parrot/commit/b39915ccba
18:19 dalek parrot/gci_fix_zeroargs_tests: 81b0439 | (Léo Grange)++ | t/ (12 files):
18:19 dalek parrot/gci_fix_zeroargs_tests: Fix all tests which failed in zeroargs_subs_checking branch
18:19 dalek parrot/gci_fix_zeroargs_tests: review: https://github.com/parrot/parrot/commit/81b0439853
18:19 dukeleto fbrito++
18:19 Kristaba cotto: Ok, so I'll simply fix all the test failures reported by make test?
18:19 dukeleto Simple Matter Of Programming.
18:19 cotto_work Kristaba: yes
18:20 dalek parrot/gci_tt1199: 984cb27 | Yuki`N++ | / (2 files):
18:20 dalek parrot/gci_tt1199: Mod config script to tee output to config.log
18:20 dalek parrot/gci_tt1199: review: https://github.com/parrot/parrot/commit/984cb27a7d
18:25 JimmyZ left #parrot
18:25 dalek TT #483 closed by whiteknight++: :invocant flag
18:25 dalek TT #483: http://trac.parrot.org/parrot/ticket/483
18:28 plobsing joined #parrot
18:28 darbelo joined #parrot
18:29 cotto_work seen tcurtis
18:29 aloha tcurtis was last seen in #parrot 1 days 22 hours ago joining the channel.
18:29 dukeleto Yuki`N++
18:29 dukeleto karma N
18:29 aloha N has karma of 50.
18:29 * dukeleto giggles
18:30 cotto_work That N is really a stand-up guy.
18:32 tadzik (Yuki`N)++
18:32 tadzik karma (Yuki`N)
18:32 aloha (Yuki`N) has karma of 0.
18:32 tadzik nah
18:33 cotto_work karma Yuki`N
18:33 aloha Yuki`N has karma of 0.
18:47 dukeleto Evidently, nobody with a ` can have karma. Who knew?
18:48 dukeleto karma Little`Bobby`Tables";drop database;
18:48 aloha Little`Bobby`Tables";drop database; has karma of 0.
18:49 darbelo left #parrot
18:52 davidfetter joined #parrot
18:52 cotto_work bacek: ping
18:54 moritz $ ./perl6 -e 'say 1'
18:54 moritz ===SORRY!===
18:54 moritz Confused at line 1, near ".loadlib '"
18:54 moritz that's on newest parrot
18:56 fbrito dukeleto: thank you for having a look on my task :)
18:57 cotto_work moritz: will try to duplicate
19:00 davidfetter left #parrot
19:01 Kristaba Can someone explain me how to debug this kind of error "current instr.: 'anon' pc 0 (EVAL_1:1)"?
19:01 dzoe left #parrot
19:03 Kristaba I don't understand what the EVAL_1 "files" refers to
19:05 bkuhn joined #parrot
19:05 moritz it means that it's not directly compiled from a file, but has gone through an eval() or so
19:07 dukeleto Kristaba: it basically means you are inside an anonymous function evaluation
19:08 plobsing left #parrot
19:09 DPini left #parrot
19:10 Kristaba Okay, thanks
19:16 davidfetter joined #parrot
19:17 cotto_work moritz: verified
19:20 cotto_work checking if it's something fulltest catches
19:29 plobsing joined #parrot
19:30 rfw joined #parrot
19:31 dalek TT #1935 created by pmichaud++: nqp-rx on Parrot RELEASE_2_11_0-771-g40e018d fails t/nqp/33-init.t
19:31 dalek TT #1935: http://trac.parrot.org/parrot/ticket/1935
19:36 dalek parrot: 766fed5 | cotto++ | MANIFEST.generated:
19:36 dalek parrot: update MANIFEST.generated after opmap_aware_pmc merge
19:36 dalek parrot: review: https://github.com/parrot/parrot/commit/766fed586e
19:36 Khisanth left #parrot
19:41 cotto_work whiteknight: are you looking into the breakage?  I want to get this fixed.
19:41 whiteknight yeah, I'm on it
19:41 whiteknight stupid tailcalls are stupid
19:41 pmichaud note that it reportedbly breaks rakudo as well.
19:41 cotto_work is that what's broken?
19:41 pmichaud I'm testing that and will file a ticket on that too.
19:41 cotto_work pmichaud: confirmed that rakudo's broken
19:41 cotto_work even hello world
19:41 whiteknight cotto_work: I suspect so. The issues are looking very similar to problems I fixed yesterday in #1029
19:41 pmichaud so, we don't need a ticket?
19:42 whiteknight i'm exploring that issue now
19:42 cotto_work pmichaud: no.  There's no way we'll forget about this.
19:42 particle1 left #parrot
19:45 cotto_work Whatever failure this is isn't caught by fulltest.
19:46 pmichaud according to the nqp failure, it has something to do with :load subroutines
19:46 pmichaud (rakudo fails its 'make test', yes.)
19:46 particle joined #parrot
19:47 cotto_work whiteknight: have you tested if rakudo works with that #1029 fix reverted?
19:47 whiteknight cotto_work: working on it
19:47 pmichaud (hint:  test nqp -- it's faster :)
19:47 cotto_work or that
19:48 * moritz currently bisects the nqp failure
19:49 moritz I'm close to finishing
19:49 moritz last or second-to-last step
19:49 pmichaud it looks to me as though the :load subroutines aren't executing in sequence.
19:50 pmichaud (but that's just a guess at this point)
19:50 pmichaud oh, nm
19:50 pmichaud something else must be happening
19:50 cotto_work nqp-rx passes all tests with the 1029 fix reverted (apart from one related to ICU, whcih I haven't installed
19:50 whiteknight yeah, it's stupid .tailcall again
19:50 cotto_work )
19:50 whiteknight damnit
19:51 cotto_work If Rakudo hello world works, we need to revert that fix.
19:51 whiteknight I'm working on it
19:51 cotto_work awesome
19:51 pmichaud I've been trying to eliminate .tailcall from all of my sources
19:51 plobsing left #parrot
19:52 pmichaud (because it's been problematic with profiling as well)
19:52 moritz my bisect blames 6231646a848d71d90185bbeb0ef4b8383f29a767, which I somehow doubt
19:52 cotto_work pmichaud: couldn't you just hack tailcall to behave like a normal call?
19:52 dalek parrot: 9328cff | Whiteknight++ | compilers/pct/src/PCT/HLLCompiler.pir:
19:52 dalek parrot: remove a .tailcall optimization from HLLCompiler.pir.
19:52 dalek parrot: This fixes a build failure in NQP (and, presumably, Rakudo as well)
19:52 dalek parrot: review: https://github.com/parrot/parrot/commit/9328cfff49
19:52 dalek parrot/zeroargs_subs_checking: 81b0439 | (Léo Grange)++ | t/ (12 files):
19:52 dalek parrot/zeroargs_subs_checking: Fix all tests which failed in zeroargs_subs_checking branch
19:52 dalek parrot/zeroargs_subs_checking: review: https://github.com/parrot/parrot/commit/81b0439853
19:53 whiteknight I can't build rakudo here. So can somebody test that fix with Rakudo?
19:53 pmichaud cotto_work: I don't understand
19:53 cotto_work whiteknight: will do
19:53 whiteknight ls
19:53 cotto_work .
19:53 cotto_work ..
19:54 moritz no such file or directory
19:54 cotto_work I'm checking how the revert of 8b188db does first
19:54 cotto_work whiteknight: if your fix doesn't work, that one looks promising
19:55 Khisanth joined #parrot
19:56 whiteknight cotto_work: 8b188db? You have a link?
19:56 cotto_work https://github.com/parrot/parrot/commit/8b188db
19:56 dalek parrot: 4bc0723 | nwellnhof++ | / (2 files):
19:56 dalek parrot: [io] Remove platform specific code from socket_api.c
19:56 dalek parrot: review: https://github.com/parrot/parrot/commit/4bc07237bb
19:56 dalek parrot: 84f8d95 | nwellnhof++ | / (2 files):
19:56 dalek parrot: [io] Remove Parrot_io_is_encoding
19:56 dalek parrot:
19:56 dalek parrot: Unneeded after unicode_io merge
19:56 dalek parrot: review: https://github.com/parrot/parrot/commit/84f8d957cd
19:56 dalek parrot: d5d4070 | nwellnhof++ | / (3 files):
19:57 whiteknight oh, right. Yes, my most recent commit reverts part of 8b188db
19:57 dalek parrot: Fix headerizer warnings
19:57 dalek parrot: review: https://github.com/parrot/parrot/commit/d5d4070712
19:57 cotto_work may look familiar
19:58 pmichaud fwiw, I'd prefer that the .tailcall in HLLCompiler be simply used to track down a confirmed parrot bug -- not that we want the optimization permanently applied
19:58 pmichaud there's almost no significant improvement to be had by using the .tailcall
19:58 pmichaud (other than we get failures)
19:58 pmichaud I'll add that note to the ticket
20:04 dalek TT #1029 reopened by coke++: taillcall into PIR compiler broken
20:04 dalek TT #1029: http://trac.parrot.org/parrot/ticket/1029
20:06 pmichaud afk, lunch
20:09 Kristaba bacek: ping
20:12 bluescreen left #parrot
20:17 cotto_work nqp-rx and Rakudo seem to be back to normal
20:18 whiteknight I can't get over how stupid IMCC is
20:18 whiteknight NEED_CONTINUATION is defined as (PMC*)0x01
20:18 whiteknight and during all my testing, the PIR compreg always set interp->current_cont = NEED_CONTINUATION
20:18 whiteknight but in some tests, it appears to be setting interp->current_cont = NULL instead
20:18 whiteknight so my previous fix didn't catch that, and NQP explodes
20:20 cotto_work two things:
20:20 cotto_work 1 - We need automated HLL testing.  This is my new mission in life.
20:20 cotto_work 2 - can you add a test so that this particular bit of idiocy doesn't bite us or HLLs again?
20:21 whiteknight way ahead of you
20:21 cotto_work that's not too hard
20:21 dalek parrot: ef82f5c | Whiteknight++ | / (2 files):
20:21 dalek parrot: remove special-case code in NCI.invoke. We never assume the interp->current_cont coming out of func is valid now. Add a test to prove that this no longer breaks
20:21 dalek parrot: review: https://github.com/parrot/parrot/commit/ef82f5ccda
20:21 whiteknight the test is ugly, but does fail before this fix, and passes after
20:22 cotto_work tests can be ugly
20:23 cotto_work dukeleto: ping
20:28 dukeleto cotto_work: pong
20:28 cotto_work dukeleto: setting up automated HLL testing of the latest Parrot just got important.  Does the gcc compile farm allow any sort of web access?
20:28 dukeleto cotto_work: that is exactly where i am blocked right now. I don't know that answer to that. I most likely have to ask
20:29 cotto_work dukeleto: ok
20:29 dukeleto cotto_work: why did it recently get a lot more important?
20:29 * dukeleto backlogs a little
20:30 cotto_work we broke Rakudo again
20:30 cotto_work and automated testing most likely would have caught it
20:32 dukeleto cotto_work: yes, it would
20:32 dukeleto cotto_work: i plan to use jitterbug to automate testing (in addition to using plumage)
20:33 dukeleto cotto_work: https://github.com/franckcuny/jitterbug
20:33 dukeleto cotto_work: i've spend the last few days improving jitterbug, because i am using it at $work as well
20:33 dukeleto cotto_work: i have a commit bit
20:34 cotto_work I see
20:34 dukeleto cotto_work: http://sgn-build-64.sgn.cornell.edu:8090/ is an example
20:34 dukeleto cotto_work: it integrates with perlbrew to be able to test under all versions of perl that perlbrew knows about
20:34 dukeleto cotto_work: so we can test parrot with all the perls we support
20:35 whiteknight okay, my latest fix fixes .tailcall for the nqp failures
20:35 cotto_work dukeleto: that's good, but we really need testing against HLLs.
20:35 whiteknight I'm not going to replace it in the HLLCompiler code, but if we did I think it would work
20:35 cotto_work whiteknight: I verified that you fixed it.
20:35 dukeleto cotto_work: i hear ya
20:35 whiteknight I'm going to re-close #1029 now
20:36 dukeleto cotto_work: how about you start a wiki page and describe all the features that we need from automated testing?
20:36 pmichaud I'd like to also interject here the comment I just made on #perl6 to the effect that I'm disappointed with the breakages, but I'm equally disappointed with the ongoing attitude of committers who "fix bugs reported by rakudo devs" but don't actually check that the reported bug was really fixed.
20:36 whiteknight cotto: there are two "fixes". The first was removing .tailcall. The second was fixing NCI.invoke
20:36 whiteknight I've done both
20:36 cotto_work ok
20:36 pmichaud whiteknight: I'm not sure that #1029 can be closed.
20:36 dukeleto cotto_work: and how important each feature is relative to others
20:36 pmichaud the original reported bug apparently still exists.
20:36 cotto_work dukeleto: that's a good idea
20:36 whiteknight pmichaud: I've put in a fix, added a test for the fix, and verified that NQP continues to work if we chose to replace the .tailcall. What else would be needed to close it?
20:36 dukeleto pmichaud: i just want to say that I don't dismiss HLL concerns
20:37 pmichaud dukeleto: I did say committers.
20:37 dukeleto pmichaud: but i do agree that Parrot and HLLs currently don't effectively communicate
20:37 pmichaud dukeleto: I agree that Parrot's managers (in their manager role) have been extremely response.
20:37 pmichaud *responvie.
20:37 pmichaud *responsive.
20:37 fbrito1 joined #parrot
20:37 pmichaud whiteknight: oh, I didn't see the ef82f5c fix (although I did look for it)
20:38 pmichaud nm then -- I agree we can close the ticket, and Whiteknight++
20:38 pmichaud afk, kid pickup from school
20:38 whiteknight pmichaud++ Thanks for your patience. I can close #1935 too?
20:38 pmichaud yes.
20:39 dukeleto cotto_work: i don't think smolder does what we need now. It is a black hole of information, has no notifications or summaries, and loses lots of test reports
20:39 dukeleto I am not happy with Smolder.
20:39 plobsing joined #parrot
20:40 whiteknight dukeleto: so the million dollar question: What can we get that's better?
20:40 cotto_work dukeleto: I'll think about what's needed, start a wiki page and we can iterate from there
20:40 whiteknight do we fix/patch smolder?
20:40 dukeleto whiteknight: what do we need?
20:40 cotto_work time for food
20:40 dukeleto whiteknight: have you read the source of smolder? The code for it is not bad, but it has an amazing amount of dependencies, and there are bugs in the dependencies
20:41 dukeleto whiteknight: like memory leaks and such
20:41 fbrito left #parrot
20:41 fbrito1 is now known as fbrito
20:41 dukeleto whiteknight: the better question is: What do we actually need?
20:41 whiteknight memory leaks are not necessarily a huge problem, on a short-lived server process
20:41 plobsing ~~
20:41 whiteknight dukeleto: I have been perfectly happy with smolder. For the most part I think it addresses the needs I have
20:41 dukeleto whiteknight: until recently, a smolder server had to be restarted daily or it would consume all memory
20:42 whiteknight other community members may be more demanding
20:42 fbrito (can someone please close http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129360922947 ? :))
20:42 dukeleto whiteknight: well then you can continue to enjoy to use it :) I don't want to get rid of it, i just don't think it serves all our needs
20:42 dukeleto fbrito: looking
20:42 whiteknight dukeleto: I'm certainly not going to argue about anything. I don't love smolder
20:42 whiteknight but I am not inclined to spend my own tuits replacing it
20:43 dukeleto fbrito: closed
20:43 dukeleto whiteknight: i hear ya
20:44 Kristaba cotto: I need the confirmation of bacek, but I think all the tests fails are due to the compilers generated code. If it is, I think I'm unable to fix the code in less than few days...
20:46 plobsing whiteknight: (re: TT #749) (1) the commit I reverted did not address this issue. (2) we in fact *do* do some validation of fpmcs before thawing that is shared from pbc thaw.
20:46 plobsing I'm not entirely sure what that ticket expects
20:47 whiteknight plobsing: okay, if you don't understand it, and I dont uderstand it enough to even know if a fix was committed to it, maybe we just close it
20:47 whiteknight plobsing: I recently did add a verification function for fpmcs used by the embedding API to verify serialized config hashes. I don't know if I duplicated effort there
20:47 plobsing I'd expect cotto to know what was expected. He did report it in the first place.
20:48 plobsing whiteknight: there is already sanity checking built in to imageiothaw
20:48 whiteknight Parrot_pf_verify_image_string
20:49 dukeleto whiteknight: did you see my comments earlier about embed docs?
20:49 plobsing the problem with having the sanity checking in the deserialization (where it belongs) is that it is difficult to handle exceptions generated in a sane fashion.
20:49 whiteknight dukeleto: no
20:49 dukeleto whiteknight: lost in the sea of irc
20:50 whiteknight msg cotto_work when you get back can you take a look at TT #749. I have some questions about what it really wants and whether it's worth doing at this point
20:50 aloha OK. I'll deliver the message.
20:50 dukeleto 09:58 <@dukeleto> whiteknight: i see you made a huge change to the embed docs
20:50 dukeleto 10:00 <@dukeleto> whiteknight: we still want to keep the old docs around while the old embed system is deprecated, for people that are still using the old embed api (like everyone)
20:50 dukeleto 10:01 <@dukeleto> whiteknight: what about docs/embed_old.pod ? And then when the old embed api is deleted from a supported release, we remove embed_old.pod ?
20:50 dukeleto 10:02 <@dukeleto> whiteknight: for instance, for people that want to hack on converted PL/Parrot to the new embed API, it is useful to compare the old docs and new
20:50 dukeleto 10:10 <@dukeleto> whiteknight: is there no way to choose runcores with the new embed API?
20:50 * dukeleto should have used a link, but was lazy
20:50 plobsing so we "need" a separate validator so that we can make decent error messages (eg: for fpmc config hash thaw in fakecutables)
20:51 plobsing whiteknight: the *real* problem is that we don't have a decent (or at least well known) mechanism for catching thrown exceptions from within C internals
20:51 dukeleto whiteknight: also, some of the work that i have done for my current TPF grant was removed in your embed.pod commit
20:52 whiteknight dukeleto: Ah, I wasn't aware that it conflicted with your grant
20:52 whiteknight I'll bring back the old file and rename it to embed_old.pod
20:52 dukeleto whiteknight: danke
20:52 whiteknight and include a note that the newer file is for the newer API
20:53 dukeleto whiteknight: awesome. I can add a mention in embed_old to the current one
20:54 dalek TT #1029 closed by whiteknight++: taillcall into PIR compiler broken
20:54 dalek TT #1029: http://trac.parrot.org/parrot/ticket/1029
20:54 dalek TT #1935 closed by whiteknight++: nqp-rx on Parrot RELEASE_2_11_0-771-g40e018d fails t/nqp/33-init.t
20:54 dalek TT #1935: http://trac.parrot.org/parrot/ticket/1935
20:54 plobsing whiteknight: I feel such separate validation functions are a workaround for a completely separate issue. They are hackish, wrong, and if I break them, I will simply remove them.
20:56 whiteknight plobsing: That function is basically just an encapsulation for preexisting logic that was in an even worse place
20:56 whiteknight so if you want to remove it and you don't think it's worthwhile, do it
20:56 whiteknight I didn't create the logic, I just moved it to a function in the correct subsystem
20:56 whiteknight it had been in src/global_setup.c, I believe
20:56 plobsing whiteknight: yes. it was added to give a "nice" error message for fakecutables linked against the wrong libparrot and close out a ticket.
20:57 plobsing the correct fix would be difficult, so this happened in stead.
20:57 whiteknight plobsing: what is the correct fix?
20:57 plobsing catch the thrown exception
20:57 whiteknight the embed API does that, in a fashion
20:57 plobsing append to the error message to give more context
20:58 moritz whiteknight: is there test coverage for http://trac.parrot.org/parrot/ticket/1935 in the parrot test suite?
20:58 whiteknight moritz: Not directly, but yes.
20:58 whiteknight I added a test for it and #1029
20:58 whiteknight so the tailcall issue there should be resolved
20:59 bacek left #parrot
20:59 plobsing I have no immediate desire to eliminate it. There are bigger fish to fry. But I have no intention of expending effort to keep it working.
21:00 moritz whiteknight: ah, the test is hidden in ef82f5ccda42b44738c2eca023693fd95d10b092
21:00 moritz well, not hidden
21:01 pmichaud does this help, at all?  http://gist.github.com/768575
21:01 moritz just at the end of teh line :-)
21:01 whiteknight moritz: yes
21:01 cotto_work ~~
21:01 whiteknight pmichaud: yes
21:02 plobsing whiteknight: (re: PackFile_new_dummy) we can get rid of this as soon as we no longer special-case the first packfile.
21:03 whiteknight plobsing: Okay, awesome. That's what I want to do
21:03 cotto_work whiteknight: what about 749?
21:04 whiteknight I want to get rid of that, and I want to remove some of the special-case crap from the embedding API. I want one function for "Add a packfile to the interpreteR"
21:04 pmichaud should I post the nqp-rx steps to parrot-dev, or ... ?
21:04 whiteknight instead of one function to load the first one, one to load subsequent ones, one to run it, etc
21:04 whiteknight pmichaud: parrot-dev
21:05 plobsing one function to load them function to load them all, one function to run them...
21:05 whiteknight cotto_work: Do we need that ticket still? with other refactors in that system the ticket may be redundant
21:06 dukeleto pmichaud: steps to test nqp-rx + parrot latest together?
21:07 cotto_work whiteknight: closed\
21:08 whiteknight cotto_work: we'll make it better en passant, I promise
21:08 cotto_work yes we will
21:09 pmichaud message sent to parrot-dev
21:10 bacek joined #parrot
21:10 whiteknight I was very happy to see opmap_aware_pmcs merged
21:10 whiteknight bacek++ times infinity for that one
21:11 dalek TT #749 closed by cotto++: [CAGE] factor Packfile_unpack into validation and unpacking functions
21:11 dalek TT #749: http://trac.parrot.org/parrot/ticket/749
21:12 whiteknight we're down to 575 tickets
21:12 cotto_work indeed
21:12 whiteknight It's possible (unlikely) that we can get down to 550 by 3.0
21:13 dukeleto pmichaud++ # concise instructions
21:13 dukeleto pmichaud: i can setup something to do that on the GCC compile farm pretty easily
21:14 dukeleto pmichaud: are there any Configure.pl flags for the parrot that is used that you prefer ?
21:14 pmichaud I tend to use --optimize
21:14 pmichaud and of course a --prefix for installing
21:15 pmichaud so, --optimize and --prefix
21:15 Kristaba bacek: ping
21:15 whiteknight msg kid51 during your spaces_in_paths work, can you take a gander at TT #196? I'm on MSVC+ActivePerl and I can't reproduce it. When you merge that branch and all my tests come out clean I'm inclined to close that ticket
21:15 aloha OK. I'll deliver the message.
21:15 dukeleto pmichaud: does nqp-rx have a place to send smoke reports ?
21:16 plobsing left #parrot
21:17 fbrito left #parrot
21:18 whiteknight we can easily add it to smolder if desired
21:18 whiteknight though nqp-rx would need to write a client to upload the reports
21:19 cotto_work one issue that the automation should mitigate is ephemeral failures in nqp-rx and other HLLs under test
21:19 cotto_work false positives are not going to be helpful
21:20 pmichaud dukeleto: no smoke reporting on nqp-rx at the moment.  we haven't need them -- test failures are pretty rare
21:20 Kapace will http://trac.parrot.org/parrot/ticket/1038 be made into a GCI task? It looks interesting
21:21 pmichaud if we want to put together some smoke reporting for nqp-rx, I be supportive of that
21:21 pmichaud afk, homework
21:21 pmichaud *I'd
21:21 cotto_work dukeleto: do you have tuits to quickly turn that into a gci task?
21:25 cotto_work Kapace: start on it if you're interested.  I'll make it a task tonight if nobody else does.  Just send a pull request when you're ready.
21:26 Kapace cotto_work: ok cool
21:28 plobsing joined #parrot
21:29 whiteknight left #parrot
21:37 bacek aloha, humans
21:37 bacek cotto, Kristaba pong
21:38 cotto_work bacek, can you change aloha to allow backticks in nicks?
21:40 dukeleto cotto_work: i have no time right now to do anything
21:40 cotto_work dukeleto: ok
21:42 dukeleto cotto_work: http://smolder.parrot.org/a​pp/projects/smoke_reports/8
21:42 dukeleto nqp-rx has a home on smolder now
21:42 dukeleto should be trivial to send reports to it
21:42 dukeleto cotto_work: feel free to make it a GCI task
21:43 gottreu left #parrot
21:44 rurban_ joined #parrot
21:44 Coke joined #parrot
21:44 dukeleto pmichaud: http://smolder.parrot.org/a​pp/projects/smoke_reports/8 <-- nqp-rx is on smolder now
21:46 rurban left #parrot
21:46 rurban_ is now known as rurban
21:47 mtk left #parrot
21:49 Kristaba bacek: For my fix task, I fixed all the test I knew fix (all tests that failed using make fulltest except in t/compilers/imcc/opt1.t and opt2.t), but now, with 'make test', I have a lot of test failings
21:51 cotto_work Kristaba: do you have a parrot/parrot commit bit or did someone else pull your commits?
21:51 Kristaba bacek: And I think they are dues to the compilers code output (so I really have no idea of how to fix them)
21:52 Kristaba cotto: Someone else pull my commits
21:52 nopaste "Kristaba" at 192.168.1.3 pasted ""make test" error output with my changes" (619 lines) at http://nopaste.snit.ch/27578
21:53 Kristaba bacek: See the nopaste if you have some time
21:56 Kristaba bacek: I think the HLL compiler need to be fixed, but I have no more idea of the cause of this issue
21:56 bacek Kristaba, pct test failures are bad. They have to fixed (either tests or PCT by it self). You can ignore IMCC failures.
21:57 bacek Kristaba, last test - just change OFFSET=>13 to 15
21:59 cotto_work Kristaba: you're being extraordinarily patient with a task that was probably too difficult for gci.
21:59 cotto_work Once you're done with this round of changes, I'd be ok considering your part complete.  Do you want me to mark it complete so you can move on?
22:00 bacek PCT failures - :main subs should have ".param args :slurpy"
22:00 bacek (It's from #1705)
22:01 Kristaba bacek: Oh, thank you!
22:01 cognominal left #parrot
22:02 Kristaba cotto: Thank you too, I'm ok for this :)
22:03 cotto_work excellent tenacity, then
22:04 aantn joined #parrot
22:04 cotto_work dukeleto: are you happy with the config log gci task?
22:08 dalek pir: ea8a091 | bacek++ | src/POST/Compiler.pm:
22:08 dalek pir: Simplify emiting of POST::Label
22:08 dalek pir: review: https://github.com/parrot/pir/commit/ea8a091e4d
22:08 dalek pir: 902650c | bacek++ | src/POST/Compiler.pm:
22:08 dalek pir: Fix emiting bytecode for method call.
22:08 dalek pir: review: https://github.com/parrot/pir/commit/902650c7ba
22:08 dalek pir: 10601f1 | bacek++ | src/POST/Compiler.pm:
22:08 dalek pir: Charset is gone.
22:08 dalek pir:
22:08 dalek pir: Handling of strings require major rehaul.
22:08 dalek pir: review: https://github.com/parrot/pir/commit/10601f144f
22:10 nwellnhof joined #parrot
22:19 aantn left #parrot
22:21 Matt221 joined #parrot
22:21 Matt221 Hi guys
22:21 sorear Hello
22:21 Matt221 Just FYI: I wont be able to finish the 'Take 100 lines off fill_params' GCI task until tomorrow (swamped with school work)
22:22 hercynium left #parrot
22:23 cotto_work Matt221: fbrito is also taking a related task.  Please coordinate with him so you're not both changing the same code.
22:24 cotto_work Matt221: never mind.  He's not on it any longer.
22:25 cognominal joined #parrot
22:25 cotto_work Matt221: I look forward to seeing what you come up with.
22:30 dukeleto cotto_work: yes, i think it is better than what we have
22:30 cotto_work dukeleto: so approve?
22:31 dukeleto cotto_work: yes
22:31 cotto_work done
22:32 dukeleto cotto_work: you making a task to add "make smoke" to nqp-rx ?
22:32 dukeleto cotto_work: the code can basically be copy-pasted from rakudo
22:32 cotto_work i wasn't planning on it
22:32 cotto_work wouldn't be too hard though
22:32 dukeleto cotto_work: writing the task will take longer than just doing it
22:32 * dukeleto has an nqp-rx commit bit
22:33 cotto_work It's getting close to the point where I don't see much value in adding tasks.
22:33 cotto_work apart from ones that we know for certain a student will work on
22:33 dukeleto cotto_work: yep
22:36 Kapace How's this looking: http://dpaste.com/295039/
22:38 cotto_work ...
22:39 cotto_work speedy
22:40 Kapace (I had been looking at that ticket from earlier :P)
22:40 cotto_work does that expose everything that the current version does?
22:40 Kapace Only has 3 exposed subroutines, so yeah, as far as I can tell
22:41 Kapace i don't think the internal functions are of much use...
22:41 cotto_work no
22:42 cotto_work make sure to add some tests and docs
22:43 cotto_work task created
22:43 Kapace ah, yes, forgot about that. I remade the example for comparison, but I'll make some tests
22:43 Kapace ah, thanks cotto_work
22:44 cotto_work sure.  if you're trolling through our tickets and find anything else before the gci deadline, let me know
22:44 Kapace cool
22:45 cotto_work do you know the parrot-specific parts of adding a file to our repo?
22:45 Kapace no
22:45 Kapace but, I'm not adding a source file, just a test maybe..
22:45 cotto_work ok'
22:45 cotto_work that's easy then
22:46 Kapace (there are no existing tests for md5sum, so I probably have to add one)
22:46 cotto_work t/library/md5.t
22:46 rfw [11:23:11] <@cotto_work> Matt221: fbrito is also taking a related task.  Please coordinate with him so you're not both changing the same code. <-- no actually this is me
22:47 rfw but i dropped it to take up a sahana bug
22:47 cotto_work rfw: ok.  I unpublished it for now.
22:47 rfw alright
22:47 rfw i'll pick it up when Matt221 finishes
22:47 cotto_work if you want to do it, let me know and coordinate with matt
22:47 cotto_work er, Matt221
22:47 rfw actually
22:48 rfw can we swap?
22:48 rfw i can do it before Matt221
22:48 rfw then he can pick up the task after i'm done
22:48 cotto_work Matt221: is that ok with you?
22:48 dukeleto tools/bisect-parrot.pl in rakudo.git looks useful
22:49 cotto_work rfw: I republished.  Go for it, but do your best to coordinate.
22:49 rfw okay
22:49 rfw but i have to hang the washing first
22:51 rfw cotto_work: claimed
22:52 nwellnhof left #parrot
22:54 cotto_work accpeted
22:55 plobsing left #parrot
22:57 dalek nqp-rx/smoke: e1f6b8b | dukeleto++ | build/Makefile.in:
22:57 dalek nqp-rx/smoke: Begin to add a smoke target to send smolder reports
22:57 dalek nqp-rx/smoke: review: https://github.com/perl6/nqp-rx/commit/e1f6b8b570
22:59 Kapace does method documentation go at the header of the file, or before each method? (or can someone point me to a properly documented source file?)
23:00 plobsing joined #parrot
23:04 mikehh make corevm/make coretest FAIL - t/pmc/packfileopmap.t - Could not find oplib `math_ops' - remaining tests pass
23:04 mikehh all other tests PASS (pre/post-config, smoke (#2098) fulltest) at 2_11_0-777-gef82f5c - Ubuntu 10.10 i386 (g++-4.5)
23:06 cotto_work Kapace: before each method
23:07 whiteknight joined #parrot
23:09 Kapace ok
23:10 plobsing left #parrot
23:17 fperrad left #parrot
23:22 Kapace btw, I had to add :main to the current md5 example for it to work
23:26 cotto_work Kapace: ok
23:26 cotto_work that's not terribly surprising
23:28 bkuhn left #parrot
23:29 Kapace cotto_work: ok. I see sha256 in Digest too, how about an object interface for that as well, for consistancy?
23:30 cotto_work sure
23:30 cotto_work claim accepted
23:31 Kapace cotto_work: cool 100th pull :) https://github.com/parrot/parrot/pull/100
23:31 cotto_work I was hoping we'd get there before the end of gci
23:32 rfw how does headerizer work?
23:34 rfw it doesn't seem to, uh, headerize, as it were
23:38 cotto_work rfw: you run make headerizer after a build
23:38 rfw oh, okay
23:38 cotto_work rfw: what are you doign
23:39 rfw cotto_work: ripping fill_params into bits
23:39 rfw since matt is doing it tomorrow i'll do it today :D
23:40 cotto_work great
23:40 cotto_work chose the function names carefully
23:40 rfw it won't headerize my new function though
23:40 cotto_work do you have a branch?
23:40 rfw i can make one for my fork
23:42 Kristaba My god, the PCT tests errors make me crazy
23:43 Kristaba I still don't understand from where the " 'anon' pc 0 (EVAL_1:1) " error can be fixed
23:50 Matt221 left #parrot
23:58 Kapace hmm, the output of a simple sha256 test is not equal to one from an online calculator..
23:59 Kapace (the online test concurs with my sha256sum tool, but not the pir program)

| Channels | #parrot index | Today | | Search | Google Search | Plain-Text | summary

Parrot | source cross referenced