Camelia, the Perl 6 bug

IRC log for #parrot, 2011-01-04

Parrot | source cross referenced

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

All times shown according to UTC.

Time Nick Message
00:00 rfw nwellnhof: i can't get the error_output_is to fully conform to the error_output tests
00:00 nwellnhof bacek: i thought more about constant PObjs, and +1 for removing constant PMCs. but i'd like to keep constant strings.
00:00 rfw because it has a realy weird way of running commands
00:00 rfw and i can't easily get the exit code from that
00:01 Kristaba left #parrot
00:02 bacek nwellnhof, no way. We can use "constant" flag as hint for GC to allocate such PObj from "very-old-generation". But contract between GC and Parrot must not have such assumption that "constant" objects are never removed.
00:05 nwellnhof bacek: yes, it should be a hint. but for strings this optimization can be implemented very cheaply. strings don't reference other objects, so we don't run into problems like with constant PMCs.
00:05 bacek left #parrot
00:10 rfw cotto_work: okay, i wrote a naughty regex to compensate for weird stdin handling
00:11 dalek TT #1028 closed by whiteknight++: can't iterate a dynlexpad
00:11 dalek TT #1028: http://trac.parrot.org/parrot/ticket/1028
00:11 dalek TT #731 closed by whiteknight++: can't tell if an OS file is executable from parrot
00:11 dalek TT #731: http://trac.parrot.org/parrot/ticket/731
00:12 cotto_work rfw: ok.  Why is that weirdness necessary?
00:12 rfw cotto_work: to be honest i have no idea
00:12 rfw mine seems to plonk what i offer from stdin there
00:12 rfw but yours disregards that
00:13 plobsing nwellnhof: I don't see how that would be an optimization. the performance win in "constant" PMCs is not having to mark them. but strings don't need to be marked anyways. The distinction between "constant" and "very-old, infrequently mark/swept" is meaningless; "constant" as a designation has a lot of baggage.
00:13 cotto_work ok.  I'll review it in a few minutes or so.
00:14 nwellnhof plobsing: it helps with string buffers.
00:14 plobsing nwellnhof: how? they don't need to be marked.
00:15 nwellnhof plobsing: string buffers are compacted.
00:16 plobsing nwellnhof: I don't follow. Are you suggesting that "constant" now also imply "pinned"
00:16 plobsing ?
00:16 nwellnhof plobsing: no, but it simply helps if you don't have to compact the buffers of constant strings.
00:17 nwellnhof maybe it's not a big win, but i think it's an easy optimization.
00:17 nwellnhof i'm all for using the constant flag only as a hint to the GC, like bacek said.
00:17 Kovensky left #parrot
00:19 nwellnhof so it means "the GC *may* keep the object around forever", but it doesn't have to.
00:19 bacek joined #parrot
00:20 plobsing so long as we don't have an API for creating mandatory memory leaks like happens now.
00:27 cotto_work rfw: I'd like to know why that regex is necessary.  It makes me a bit uneasy.
00:27 cotto_work it's not a blocker, but it strikes me as something that shouldn't be needed.
00:28 gbacon left #parrot
00:28 kid51 is now known as kid51_at_dinner
00:29 cotto_work there's also a lot of duplication between pir_stdin_output_like and _is
00:30 rfw cotto_work: yeah, i don't know either
00:31 cotto_work you should very easily be able to factor out the common code in those two functions.  The tests look fine, so do that and I'll pass.
00:32 rfw should i make a sub _pir_stdin_output_slurp?
00:32 cotto_work whatever fits in with the surrounding code's style
00:33 rfw looks like it
00:35 aantn left #parrot
00:35 rfw cotto_work: done
00:36 bacek_ joined #parrot
00:40 dalek parrot: d28f54d | rfw++ | / (3 files):
00:40 dalek parrot: Hopefully fixed debug_break test; moved pir_stdin_output_is to Parrot::Test.
00:40 dalek parrot: review: https://github.com/parrot/parrot/commit/d28f54df23
00:40 cotto_work rfw: done and thanks
00:40 dalek parrot: 279b451 | rfw++ | lib/Parrot/Test.pm:
00:40 dalek parrot: Factored out common code in Test.pm.
00:40 dalek parrot: review: https://github.com/parrot/parrot/commit/279b451634
00:40 dalek parrot: 256acfd | cotto++ | / (6 files):
00:40 dalek parrot: Merge branch 'master' of github.com:parrot/parrot
00:40 dalek parrot: review: https://github.com/parrot/parrot/commit/256acfd999
00:40 rfw cotto_work: no, thank you :D
00:41 bacek left #parrot
00:42 gbacon joined #parrot
00:50 nwellnhof i'm going to merge nwellnhof/unicode_io now if noone objects.
00:51 cotto_work nwellnhof: how does it do with Rakudo?
00:52 nwellnhof cotto: i didn't test the latest revision, but it did test it extensivley with Rakudo's spectest a while before.
00:53 cotto_work nwellnhof: ok.  That at least implies that there are no major design problems.
00:54 cotto_work thanks for the fix for the implicit :main problems
00:54 nwellnhof cotto: which one?
00:54 cotto_work both, but I was thinking of the revert
00:55 nwellnhof cotto: i think someone else made those commits.
00:55 cotto_work indeed.  it was plobsing
00:55 cotto_work plobsing++ then
00:57 chromatic I still think removing implicit :main makes sense.
00:57 cotto_work chromatic: me too but only when Rakudo and nqp-rx are ready.  I like reducing the amount of magic we rely on.
00:58 plobsing it won't be much work to remove. it was something like a 10 line patch.
00:58 plobsing and it is determined *statically*, so it doesn't exist except in the minds of IMCC users
00:59 chromatic Yeah, that's why Patrick's concern confused me.
01:00 nwellnhof src/packfile/api.c:937: warning: ‘main’ is usually a function
01:00 whiteknight plobsing: Whe you get a chance can you take a look at the NCITasklist page on the trac wiki, and let me know if it's still active/current?
01:01 dalek tracwiki: v4 | whiteknight++ | NamespaceTasklist
01:01 dalek tracwiki: +some more task ideas
01:01 dalek tracwiki: http://trac.parrot.org/parrot/wiki/Name​spaceTasklist?version=4&action=diff
01:02 plobsing silly C compiler, trying to tell me what I can and can't name my variables. I have C++ for that!
01:03 whiteknight msg Coke can you take a look at the BuildTasklist page on the wiki and let me know if it's still relevant/active/accurate? Thanks
01:03 aloha OK. I'll deliver the message.
01:04 nwellnhof testing 'make moretest' now
01:06 plobsing whiteknight: that tasklist (goallist really) was written back when I thought parrot wanted a "fat" NCI. I am now of the opinion that parrot needs a slim, bare-minimum NCI.
01:06 whiteknight plobsing: Okay, so is it worthwhile to salvage that page, or do we delete it and start over with a new tasklist?
01:08 mikehh left #parrot
01:08 plobsing whiteknight: it has the completely wrong mindset (esp. when considered in context of lorito). I've scrapped it.
01:09 whiteknight plobsing: okay. I'm doing some wiki cleanups now. You want me to delete it?
01:09 plobsing counted strings, for example, are *not* something that should be happening at the C layer.
01:09 plobsing whiteknight: note the use of the past tense. it is done.
01:09 cotto_work NCI and Lorito is tricky.  We don't currently have a good solution apart.
01:09 cotto_work from thunks, and they aren't really that great
01:10 plobsing cotto_work: as a rule of thumb though, if it *can* happen parrot-side, it should
01:10 cotto_work "parrot-side"?  Parrot has many sides.
01:11 chromatic As opposed to C-side, which is Smalltalk.
01:11 dalek parrot: 5690950 | nwellnhof++ | / (149 files):
01:11 dalek parrot: Merge branch 'master' into nwellnhof/unicode_io
01:11 dalek parrot: review: https://github.com/parrot/parrot/commit/5690950629
01:11 dalek parrot: 7cad06a | nwellnhof++ | src/io/buffer.c:
01:11 dalek parrot: Fix a compiler warning
01:11 dalek parrot: review: https://github.com/parrot/parrot/commit/7cad06a594
01:11 dalek parrot: 6f96b02 | nwellnhof++ | / (34 files):
01:11 dalek parrot: Merge branch 'nwellnhof/unicode_io'
01:11 dalek parrot: review: https://github.com/parrot/parrot/commit/6f96b02878
01:11 plobsing as opposed to inside the shim layer (native-side)
01:11 cotto_work ok
01:12 plobsing taken to the extreme, we should even get rid of automatic null-padding of strings in the shim-layer.
01:12 plobsing that *can* be done by parrot.
01:12 plobsing there's way too much stuff in NCI that is merely convenient in stead of necessary
01:16 plobsing cotto_work: thunks aren't all that bad either. if you know what function signatures you want to call in advance, which *is* possible in most cases, it is possible to provide these.
01:18 dalek tracwiki: v181 | whiteknight++ | WikiStart
01:18 dalek tracwiki: Start rearranging tasklist pages by their relevance and current state. We basically have "current", "in active planning", and "everything else".
01:18 dalek tracwiki: http://trac.parrot.org/parrot/wiki/W​ikiStart?version=181&action=diff
01:18 cotto_work plobsing: sure.  It just limits future expansion to functions for which we know signatures and means we need to carry around a big chunk of object code.
01:20 mikehh joined #parrot
01:20 plobsing cotto_work: but it is possible to create dynext libraries to provide additional thunks. these can be provided by library writters fairly easily.
01:21 plobsing cotto_work: parrot can already be configured without these thunks
01:22 cotto_work plobsing: It's workable but not ideal.
01:22 plobsing cotto_work: any dynamic approach is going to be unportable. thunks are the only solution as portable as C.
01:26 plobsing and the thunks only constitute ~4% of my libparrot.so "big chunk of object code" may be a bit of a mischaracterisation
01:43 bacek__ joined #parrot
01:44 dmalcolm left #parrot
01:48 bacek_ left #parrot
01:48 cotto_work seen notfound
01:48 aloha notfound was last seen in #parrot 5 days 1 hours ago saying "Hurrah".
01:50 kapace__ i wonder...
01:50 kapace__ seen kapace__
01:50 aloha kapace__ was last seen in #parrot 0 seconds ago saying "seen kapace__".
01:51 kapace__ in my mind, aloha got caught in infinite recursion, and exploded
01:54 cotto_work kapace__: seen kapace__
01:55 kid51_at_dinner Is NQP building once again?
01:55 kid51_at_dinner is now known as kid51
02:01 cotto_work kid51: it was building before.  Some tests look broken.
02:01 cotto_work (building before, iirc)
02:05 whiteknight msg dukeleto can you take a quick look at the GitMigration page on the wiki and see if it is worth keeping? Thanks
02:05 aloha OK. I'll deliver the message.
02:05 * cotto_work decommutes
02:06 whiteknight msg cotto Can you evaluate the L1Recap page on the wiki and see if it contains any information worth keeping, if it can be merged into the Lorito page, or if it can be deleted? Thanks
02:06 aloha OK. I'll deliver the message.
02:11 dalek parrot: e1a1d78 | jkeenan++ | lib/Parrot/Test.pm:
02:11 dalek parrot: [codingstd] Correct POD formatting error.
02:11 dalek parrot: review: https://github.com/parrot/parrot/commit/e1a1d78e4b
02:12 bluescreen left #parrot
02:14 whiteknight aloha, coverage?
02:14 aloha whiteknight: coverage is http://cv.perl6.cz or http://tapir2.ro.vutbr.cz/cover/cover-results/
02:15 bacek_ joined #parrot
02:20 bacek__ left #parrot
02:20 contingencyplan joined #parrot
02:24 dukeleto whiteknight: git migration wiki page?
02:24 dukeleto whiteknight: will see if anything is valuable
02:24 whiteknight dukeleto: it's called GitMigration
02:24 dalek tracwiki: v182 | whiteknight++ | WikiStart
02:24 dalek tracwiki: Move the smolder tasklist in with the rest of the tasklists. Other stuff
02:24 dalek tracwiki: http://trac.parrot.org/parrot/wiki/W​ikiStart?version=182&action=diff
02:25 dukeleto whiteknight: you want to delete the page? or just move it somewherE?
02:25 whiteknight dukeleto: that's what I'm asking you. If the page serves no further purpose, we can delete it
02:29 cotto ~~
02:29 * cotto is popular
02:31 dukeleto whiteknight: you can delete GitMigration, but i think GitObjections should stay around
02:32 dukeleto whiteknight: and we need a new wiki page to keep track of Git-related stuff. or perhaps there is another wiki page
02:33 whiteknight ok
02:35 cotto dukeleto, you mean one that says "go read git_workflow, then we can talk"?
02:35 cotto ;)
02:39 dalek parrot: e2e9c56 | plobsing++ | DEPRECATED.pod:
02:39 dalek parrot: add dep notice for TT #1931
02:39 dalek parrot: review: https://github.com/parrot/parrot/commit/e2e9c56e9f
02:41 dalek tracwiki: v4 | whiteknight++ | ParrotHackList
02:41 dalek tracwiki: REmoved a hack in PCC
02:41 dalek tracwiki: http://trac.parrot.org/parrot/wiki/Par​rotHackList?version=4&action=diff
02:41 dalek tracwiki: v183 | whiteknight++ | WikiStart
02:41 dalek tracwiki: http://trac.parrot.org/parrot/wiki/W​ikiStart?version=183&action=diff
02:41 dalek TT #1931 created by plobsing++: [DEPRECATED] advanced NCI parameter types
02:41 dalek TT #1931: http://trac.parrot.org/parrot/ticket/1931
02:41 plobsing whiteknight: TT #1931 represents the slimming down of NCI I envision
02:43 * dukeleto just ran "make test" for parrot on the gcc compile farm in 50 seconds
02:44 dukeleto cotto: a place to keep track of what we still need to figure out with git
02:44 dukeleto Our first smoke report from the GCC Compile Farm: http://smolder.parrot.org/app​/projects/report_details/2002
02:45 dukeleto pbc_merge tests have wonkiness. I was using TEST_JOBS=10, could be related
02:46 dalek parrot: f7a044b | Whiteknight++ | src/call/args.c:
02:46 dalek parrot: move error-message creation for named parameter/argument mismatch into a new function. This doesn't even make a dent in the huge fill_params functions, but it's a start
02:46 dalek parrot: review: https://github.com/parrot/parrot/commit/f7a044b239
02:46 dalek parrot: b863c4d | Whiteknight++ | / (38 files):
02:46 dalek parrot: Merge branch 'master' of github.com:parrot/parrot
02:46 dalek parrot: review: https://github.com/parrot/parrot/commit/b863c4dc4e
02:47 whiteknight plobsing: oh, okay
02:48 whiteknight left #parrot
02:53 nopaste "kid51" at 192.168.1.3 pasted "t/tools/pbc_merge.t: New test failure observed on Darwin/PPC." (4 lines) at http://nopaste.snit.ch/27551
02:54 dukeleto seems like pbc_merge.t fails with TEST_JOBS=1 as well
02:54 dukeleto kid51: i am seeing pbc_merge.t fail on linux as well
02:55 dukeleto pbc_merge.t failure: http://smolder.parrot.org/app​/projects/report_details/2004
02:55 plobsing funny, it doesn't fail here
02:55 plobsing but I know *exactly* why it is printing that
02:56 bacek_ left #parrot
02:56 dukeleto plobsing: i just installed a fresh parrot repo on a gcc compile farm machine. everything is fresh
02:56 plobsing dukeleto: mine also spits out those messages, but it doesn't cause a test failure. not sure why.
02:57 plobsing the messages come from pbc_merge encountering multiple packfiles which specify a :main sub. Implicit :main means that *every* packfile generated by IMCC will have a :main sub.
02:57 plobsing so whenever merging any packfile generated by IMCC, this helpful warning is printed
03:04 * kid51 must sleep
03:05 bacek_ joined #parrot
03:06 kid51 left #parrot
03:12 dalek parrot: 84a7d28 | plobsing++ | frontend/pbc_merge/main.c:
03:12 dalek parrot: silence pbc_merge warnings
03:12 dalek parrot: review: https://github.com/parrot/parrot/commit/84a7d28e01
03:12 plobsing dukeleto: 84a7d28 should resolve your test failures
03:35 gbacon left #parrot
03:35 bacek__ joined #parrot
03:40 bacek_ left #parrot
03:57 fbrito joined #parrot
04:02 rfw hey fbrito
04:02 bacek__ left #parrot
04:02 fbrito rfw: hi :D
04:03 rfw i've been taking all the parrot tasks :(
04:04 fbrito rfw: 250 points?! wow! congratulations!
04:04 fbrito 69 tasks, heheh
04:04 rfw fsagsdhshsdgh goddammit
04:06 fbrito rfw: I have to say that I am getting a bit tired of GCI :S. I am working a lot to my father and going through some things in my personal life
04:06 rfw i am too, but there's only 7 more days left!
04:06 fbrito left #parrot
04:06 rfw no, come back fbrito!
04:07 fbrito joined #parrot
04:09 fbrito rfw: yeah, true :P
04:09 rfw where did you go before :(
04:14 fbrito Pidgin crashed
04:14 fbrito first time I see that happening :D
04:15 fbrito rfw: or what do you mean by "before"? :P
04:15 rfw ah
04:15 rfw the crash
04:15 rfw i've just been messing around with javascript and stuff
04:16 dukeleto plobsing: i will try again
04:16 dukeleto fbrito: you need something to work on?
04:16 fbrito rfw: oh, you mentioned something about a JS physics engine :D
04:16 dukeleto rfw: what are you up to?
04:17 fbrito dukeleto: hm, actually I am stuck in a code coverage task
04:17 rfw dukeleto: physics, quadtrees and stuff
04:17 rfw i'm surprised how fast 1000 insertions into a javascript quadtree is
04:17 dukeleto rfw: sounds fun
04:17 dukeleto fbrito: what you stuck on/
04:17 dukeleto fbrito: ?
04:18 fbrito test get_attr_keyed on Object PMC
04:18 fbrito dukeleto: I made an example with get_attr_str: http://pastie.org/1424739 but I don't know how to use the Key in this case
04:20 bacek joined #parrot
04:24 rfw fbrito: is this for the index == -1 branch?
04:24 cottoo joined #parrot
04:24 cotto left #parrot
04:24 cottoo is now known as cotto
04:24 rfw fbrito: ah, you're not doing it correctly i don't think
04:25 rfw get_attr_keyed is a VTABLE function
04:25 rfw so you mean
04:25 rfw hold on...
04:26 rfw fbrito: http://pastie.org/1427505
04:26 rfw i think this is what you want
04:26 fbrito ahh
04:27 rfw i'm not sure about add_attribute
04:28 dukeleto fbrito: i need to go eat dinner
04:28 fbrito rfw: have you try to run that code? :D
04:28 fbrito dukeleto: ok, no problem
04:28 rfw fbrito: not yet
04:28 rfw i'm just assuming it works :3
04:29 fbrito it gives me "set_string_keyed() not implemented in class '' " on object["color"] = "blue"
04:29 rfw set_string_keyed() not implemented in class ''
04:29 rfw i lied :(
04:30 rfw hold on
04:31 fbrito default.pmc has a comment on set_attr_keyed
04:31 fbrito "Default version of keyed attribute set. Discards the key and does a set by the string name passed in."
04:31 fbrito discards the key -.-
04:31 rfw fbrito: http://pastie.org/1427511
04:32 rfw sorry abou that, i'm a terrible person
04:32 dalek parrot/nwellnhof/read_chars: 5210465 | nwellnhof++ | / (5 files):
04:32 dalek parrot/nwellnhof/read_chars: [io] Switch read to use characters
04:32 dalek parrot/nwellnhof/read_chars: review: https://github.com/parrot/parrot/commit/521046522d
04:33 rfw oh wait
04:33 rfw you want get_attr_keyed
04:33 fbrito ah, true
04:33 fbrito sorry :D
04:33 cotto Wow.  fill_params just doesn't end.
04:33 rfw no, my bad
04:34 fbrito http://tapir2.ro.vutbr.cz/cover/cov​er-results/2011-01/2011-01-04-e1a1d​78/c_cover/src-pmc-object-pmc.html
04:34 fbrito set_attr_keyed is fully tested
04:34 fbrito I mean, get_attr_keyed
04:34 fbrito I mean, set_attr_keyed
04:34 fbrito ahhaha
04:34 rfw get_attr_str is fully covered
04:34 rfw as well
04:35 fbrito I want to test line 396
04:35 rfw ah
04:35 nwellnhof left #parrot
04:35 rfw get_attr_keyed?
04:36 fbrito and line 186 and 188 (static get_attrib_index_keyed c function)
04:37 rfw huh, how _do_ you call get_attr_keed
04:37 rfw keyed*
04:37 fbrito that's my question
04:37 fbrito it has 2 params, a key and a string -.-
04:38 fbrito if I want to look for an attrib by it index position, why should I need a string?
04:39 fbrito ah, I think I understood
04:40 rfw exists_keyed_str() not implemented in class 'Integer' is what i'm getting :(
04:40 fbrito ah, nevermind -.-
04:41 fbrito I was trying something like this: http://pastie.org/1424739
04:41 rfw http://trac.parrot.org/parro​t/wiki/MakeEveryPMCAnObject
04:41 rfw i found an example of it
04:42 fbrito I have also found this page by googling for "get_attr_keyed"
04:42 rfw ah
04:42 fbrito but the example is for get_attr_str
04:42 rfw oh
04:42 fbrito I think
04:42 rfw ah yeah
04:42 rfw it is
04:42 rfw damn
04:42 dalek parrot/pasm-serializer: 7e145a8 | plobsing++ | / (2 files):
04:42 dalek parrot/pasm-serializer: add pasmserializer pmc
04:42 dalek parrot/pasm-serializer: review: https://github.com/parrot/parrot/commit/7e145a8b79
04:42 dalek parrot/pasm-serializer: e8c84ec | plobsing++ | MANIFEST:
04:42 dalek parrot/pasm-serializer: mk_manifest_and_skip
04:42 dalek parrot/pasm-serializer: review: https://github.com/parrot/parrot/commit/e8c84ec9b7
04:43 dalek parrot/pasm-serializer: 4abf3e3 | plobsing++ | / (2 files):
04:43 dalek parrot/pasm-serializer: regularize spacing slightly
04:43 dalek parrot/pasm-serializer: review: https://github.com/parrot/parrot/commit/4abf3e35e2
04:43 dalek parrot/pasm-serializer: 33acf3b | plobsing++ | / (2 files):
04:43 dalek parrot/pasm-serializer: eliminate leading space
04:43 dalek parrot/pasm-serializer: review: https://github.com/parrot/parrot/commit/33acf3b547
04:47 fbrito rfw: I really can't understand why I have to pass a key and a string as parameters
04:47 fbrito if I pass a null key, nothing happens to it
04:47 bacek fbrito, which function?
04:47 rfw i get null pmc access in get_string() when getting a weird attribute
04:47 plobsing fbrito: are you passing key objects to functions? that is magical and probably broken.
04:48 rfw aaaaaand now i'm getting segmentation fault
04:48 fbrito bacek I am trying to test get_attr_keyed on Object PMC
04:48 contingencyplan left #parrot
04:48 fbrito actually it is already tested, but there is an untested exception inside it
04:48 fbrito http://tapir2.ro.vutbr.cz/cover/cov​er-results/2011-01/2011-01-04-e1a1d​78/c_cover/src-pmc-object-pmc.html
04:49 fbrito and 2 untested lines on get_attrib_index_keyed static c function (function that get_attr_keyed relies)
04:50 rfw nol888 ripped out share_ro, didn't he
04:50 bacek fbrito, it's like C< $P0 = get_attr $P1, ["Foo";"Bar"], "Baz" >
04:51 bacek Key is "namespace", Name is localname
04:51 bacek (Yes, I know it's weird)
04:52 bacek fbrito, try to fetch some meaningless attribute
04:55 fbrito ohhh
04:56 fbrito hm, some meaningless attribute will raise "Null PMC access in get_string()"
04:57 rfw looking at the code, it might be fq_name      = VTABLE_get_string(interp, parent_class);
04:58 rfw but i tend to tell horrible lies about these things so don't listen to me
05:00 kapace__ is there a way to get the pointer of a label?
05:00 kapace__ erm, address
05:02 kapace__ nvm, got it set_addr
05:09 kapace__ Continuation's get_string returns a string of info about the current instruction etc, does it make sense to check against a hard coded string (Maybe some values will change?) or check by matching a regex?
05:23 fbrito kapace__: sorry, but I can't answer that :P
05:23 fbrito going to bed. good night guy
05:23 fbrito guys*
05:23 kapace__ lol ok fbrito
05:23 kapace__ good night :)
05:24 dalek parrot: 27c0799 | plobsing++ | DEPRECATED.pod:
05:24 dalek parrot: add deprecation notice for TT #1904
05:24 dalek parrot: review: https://github.com/parrot/parrot/commit/27c07998e0
05:24 sorear kapace__: fragile tests are ok as long as they fail not ok
05:24 * kapace__ checks out how to use nok
05:25 fbrito left #parrot
05:30 kapace__ wait i think i didn't understand that, do I use nok() somehow, or normal is()?
05:38 rurban_ joined #parrot
05:41 rurban left #parrot
05:41 rurban_ is now known as rurban
05:45 plobsing kapace__: use either. the important thing is, when creating tests likely to work incorrectly on occasion (such as regex tests), try to err on the side of false-failures in stead of false-passes.
05:45 kapace__ ok, but this test checks line numbers, and program counters?
05:46 kapace__ sounds like it could fail at any minor change in the source
05:48 sorear That's OK
05:48 plobsing hair trigger failures are OK. not great, but better than the alternative.
05:49 kapace__ ok.
05:53 bacek #1650 looks like good GCI task
05:54 dukeleto bacek: you are back!
05:54 dukeleto bacek: i hope you are feeling better
05:55 bacek dukeleto, yes. Will be released from hospital in couple of days.
05:55 dukeleto bacek: you have a laptop to keep you company?
05:55 bacek dukeleto, yes. And 3G internet :)
05:56 dukeleto bacek: lots of time to hack on stuff!
05:57 bacek dukeleto, 2 days :)
05:57 bacek #1038 looks like another candidate for GCI
05:58 bacek And I just need good candidate for "2 days hackatron" :)
06:02 dukeleto bacek: you thinking that you want to hack on something other than the gc?
06:02 bacek dukeleto, GC isn't "2 days task"
06:03 dukeleto bacek: i have a branch that is adding ipv6 support, that might be fun for you to hack on :)
06:03 bacek And I'm slightly tired of it.
06:03 dukeleto bacek: yeah, that happens
06:03 bacek Hmm. I don't have ipv6 support below my NIC. Idea rejected :)
06:04 dukeleto bacek: blarg!
06:04 bacek dukeleto, good try, bad luck :)
06:04 kapace__ tunnelbroker!
06:04 kapace__ http://tunnelbroker.net/
06:04 dukeleto bacek: what about merging some branches that haven't seen any love ?
06:05 dukeleto bacek: https://github.com/parrot/parrot/branches
06:06 dukeleto bacek: what about getting cardinal to work on parrot master?
06:07 dukeleto bacek: tadzik got it closer to working recently, but hit some exotic bugs
06:08 bacek dukeleto, interesting idea.
06:08 dukeleto bacek: also, nqp-rx is broke on parrot master now
06:08 dukeleto bacek: the encapsulate-main merge broke it
06:08 dukeleto bacek: pmichaud was not pleased
06:08 bacek dukeleto, wasn't it reverted?
06:09 dalek nqp-rx: 89a7459 | cotto++ | src/NQP/ (2 files):
06:09 dalek nqp-rx: add ...
06:09 dukeleto bacek: maybe. i have been out for the last few hours
06:09 dalek nqp-rx: review: https://github.com/perl6/nqp-rx/commit/89a7459bf1
06:09 dalek nqp-rx/nom: 98b0b4f | bacek++ | src/ (3 files):
06:09 dalek nqp-rx/nom: Remove usage of deprecated concat_s_s op
06:09 dalek nqp-rx/nom: review: https://github.com/perl6/nqp-rx/commit/98b0b4fb95
06:09 dalek nqp-rx/nom: 2a73b89 | cotto++ | src/stage0/ (2 files):
06:09 dalek nqp-rx/nom: fix deprecated concat_s_s usages
06:09 dalek nqp-rx/nom: review: https://github.com/perl6/nqp-rx/commit/2a73b898a9
06:09 dukeleto bacek: i think cardinal now compiles on parrot master, but fails all tests
06:11 bacek dukeleto, let me check
06:11 dalek nqp-rx: 2f2bc9a | cotto++ | src/NQP/ (2 files):
06:11 dalek nqp-rx: Revert "add ..."
06:11 dalek nqp-rx:
06:11 dalek nqp-rx: This reverts commit 89a7459bf15f084f034b4572e3121833f22aa6e3.
06:11 dalek nqp-rx: review: https://github.com/perl6/nqp-rx/commit/2f2bc9a9fe
06:16 dalek cardinal: f1b1628 | bacek++ | src/ (5 files):
06:16 dalek cardinal: concat_s_s is gone
06:16 dalek cardinal: review: https://github.com/parrot/​cardinal/commit/f1b16287f4
06:25 bacek dukeleto, bad luck. Looks like cardinal wasn't updated to nqp-rx.
06:35 bacek_ joined #parrot
06:40 bacek left #parrot
06:44 cotto nom sure loves globals
06:59 bacek__ joined #parrot
07:04 bacek_ left #parrot
07:41 theory left #parrot
07:47 rfw oh, completely forgot i had this os dynpmc task claimed
07:50 cotto lawl
07:50 tadzik bacek__: yes it wasn't, it's still using PGE
07:51 bacek__ is now known as bacek
07:51 bacek tadzik, yeah... It will take some time to update it to nqp-rx
07:52 tadzik bacek: making it work was my first step. Now the first thing failing that makes the tests fail, is require giving a Null PMC Access
07:53 tadzik I tracked down the line when it occurs, but failed to undersand the source of the problem
07:53 bacek tadzik, it's... May it is good idea. I'm not sure.
07:54 tadzik bacek: my second thought was "screw this, nqprx goes first", but got convinved the other way here
07:55 bacek tadzik, I usually use second approach :)
07:57 rfw does parrot have rmdir?
07:58 * rfw is trying to do naughty things with cwd
07:59 tadzik bacek: I have an unpushed commit, mind giving me a commit bit or can I nopaste it somewhere?
08:00 bacek tadzik, I'm not metacommiter.
08:03 tadzik bacek: http://wklej.org/id/449890/
08:06 rfw is it safe to test for a bad rm by calling "rm"("/")?
08:06 chromatic Not really.
08:06 rfw i can't think of any other way to do it
08:06 chromatic Imagine some fool runs the test with root privileges.
08:07 rfw rm("/") only removes empty directory
08:07 chromatic On an operating system written by fools.
08:07 rfw -y+ies
08:07 rfw (also i'm running tests at root atm, but that's because i'm very lazy)
08:07 rfw (and i don't particularly care about this vm)
08:07 chromatic As a rule, assume that anything destructive you do in your tests outside of Parrot itself could do Very Bad Things and test defensively.
08:08 rfw yeah
08:08 rfw hm
08:08 rfw what else could i do...
08:08 rfw i need to pass the S_ISDIR test, but not actually be able to remove the directory
08:08 rfw why is / so convenient for this?!
08:09 chromatic Create a directory with a file in it.
08:09 rfw oh, of course
08:09 rfw my brain isn't working quite right today
08:09 rfw thanks chromatic
08:09 chromatic You're welcome.
08:10 bacek tadzik, applied.
08:10 bacek chromatic, aloha
08:10 dalek cardinal: 992ef73 | (Tadeusz Sośnierz)++ | src/ (3 files):
08:10 dalek cardinal: Fixes under Parrot master, it now compiles and the REPL works
08:10 dalek cardinal:
08:10 dalek cardinal: Signed-off-by: Vasily Chekalkin <bacek@bacek.com>
08:10 dalek cardinal: review: https://github.com/parrot/​cardinal/commit/992ef73502
08:11 bacek tadzik, can you give me patch for parrot/CREDITS so dalek can properly attribute you for karma? :)
08:12 tadzik bacek: I'd like to :)
08:12 bacek tadzik, go for it. Format is trivial :)
08:13 tadzik bacek: my entry looks almost like yours, go and see. I don't know why doesn't it work properly
08:16 bacek hmm
08:16 bacek seen sorear
08:16 aloha sorear was last seen in #perl6 22 mins 53 seconds ago saying "then ... done".
08:16 bacek msg sorear Does dalek handle utf-8 from CREDITS properly?
08:16 aloha OK. I'll deliver the message.
08:16 tadzik looks like it does
08:16 tadzik +dalek | cardinal: 992ef73 | (Tadeusz Sośnierz)++
08:17 bacek tadzik, it's from RSS feed
08:18 bacek I'm asking about properly loading of CREDITS :)
08:18 bacek aloha, dalek?
08:18 aloha bacek: dalek is not picking up on trac wiki changes any more
08:18 bacek aloha, forget about dalek
08:18 aloha bacek: I don't know anything about about dalek.
08:18 bacek aloha, forget dalek
08:18 aloha bacek: I forgot about dalek.
08:20 tadzik I see
08:20 bacek meh.
08:20 bacek https://github.com/Infinoid/dalek-plugin​s/blob/master/modules/local/karmalog.pm
08:20 bacek line 6
08:20 bacek 7
08:21 bacek seen Infinoid
08:21 aloha Infinoid was last seen in #parrot 12 days 3 hours ago joining the channel.
08:21 bacek oookey.
08:21 cotto rfw, accepted
08:21 rfw cotto: thanks
08:21 bacek cotto, any ideas who can fix dalek a little bit?
08:24 cotto what's broken?
08:24 tadzik afk
08:24 bacek link to CREDITS file
08:24 mikehh bacek: I think sorear mostly deals with dalek, we need to consider branch merges, and stuff like that
08:24 bacek cotto,
08:24 bacek https://github.com/Infinoid/dalek-plugin​s/blob/master/modules/local/karmalog.pm, line 7
08:24 bacek mikehh, thanks for ifo
08:24 bacek info
08:25 cotto oic
08:25 cotto seen Infinoid
08:25 aloha Infinoid was last seen in #parrot 12 days 3 hours ago joining the channel.
08:25 mikehh bacek: how you doing, you said you were being released from hospital in a couple of days?
08:26 bacek mikehh, I will be.
08:26 bacek For now - relaxing in 500km out of Sydney :)
08:27 chromatic left #parrot
08:28 mikehh bacek: hopefully not to near the floods
08:29 bacek mikehh, actually, this town was flooded couple of weeks ago.
08:29 bacek Wagga Wagga
08:30 bacek http://www.theaustralian.com.au/news/n​ation/flood-threat-to-wagga-wagga-and-​coonamble-averted-but-more-heavy-rain-​expected/story-e6frg6nf-1225966425111
08:30 bacek Few week ago actually :)
08:36 dalek parrot: 6be431d | kapace++ | t/pmc/continuation.t:
08:36 dalek parrot: Added tests for Continuation up to 93%
08:36 dalek parrot: review: https://github.com/parrot/parrot/commit/6be431d3a7
08:36 dalek parrot: 32765a4 | cotto++ | t/pmc/continuation.t:
08:36 dalek parrot: Merge branch 'kapace/gci_test_continuation' of https://github.com/kapace/parrot into kapace-kapace/gci_test_continuation
08:36 dalek parrot: review: https://github.com/parrot/parrot/commit/32765a4f84
08:36 dalek parrot: f1c0156 | cotto++ | t/pmc/continuation.t:
08:36 dalek parrot: [t] make some Continuation tests less brittle
08:36 dalek parrot: review: https://github.com/parrot/parrot/commit/f1c015663e
08:37 cotto Kapace, task approved
08:37 * cotto thought bacek was making silly noises
08:42 * mikehh got to got out for a while - bbl
08:42 cotto Is it a reasonable gci task to take fill_params in src/call/args.c from 400 lines to 300?
08:49 bacek cotto, ouch. It will be scary task.
08:50 rfw http://puu.sh/EFb what the hell is happening here
08:51 rfw how is that even possible
08:51 cotto bacek, we have some scary students.
08:51 cotto I had a bad dream about rfw.
08:51 cotto ;]
08:51 bacek Give than "0-args subs parameters checking" :)
08:51 bacek #1033
08:51 * rfw gets some popcorn to listen to cotto recount of the bad dream
08:52 cotto actually it wasn't bad, or a dream, or about rfw
08:52 bacek It will consume about few days to figure out how to do it with minimal damages
08:52 rfw aw :(
08:52 cotto It might have been a sandwich.
08:53 fperrad joined #parrot
08:54 bacek actually, many-many tests are failing if I switch on "0-args subs param checking"
08:54 cotto rfw, what's confusing?
08:54 rfw cotto: how can that line not be executed
08:54 nopaste "bacek" at 192.168.1.3 pasted "enable 0-args subs param checking" (17 lines) at http://nopaste.snit.ch/27552
08:54 rfw and the next line work
08:56 dalek parrot/zeroargs_subs_checking: 0ba565b | bacek++ | compilers/imcc/pcc.c:
08:56 dalek parrot/zeroargs_subs_checking: Enable checking of arguments for 0-args subs
08:56 dalek parrot/zeroargs_subs_checking: review: https://github.com/parrot/parrot/commit/0ba565bf32
08:56 rfw also chroot doesn't want to work
08:56 bacek cotto, GCI task - make zeroargs_subs_checking to pass all tests :)
08:57 cotto bacek, is there a ticket for that
08:57 cotto ?
08:57 nopaste "bacek" at 192.168.1.3 pasted "Current test result" (56 lines) at http://nopaste.snit.ch/27553
08:57 cotto rfw, where's that code?
08:57 bacek cotto, #1033
08:57 cotto a tiny snipped isn't too helpful
08:57 rfw cotto: src/dynpmc/os.pmc
08:57 rfw on those lines
08:57 cotto bacek, wheeeeee
08:58 bacek cotto, I did quick check. Tests are wrong :)
08:58 bacek And have to be updated :)
08:59 bacek In many cases just adding ".param pmc foo" or similar
09:00 cotto that doesn't sound too bad
09:00 bacek Good GCI task from my pov
09:00 cotto difficult or medium?
09:00 bacek "medium boring"
09:01 bacek it's just reviewing many-many failing test cases.
09:01 rfw boy, these tests sure involve crazy backflips
09:02 rfw actually cotto that uncovered line is really hard to cover
09:02 rfw i can't figure out how to cover it
09:02 rfw because it involves unlinking a file you can't unlink
09:03 cotto rfw, I can see why that one is tricky
09:04 rfw there aren't a lot of things i can do
09:05 rfw i can't force ELOOP, ENOENT or ENOTDIR, since parrot checks for that
09:08 rfw and it's perhaps a little dangerous to delete system files
09:08 cotto I'd be disappointed if running make test killed my OS.
09:09 rfw okay, this is a very dirty and nasty thing to do
09:09 moritz I'd be disappointed by the OS if a 'make test' could kill it
09:09 rfw but i'm trying to delete something from /proc
09:10 rfw it covers it, but in a horrible, horrible way
09:10 rfw (and fails on win32, of course)
09:10 cotto quite
09:10 rfw do you mind if i just ignore it
09:11 cotto If there's no sane way to test it, the insane ways are still a bad idea.
09:11 rfw okay, not testing that!
09:14 rfw hm, i swear i'm covering chroot
09:14 rfw but apparently it's not covered at all
09:22 cotto 1033 task published
09:23 cotto rfw, it'd probably about 30 minutes work for you
09:23 cotto bacek++ for thinking of it
09:23 rfw cotto: heh, i think i'll let somebody else do it for a change
09:25 cotto your call
09:25 dalek parrot/zeroargs_subs_checking: 61052e5 | bacek++ | t/pmc/pmcproxy.t:
09:25 dalek parrot/zeroargs_subs_checking: Fix one of test as example.
09:25 dalek parrot/zeroargs_subs_checking: review: https://github.com/parrot/parrot/commit/61052e5a59
09:25 bacek cotto, it's 1033, 1704, 1705 combo :)
09:25 cotto it wouldn't be a bad introductory task
09:25 dalek parrot/zeroargs_subs_checking: a62931c | bacek++ | t/compilers/imcc/imcpasm/cfg.t:
09:25 dalek parrot/zeroargs_subs_checking: Mark :main sub explicitely. Part of #1704
09:26 dalek parrot/zeroargs_subs_checking: review: https://github.com/parrot/parrot/commit/a62931cd09
09:26 dalek parrot/zeroargs_subs_checking: 1c33c8f | bacek++ | t/compilers/imcc/imcpasm/opt0.t:
09:26 dalek parrot/zeroargs_subs_checking: Fix tests for handle 0-args param checking behavior.
09:26 dalek parrot/zeroargs_subs_checking: review: https://github.com/parrot/parrot/commit/1c33c8f3e4
09:26 cotto I'm going to go to bed instead of looking up the rest of those tickets.  Good night.
09:26 rfw cotto: http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129394564760 done
09:26 bacek cotto, it's not "introductory" from my pov. At least it requires some knowledge of parrot guts.
09:27 rfw parrot dissection?
09:28 cotto nice test cases
09:29 rfw yes one of them involves ripping the current directory out from underneath parrot
09:29 cotto one of the fails though, and I'm going afk
09:29 cotto now
09:29 cotto 'night
09:29 rfw night
09:29 rfw i'll look at it in the morning
09:33 Kovensky joined #parrot
09:42 rfw left #parrot
09:43 moritz rakudo on newest parrot complains:
09:43 moritz ./perl6: symbol lookup error: ./perl6: undefined symbol: Parrot_gbl_set_config_hash_internal
09:44 moritz but I can't find any references to that function in the rakudo sources
09:46 bacek_ joined #parrot
09:51 bacek left #parrot
09:54 bacek_ msg cotto #1033 is actually "difficult"
09:54 aloha OK. I'll deliver the message.
09:57 moritz cleaned everything, error went away
09:57 moritz instead I now get double output
09:57 moritz ./perl6 -e 'say rand'
09:57 moritz => two random numbers, each followed by \n
10:08 bacek_ moritz, linux/amd64?
10:08 moritz bacek_: yes
10:09 bacek_ moritz, let me try to reproduce it on linux/ia32
10:09 moritz parrot is currently... "entertaining" for HLL developers (or maintainers)
10:10 bacek_ moritz, how so? what was changed dramatically?
10:11 moritz bacek_: it's just that each time I update to a new parrot, something new has stopped working
10:11 moritz often the old errors are gone, and replaced by some new, entertaining stuff. Like double output.
10:12 bacek_ moritz, we are kind of ripping deprecated features. Deprecated long time ago actually.
10:12 moritz bacek_: and have all these features also had replacements long time ago?
10:13 contingencyplan joined #parrot
10:13 bacek_ moritz, some of them just have to be removed. Some have clean migration strategy. If not - we have to revert "deprecation" commit. Similar to #1704
10:15 moritz as I said - "entertaining"
10:17 bacek_ moritz, -Ofun in action!
10:27 bacek_ moritz, bug confirmed...
10:27 bacek_ I suspect it's #1704 related
10:28 dalek dalek-plugins: 372f8ed | infinoid++ | modules/local/karmalog.pm:
10:28 dalek dalek-plugins: Fix the link to CREDITS.
10:28 dalek dalek-plugins: review: https://github.com/Infinoid/da​lek-plugins/commit/372f8edcdf
10:30 Infinoid sorear, diakopter: I'm not sure where dalek actually runs nowadays, so I can't update its source.
11:02 bacek_ left #parrot
11:02 bacek_ joined #parrot
11:13 moritz if I think a bug is harmful enough that it should be fixed before the next release, should I set the severity to "release"?
11:14 * moritz just does it
11:18 dalek TT #1932 created by moritz++: rakudo on parrot RELEASE_2_11_0-656-g27c0799 executes main body twice
11:18 dalek TT #1932: http://trac.parrot.org/parrot/ticket/1932
11:23 Util joined #parrot
11:57 gbacon joined #parrot
12:31 bacek_ left #parrot
12:41 dalek TT #731 reopened by coke++: can't tell if an OS file is executable from parrot
12:41 dalek TT #731: http://trac.parrot.org/parrot/ticket/731
13:10 lucian_ joined #parrot
13:15 lucian left #parrot
13:24 Coke joined #parrot
13:37 * Coke ~~
13:38 rurban_ joined #parrot
13:41 rurban left #parrot
13:41 rurban_ is now known as rurban
13:44 bluescreen joined #parrot
13:46 smash joined #parrot
13:46 smash hello everyone
13:46 tadzik hello
13:56 lucian_ left #parrot
14:05 atrodo Morning!
14:34 dalek parrot: e9d9914 | Whiteknight++ | src/call/args.c:
14:34 dalek parrot: add parrot_hash_destroy calls back where I removed them. I thought they were redundant but on second look I think they are necessary. Other small tweaks
14:34 dalek parrot: review: https://github.com/parrot/parrot/commit/e9d9914824
14:53 bluescreen left #parrot
14:59 Khisanth joined #parrot
15:08 bluescreen joined #parrot
15:11 whiteknight joined #parrot
15:17 hercynium joined #parrot
15:17 dalek parrot: 9a74ad4 | mikehh++ | src/call/args.c:
15:17 dalek parrot: fix codetest failure - line length
15:17 dalek parrot: review: https://github.com/parrot/parrot/commit/9a74ad4bf5
15:36 dmalcolm joined #parrot
15:37 PerlPilot joined #parrot
15:37 PerlPilot left #parrot
15:38 PerlPilot joined #parrot
15:39 frodwith joined #parrot
15:44 Kovensky left #parrot
15:44 fbrito joined #parrot
15:47 mikehh All tests PASS (pre/post-config, make corevm/make coretest, smoke (#2015) fulltest) at 2_11_0-661-g9a74ad4 - Ubuntu 10.10  i386 (g++-4.5)
15:55 Kovensky joined #parrot
15:59 dalek Some 'mikehh' person just gave me a 151282 byte commit packet.  They need to be more careful in the future.
16:00 plobsing left #parrot
16:00 mikehh ha - in html_cleanup - git merge origin/master, git push
16:01 whiteknight nice
16:01 whiteknight mikehh: you getting close to a merge with that one?
16:05 mikehh whiteknight: still working on generating index/header pages, everything else seems ok
16:06 whiteknight ok
16:07 mikehh whiteknight: a whole bunch of stuff in lib/Parrot/docs will probably go
16:08 mikehh whiteknight: been mostly working on master, fixin' codetest failures etc.
16:11 PerlJam left #parrot
16:32 dalek parrot/html_cleanup: 275bc30 | mikehh++ | docs/index/tools.json:
16:32 dalek parrot/html_cleanup: remove deleted file - tools/docs/make_html_docs.pl
16:32 dalek parrot/html_cleanup: review: https://github.com/parrot/parrot/commit/275bc306cc
16:34 Coke mikehh: thanks for keeping that branch up to date.
16:35 Coke rant: logging in on trac.parrot.org loses your page.
16:37 mikehh Coke: I am trying to generate the index/header pages at the moment, did you have any ideas on that
16:37 mikehh Coke: I think I have an handle on it but proceeding slowly
16:40 mikehh the .joon files have the required info I think, need to push them through Parrot::Docs::HTMLPage.pm
16:41 mikehh .json
16:42 mikehh at the moment all the page references there are generated
16:43 mikehh we might want to include other stuff, but can handle that later
16:49 whiteknight Coke: I HATE THAT PROBLEM
16:50 whiteknight that's probably the single thing I dislike most about Trac
16:50 Coke what is the equivalent of "svn st --no-ignore" in git?
16:50 whiteknight what's svn st?
16:52 dalek parrot: da7b6ba | Whiteknight++ | src/string/encoding/shared.c:
16:52 dalek parrot: fix a library lookup bug on windows from the new unicode changes
16:52 dalek parrot: review: https://github.com/parrot/parrot/commit/da7b6baca8
16:52 dalek parrot: 7cc276c | Whiteknight++ | src/call/args.c:
16:52 dalek parrot: Merge branch 'master' of github.com:parrot/parrot
16:52 dalek parrot: review: https://github.com/parrot/parrot/commit/7cc276cebd
16:55 dukeleto ~~
16:55 Coke whiteknight: "status"
16:55 dukeleto Coke: git status
16:55 Coke dukeleto: what is the equivalent of "svn st --no-ignore" in git?
16:55 dukeleto Coke: answered before you asked ;)
16:55 Coke emphasis on "--no-igore"
16:55 Coke *ignore
16:55 dukeleto Coke: git status -u will also show untracked. Without -u, no untracked are shown
16:56 Coke not untracked. ignored.
16:56 Coke let's ignore svn then.
16:56 dukeleto Coke: git help status
16:57 Coke How can I get git to show me a list of untracked files whether or not they are ignored?
16:57 dukeleto Coke: git ls-files
16:58 tadzik dukeleto: I filed a GH issue about cardinal, also posted to parrot-dev
16:59 dukeleto tadzik: awesome!
17:01 plobsing joined #parrot
17:03 cogno joined #parrot
17:04 Coke looks like "make test-clean" isn't removing everything.
17:04 Coke e.g. t/compilers/imcc/syn/pod_3.pbc
17:04 theory joined #parrot
17:07 Coke for those playing along at home, "git ls-files -o" is the incantation.
17:07 Coke whiteknight: BuildTaskList is updated.
17:08 whiteknight Coke++
17:11 dalek tracwiki: v19 | coke++ | BuildTaskList
17:11 dalek tracwiki: remove stuff that's done, clean up what's left a bit.
17:11 dalek tracwiki: http://trac.parrot.org/parrot/wiki/Bui​ldTaskList?version=19&amp;action=diff
17:28 dalek tracwiki: v184 | whiteknight++ | WikiStart
17:28 dalek tracwiki: http://trac.parrot.org/parrot/wiki/W​ikiStart?version=184&amp;action=diff
17:48 M_o_C joined #parrot
17:54 chromatic joined #parrot
17:55 Kristaba joined #parrot
17:58 cogno left #parrot
18:03 lucian joined #parrot
18:04 bluescreen left #parrot
18:16 fbrito left #parrot
18:32 bluescreen joined #parrot
18:34 plobsing left #parrot
18:37 cotto_work ~~
18:40 pmichaud joined #parrot
18:41 cogno joined #parrot
18:41 cotto_work seen fbrito
18:41 aloha fbrito was last seen in #parrot 25 mins 16 seconds ago leaving the channel.
18:42 dalek parrot: aeacfec | Whiteknight++ | t/dynoplibs/io.t:
18:42 dalek parrot: fix some stat tests. On win32 we are looking for the file 'parrot.exe', not 'parrot' like on unix
18:42 dalek parrot: review: https://github.com/parrot/parrot/commit/aeacfec42b
18:42 dalek parrot: 5d54e34 | Whiteknight++ | t/dynoplibs/io.t:
18:42 dalek parrot: comment out tests that cause premature process abort on win32.
18:42 dalek parrot: review: https://github.com/parrot/parrot/commit/5d54e34e08
18:43 darbelo joined #parrot
18:44 dalek parrot: ca72cd0 | Whiteknight++ | t/dynoplibs/io.t:
18:44 dalek parrot: update test count
18:44 dalek parrot: review: https://github.com/parrot/parrot/commit/ca72cd027d
18:51 mtk left #parrot
18:51 mtk joined #parrot
18:51 dalek TT #1933 created by whiteknight++: stat opcode causes program crash on Win32 with invalid file descriptor
18:51 dalek TT #1933: http://trac.parrot.org/parrot/ticket/1933
19:05 bluescreen left #parrot
19:22 bluescreen joined #parrot
19:22 rfw joined #parrot
19:25 fbrito joined #parrot
19:27 jan left #parrot
19:30 cogno left #parrot
19:32 bacek joined #parrot
19:33 nwellnhof joined #parrot
19:33 jan joined #parrot
19:34 nwellnhof hey, whiteknight had another commit with only letters in the commit hash.
19:34 whiteknight awesome
19:35 nwellnhof i had one recently. plobsing computed the chances to 0.1%.
19:35 cotto_work he probably did it on purpose
19:36 plobsing joined #parrot
19:39 rfw morning cotto_work
19:39 rfw which test was broken again?
19:41 mtk left #parrot
19:41 cotto_work rfw: just a sec
19:42 nopaste "cotto_work" at 192.168.1.3 pasted "for rfw" (56 lines) at http://nopaste.snit.ch/27554
19:42 cotto_work the test names should also be unique when possible
19:42 cotto_work that makes it easier to figure out which test is broken quickly
19:42 rfw oh, haha
19:42 rfw i
19:43 rfw i almost saw that coming
19:45 cogno joined #parrot
19:47 rfw cotto_work: fixed
19:49 cotto_work rfw: all pass
19:49 rfw yay
19:49 cotto_work names look better too
19:49 cotto_work I'll review shortly.
19:50 rfw alright
19:55 mtk joined #parrot
19:55 cotto_work rfw: did you test on windows?
19:56 rfw erm, let me get on that now
19:56 rfw parrot is rather tempermental on windows
19:56 cotto_work indeed
19:57 cotto_work I have a windows machine I can use.
19:57 rfw let me fire up cygwin
19:58 tadzik ~~
19:59 TonyC left #parrot
19:59 nopaste left #parrot
20:02 rfw oh hey, parrot is actually compiling
20:02 cotto_work it tends to do that
20:03 rfw /usr//usrlib//libgcc//gcci686/-i686pc--pccygw​in-/cygwin4.3.4//4.3.4./../../../../../../../​.i686/-i686pc--pccygwin-/cygwinbin//binld/:ld :cannot  cannotfind  find- lparrot
20:03 rfw so much for actually compiling
20:03 rfw and now it's compiling again
20:04 rfw 0 [main] parrot 15460 exception::handle: Exception: STATUS_ACCESS_VIOLATION
20:04 rfw 764 [main] parrot 15460 open_stackdumpfile: Dumping stack trace to parrot.exe.stackdump
20:04 fbrito left #parrot
20:04 cotto_work rfw: https://gist.github.com/765320
20:05 rfw stupid non-conformist operating systems
20:05 cotto_work Yeah.  That Linux thing is really annoying.
20:05 cotto_work ;}
20:06 rfw okay i think i'll just not check for output with filesystem erros
20:06 cotto_work that strikes me as a good idea
20:06 cotto_work return code is fine
20:07 rfw the can_execute test is broken for reasons i don't know, though
20:07 plobsing left #parrot
20:07 whiteknight can_execute is broken? What platform?
20:07 rfw whiteknight: win32
20:07 rfw so i'll skip it
20:07 whiteknight oh yeah, I am fixing tha tnow
20:07 rfw but that's odd, because parrot an--oh
20:07 whiteknight or, I was before I got distracted by socket error failures
20:08 whiteknight actually, I think I already fixed the can_execute failure. https://github.com/parrot/parrot/commit/​aeacfec42bfa3317b767b2d0068adba600656542
20:09 whiteknight sockets are hellabroken on win32
20:10 rfw oh, the cwd test fails on win32 because you can't delete the directory you're executing in
20:10 whiteknight they may be broken everywhere, I think the tests are based on some bad assumptions
20:10 rfw i'll just skip that in win32
20:13 dalek parrot: 1c175e8 | Whiteknight++ | src/ (2 files):
20:13 dalek parrot: we are testing whether a socket is opened or closed by comparing to a closed value, but we never initialize. Fix that
20:13 dalek parrot: review: https://github.com/parrot/parrot/commit/1c175e8531
20:13 whiteknight on win32 I think headerizer screws up all the whitespace
20:15 nwellnhof whiteknight: aaargh, your last commit has windows newlines
20:16 nwellnhof src/io/socket_api.c |  776 +++++++++++++++++++++++----------------------
20:16 nwellnhof src/pmc/socket.pmc  |  878 ++++++++++++++++++++++++++​-------------------------
20:17 cogno left #parrot
20:19 tcurtis joined #parrot
20:19 whiteknight nwellnhof: yeah, I'm fixing that now
20:19 whiteknight I thought I had git on that platform configured to avoid that
20:19 rfw hooray, it works
20:19 rfw #          got: '. .. sillyname☺'
20:20 rfw parrot just needed to be a little happier
20:22 cotto_work #ps in 9
20:22 dalek parrot: 581131f | Whiteknight++ | / (3 files):
20:22 dalek parrot: make headerizer, and fix line-endings
20:22 dalek parrot: review: https://github.com/parrot/parrot/commit/581131fb5d
20:23 whiteknight okay, when I initialize handle values in Socket, tests start failing as I expected
20:23 whiteknight so we've been relying on broken behavior to make the tests look good for several months
20:24 rfw dear cygwin, please stop sucking so much
20:24 kid51 joined #parrot
20:26 rfw cotto_work: i can't get the io dynoplibs working because cygwin keeps STATUS_ACCESS_VIOLATIONing on me
20:26 cotto_work nice
20:27 Coke Do we have anyone building on cygwin regularly?
20:27 Coke we need to resurrect taptinder. (mj41++)
20:27 nwellnhof coke: fperrad++ i think
20:28 Coke nwellnhof: for cygwin? k.
20:28 kid51 #parrotsketch in 2
20:28 Coke perhaps someone braver than I could poke rurban about it.
20:28 perlite_ joined #parrot
20:29 kid51 Any GCI students on channel?  Feel free to join us in #parrotsketch
20:31 mikehh #ps time
20:31 dalek parrot: 7295dd1 | Whiteknight++ | / (2 files):
20:31 dalek parrot: get_bool on a Socket returns whether the Socket is open. The is_closed method returns the opposite value. Tests for creating a new socket shouldn't use the get_bool vtable, since that shows whether the socket was opened, which these tests aren't doing
20:31 dalek parrot: review: https://github.com/parrot/parrot/commit/7295dd1df9
20:32 rfw cotto_work: all fixed on cygwin
20:32 perlite left #parrot
20:32 cotto_work rfw: great
20:32 perlite_ is now known as perlite
20:32 rfw cotto_work: pushed
20:34 cotto_work rfw: still explodes after test 8 with "The symlink function is unimplemented at t/dynpmc/os.t line 266."
20:35 rfw oh, you're using mingw32 aren't you
20:35 rfw god i hate windows so much
20:36 cotto_work msvc/strawberry and mingw/strawberry
20:37 rurban why so shy
20:39 rurban Do I bite that much?
20:39 rfw cotto_work: that should do it
20:40 cotto_work rurban: ?
20:40 rurban coke didnt dare to ask me something. what?
20:40 cotto_work rfw: bad plan (33 vs 34) but otherwise good
20:41 rfw cotto_work: my plan is good
20:41 rurban I'm just smoking my perl5 compiler, I cannot smoke parrot on cygwin today
20:41 cotto_work rfw: on msvc only 33 tests are fun
20:42 cotto_work run
20:42 cotto_work odd typo
20:43 rfw aha, got it
20:44 rfw cotto_work: done
20:44 Coke rurban: you can be a little sharp, sure. I was mainly just dodging anything involving effort atm due to $DAYJOB
20:45 Coke but it sounds like things are improved already.
20:45 rurban symlink not defined in os.t is very old, I remember
20:51 whiteknight does cygwin even support symlinking?
20:51 dalek parrot: d7eaaba | rfw++ | t/dynpmc/os.t:
20:51 dalek parrot: [t] Fixing OS dynpmc tests on cygwin
20:51 dalek parrot: review: https://github.com/parrot/parrot/commit/d7eaabab13
20:51 dalek parrot: de4d3e2 | rfw++ | t/dynpmc/os.t:
20:51 dalek parrot: [t] Fixing OS dynpmc tests to work on other Windows silliness
20:51 dalek parrot: review: https://github.com/parrot/parrot/commit/de4d3e2135
20:51 dalek parrot: 9f0a8b9 | rfw++ | t/dynpmc/os.t:
20:51 dalek parrot: [t] fixing bad MSVC plan
20:51 dalek parrot: review: https://github.com/parrot/parrot/commit/9f0a8b9041
20:51 dalek parrot: e25181b | cotto++ | t/dynpmc/os.t:
20:51 dalek parrot: Merge branch 'os_dynpmc_coverage_2' of https://github.com/rofflwaffls/parrot into rofflwaffls-os_dynpmc_coverage_2
20:51 dalek parrot: review: https://github.com/parrot/parrot/commit/e25181b6f8
20:51 dalek parrot: a7c3149 | cotto++ | t/dynpmc/os.t:
20:51 dalek parrot: Merge branch 'rofflwaffls-os_dynpmc_coverage_2'
20:51 dalek parrot: review: https://github.com/parrot/parrot/commit/a7c3149c6d
20:52 whiteknight t/dynoplibs/debug.t and t/tools/pbc_disassemble.t are both failing tests on win32.
20:52 cotto_work rfw: approved
20:53 rfw thanks
20:53 whiteknight socket tests are all passing on win32, for the first time in months
20:53 dalek parrot: 7b94fae | Coke++ | src/string/encoding/shared.c:
20:53 dalek parrot: remove tabs.
20:53 dalek parrot: review: https://github.com/parrot/parrot/commit/7b94fae951
20:59 dalek parrot: c5013cf | cotto++ | t/dyn (2 files):
20:59 dalek parrot: [t] codingstd fixes in recent tests
20:59 dalek parrot: review: https://github.com/parrot/parrot/commit/c5013cfcfb
21:01 plobsing joined #parrot
21:02 bacek_ joined #parrot
21:06 bluescreen left #parrot
21:07 * Coke thinks whiteknight should run "make codetest" more often. ;)
21:07 bacek left #parrot
21:08 whiteknight It's easier on my normal dev machine, where all the text editors are configured with the right settings
21:08 cotto_work mikehh: codingstds needs some love if you have the tuits
21:08 whiteknight I've been bouncing back and forth between environments and half-configured editors
21:09 mikehh cotto_work: have a look in a minute
21:09 cotto_work mikehh: thanks
21:09 jsut joined #parrot
21:11 fperrad left #parrot
21:14 jsut_ left #parrot
21:22 TonyC joined #parrot
21:24 nopaste joined #parrot
21:26 Kristaba Someone claimed the fill_params task?
21:26 whiteknight left #parrot
21:26 cotto_work Kristaba: no.  I just unpublished it.
21:28 Kristaba cotto_work: Oh, you think it's not a good GCI task or it's for another reason?
21:28 dukeleto ~~
21:28 cotto_work It's not a good gci task.
21:29 bacek__ joined #parrot
21:30 cotto_work Changes to that code require intense scrutiny and care.
21:30 Yuki`N joined #parrot
21:30 dalek parrot: 4e20ec0 | jkeenan++ | src/io/socket_api.c:
21:30 dalek parrot: [codingstd] Needed closing paren in POD.
21:30 dalek parrot: review: https://github.com/parrot/parrot/commit/4e20ec0495
21:30 chromatic I think you're all making too much of that code.  It's super easy to break tests there.
21:31 dukeleto chromatic: good to see you around
21:31 cotto_work +1
21:31 Kristaba Yes, I understand, no problem
21:32 Kristaba (It would have been interesting and fun, but not necessarily a good thing for Parrot :p)
21:33 cotto_work Kristaba: if you want to take a shot at it, we can make it a retroactive task.  I'm still going back and forth on whether it'd actually be good for a gci student.
21:35 Yuki`N So what exactly should be printed to config.log.
21:35 Yuki`N Everythinng?
21:35 Yuki`N *everything including the header stuff?
21:35 bacek_ left #parrot
21:36 cotto_work Yuki`N: yes.  There's no reason to be sparse when spitting to a file since we can assume that users have grep or a decent text editor.
21:38 Kristaba cotto: Thank you for the proposal, if I'll want to do this I'll tell you.
21:38 rurban_ joined #parrot
21:39 cotto_work Kristaba: ok
21:39 Yuki`N I wish Perl was like Python and you could redefine the print function.
21:39 Yuki`N I'm not sure sure how I'm going to do this anymore..hm/
21:41 rurban left #parrot
21:41 rurban_ is now known as rurban
21:43 mikehh opbots, names
21:45 Andy joined #parrot
21:45 dukeleto Yuki`N: sub print { ... }
21:46 dukeleto Yuki`N: the perl 5 debugger redefines the sub named 'sub' ;)
21:46 * dukeleto has read the source of the perl 5 debugger, fixed a bug in it, and lived to tell the tale
21:46 Yuki`N dukeleto, aha.
21:47 dukeleto we are almost at pull request 100. Wow.
21:47 Yuki`N Where are the actual config steps declared?
21:48 Coke lib/Parrot/Configure/Step/List.pm
21:48 Yuki`N Where's the code that's run, though?>
21:48 Coke you said declared. :)
21:49 tcurtis left #parrot
21:49 Coke they are defined in config/<name1>/<name2>.p
21:49 Coke they are defined in config/<name1>/<name2>.pm
21:49 dukeleto bacek_at_work++ tadzik++ # cardinal hacking
21:49 dukeleto Yuki`N: the config steps are a deeeeeep rabbit hole. Have fun :)
21:49 * tadzik is excited about being on #ps :)
21:50 Yuki`N Oh dear.
21:50 Coke eh. it's not super crazy.
21:50 * dukeleto realizes he has missed #ps
21:50 Coke ask and we'll answer.
21:50 * dukeleto backlogs
21:52 Yuki`N I think redefining print will work for my needs, :P
21:53 Patterner left #parrot
21:53 Psyche^ joined #parrot
21:53 Psyche^ is now known as Patterner
21:54 Yuki`N Although, there's no way for me to see whether or not verbosity is set...
21:55 kid51 left #parrot
21:55 darbelo left #parrot
21:56 Yuki`N Oh wait, there's this cool filter package.
21:58 Yuki`N Oh, it's in CPAN, hmm.
21:59 M_o_C left #parrot
22:01 dukeleto pmichaud: i am working on the "test HLLs" issue (backlogging from #ps today)
22:03 dukeleto pmichaud: i plan on using rakudo's "make test" and nqp-rx's "make test" on parrot master as the first tangible goals
22:04 Coke any chance we can revert the recent :main changes (or whatever introduced the borkage?)
22:04 Coke and then only re-apply them once the testing bears it out?
22:04 dalek parrot: df64b9a | mikehh++ | src/io/socket_api.c:
22:04 dalek parrot: fix codetest failure - pod syntax and c_func_docs
22:04 dalek parrot: review: https://github.com/parrot/parrot/commit/df64b9a50d
22:04 dukeleto pmichaud: i could have this done in a few hours if i had uninterrupted time to work on it. So it may take a few days
22:06 dukeleto do y'all want me to revert the encapsulate-main merge ?
22:07 dukeleto someone told me it was reverted, but I am not seeing it that way
22:07 dukeleto the encapsulate-main merge was cbaad3757d3d5b5c21f5a334d4372801e5cde879
22:09 * dukeleto hears crickets
22:11 dalek parrot: d69e236 | mikehh++ | t/dynpmc/os.t:
22:11 dalek parrot: fix perlcritic failure, replace two argument open(s) with three argument open
22:11 dalek parrot: review: https://github.com/parrot/parrot/commit/d69e236007
22:12 Coke I thought someone anti-cherry-picked something post merge.
22:12 Coke (but that the mergeback as a whole was untouched)
22:13 Yuki`N Do you want --verbose=2 going to config.log?
22:15 Coke I would imagine any amount of verbosity would just go to the log.
22:16 Coke but as was mentioned on the ticket, I have little imagination on this ticket.
22:17 dukeleto Yuki`N: yes, as much info as possible, for now
22:18 dukeleto Yuki`N: we can ratched down the verbosity later if we want
22:18 dukeleto Coke: i don't see any anti-cherry-picking about encapsulate-main
22:18 dukeleto cotto_work: encapsulate-main . Does it need to be reverted ?
22:18 Yuki`N I'm still not quite sure how I can pull this off.
22:19 cotto_work dukeleto: it was, partially
22:19 Yuki`N There's really no way as it holds to mark some prints verbose and some prints normal.
22:19 dukeleto cotto_work: in what commit?
22:19 bacek_ joined #parrot
22:19 plobsing I didn't revert it to get implicit-:main back. I augmented it.
22:19 dukeleto cotto_work: i don't see evidence of the partial revert, but i could be blind
22:19 * plobsing searches for commit
22:19 Coke "it was, partially" - that's what I said. er, meant.
22:20 plobsing fdb6fa57549b91ab6024d046909fff7eedfdb75d
22:21 Yuki`N Just the way verbose printing is handled in config makes this impossible without rewriting every print line. :/
22:22 dukeleto Yuki`N: don't worry about it. Just make a config.log that useful in some way :)
22:23 Yuki`N I'd probably end up re-doing the print system.
22:24 bacek__ left #parrot
22:25 dukeleto Yuki`N: don't go down that road
22:25 dukeleto Yuki`N: just generate the simplest-yet-useful config.log for this task
22:25 Yuki`N The current architecture makes it impossible.
22:25 dukeleto Yuki`N: we want small useful iterations that can be improved in the future
22:25 Matt221 joined #parrot
22:26 dukeleto Yuki`N: why can't you just duplicate all output from Configure.pl and put it into config.log ?
22:26 dukeleto Yuki`N: create a dup of STDOUT/STDERR and redirect to config.log . Done.
22:26 Yuki`N Well that would be the same as running `perl Configure.pl | tee config.log`
22:27 cotto_work why is that bad?
22:27 Yuki`N Wasn't the point to have verbose info feed to config.log, while terse info goes to console unless --verbose is specified?
22:27 dukeleto Yuki`N: basically. but we don't want users to have to redirect the output themselves. We want it to be "automagical"
22:27 dukeleto Yuki`N: that would be nice, but if that is hard, let's just do the simple option first
22:28 * dukeleto migrates back home from the coffee shop
22:30 Yuki`N Oh, ok.
22:30 Yuki`N That's actually easy then.
22:30 dukeleto yay, easy!
22:30 * dukeleto actually goes
22:30 Yuki`N There's a CPAN module that does that...hmm.
22:30 Yuki`N I really want to use it, but >dependencies.
22:30 dukeleto Yuki`N: be careful about new dependencies
22:30 dukeleto Yuki`N: we can include CPAN modules in the parrot repo, but we need to think carefully about that
22:30 Yuki`N The concept is simple, the source is 3kb w/ documentation.
22:30 * dukeleto actually actually goes
22:31 Yuki`N I will discuss later then.
22:35 simon_ left #parrot
22:37 Kristaba cotto: Finally, I claimed the zero-arity task, and I have already a question...
22:38 Kristaba cotto: There are a lot of test fail, as expected, but really I don't understand why the imcpasm tests fail that way
22:38 nwellnhof left #parrot
22:38 Kristaba cotto: What am I supposed to do with that? Stupidly copy the many "get_params" into the expected output?
22:39 cotto_work Kristaba: building now so I can play along at home
22:42 cotto_work ok. which test now?
22:42 Matt221 left #parrot
22:43 Matt221 joined #parrot
22:43 Kristaba t/compilers/imcc/imcpasm/opt1.t for example
22:45 dalek TT #1934 created by mikehh++: Some Python programs added to tools/dev
22:45 dalek TT #1934: http://trac.parrot.org/parrot/ticket/1934
22:46 davidfetter joined #parrot
22:47 cotto_work bacek_: ping
22:48 bacek_ cotto_work, pong
22:49 cotto_work bacek_: Kristaba has some questions about zero-arity test fixes
22:49 bacek_ ah, did you see my msg about "difficult task"? :)
22:50 bacek_ Kristaba, opt1/opt2 in imcc?
22:50 cotto_work yes.  I changed the difficulty
22:50 bacek_ Kristaba, it's about testing optimizer in IMCC. Which looks like broken.
22:51 cotto_work urgh
22:51 Kristaba bacek_: Yes, indeed :(
22:51 bacek_ Kristaba, ignore them for now. There are many other tests which can and should be fixed.
22:52 Kristaba bacek_: Ok, no problem
22:52 bacek_ Kristaba, mostly related to #1705 and #1033
22:53 cotto_work bacek_: should those failing tests just be TODO'd?
22:53 cotto_work or skipped
22:53 bacek_ cotto, not yet.
22:53 bacek_ Best way - implement PIRATE and strip IMCC. Or at least strip it down to bare minimum for bootstrapping.
22:53 pmichaud dukeleto: (from backlog)   TT #1704 has the history of where we stand.  (more)
22:54 pmichaud the branch merge was commit cbaad37
22:54 plobsing left #parrot
22:54 pmichaud in order to "fix" the problems introduced by the merge, plobsing++ re-added the implicit :main capability in fdb6fa5
22:55 pmichaud that appears to have gotten rid of the "No main sub found" problem, but now we have difficulties with mainline code being executed twice
22:56 pmichaud it only appears to affect p6/nqp programs that define new classes
22:56 pmichaud since p6 always declares/defines new classes, that's just about all of them :-)
22:56 pmichaud nqp's tests only have a few tests that declare new classes -- those are the tests in which we end up with double-execution of main
22:57 pmichaud the thing that is significant about classes defined in nqp is that they're established by :load/:init subs, if that makes a difference.
22:57 pmichaud (whereas other nqp programs that don't declare :load/:init subs don't appear to have the double-main execution)
22:57 pmichaud I'm not sure if they're related, but there appears to be a correlation there.
22:57 pmichaud (end)
22:58 Matt221 left #parrot
22:58 pmichaud as far as whether we need to completely revert the branch merge, or patch things to work going forward... I don't know the pain points involved with either so I think that's really up to others to decide.
23:00 pmichaud also, thinking about it a bit more.... given that the tests that failed in nqp are the ones that declare new classes, I suspect the problem isn't really the lack of implicit :main (although that will likely cause problems for rakudo modules and standalone-compiled code)
23:00 pmichaud actually, disregard that last statement
23:00 pmichaud since nqp-rx was seeing the "No main sub found" errors in its tests, that still points to a problem with :main somewhere
23:07 cotto_work bacek_: what's the best thing for Kristaba to do about those tests?
23:08 bacek_ left #parrot
23:09 hercynium left #parrot
23:09 cotto_work Kristaba: ignore them for now.  I won't let them block your task completion if we can't give you a good answer wrt what to do with them.
23:10 mikehh All tests PASS (pre/post-config, make corevm/make coretest, smoke (#2023) fulltest) at 2_11_0-683-gd69e236 - Ubuntu 10.10 i386 (gcc-4.5)
23:12 lucian left #parrot
23:14 Kristaba cotto: Okay, no problem I'll try to fix the other test for now
23:16 dmalcolm left #parrot
23:20 bacek_ joined #parrot
23:27 Kristaba bacek_: Sorry to bother you again, but I think there is an other "problem"
23:27 bacek_ Kristaba, which one?
23:27 Kristaba The test at at t/compilers/imcc/syn/pcc.t line 525 is "calling a sub with way too many args", and as its name suggests, it try to call a subroutine (with 0 parameters) with 2000 arguments, and it expects to succeed oO
23:29 Kristaba I think it's the exact opposite of the goal of this branch, isn't it?
23:29 bacek_ Kristaba, yes. Test is correct. It test absence of segfault.
23:30 bacek_ It should say "Too many args" (or something similar)
23:30 bacek_ hang on
23:30 bacek_ I probably on wrong branch
23:31 bacek_ No, still correct answer. Test check for segfaults
23:32 bacek_ But should be rewritten to properly test it.
23:32 Kristaba bacek_: But now, when a function with 0 parameter is called with arguments, it returns an error :/
23:33 bacek_ Kristaba, yes. "should be rewritten" :)
23:33 cotto_work I remember that test.
23:34 Kristaba Ok, so I ignore it too?
23:34 Kristaba Or I rewrite it?
23:34 bacek_ E.g. push_eh bang; foo(_ARGS_); pop_eh; bang: say "didn't segfault"
23:34 cotto_work Kristaba: just check that it doesn't segfault.
23:34 bacek_ Or something like this
23:34 bacek_ Kristaba, nope, fix the test :)
23:35 Kristaba Okay :p
23:42 bacek_ cotto, ping
23:43 cotto_work bacek_: pong
23:44 bacek_ cotto, speaking of opmap_aware_pmc. What is relation between PackfileOpMap and OpLib?
23:44 bacek_ (In your design)
23:45 cotto_work bacek_: do you understand how the dynop mapping code works?
23:45 bacek_ barely
23:45 bacek_ Looks like PFOM "has many" OpLib. Correct?
23:45 cotto_work let me refresh myself
23:46 bacek_ And for each Op we have tuple (oplib, index) as mapping
23:46 cotto_work yes and yes
23:47 plobsing joined #parrot
23:47 cotto_work each set of op maps has a single OpLib
23:48 bacek_ hmmm
23:48 bacek_ Ok. Let me dig little bit deeper
23:49 cotto_work alright
23:59 gbacon left #parrot

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

Parrot | source cross referenced