Camelia, the Perl 6 bug

IRC log for #parrot, 2012-05-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:03 dalek parrot/coke/rm_pasm: a7d5324 | coke++ | / (3 files):
00:03 dalek parrot/coke/rm_pasm: remove trailing whitespace
00:04 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/a7d5324c86
00:45 lucian joined #parrot
00:49 whiteknight Coke: pbc_disassemble doesn't really generate runnable PASM, it's a way to see what bytecode is doing. I think it can stay
00:50 Coke whiteknight: ok. the docs seem to indicate it does.
00:51 whiteknight I may be confusing it with something else, but I think that what it generates cannot be round-trip compiled
00:53 Coke who is our opengl guy?
00:53 Coke *person
00:54 Coke (opengl isn't being detected on my 10.7.3 OS X box.)
01:02 dalek parrot: 55eec22 | Whiteknight++ | src/exit.c:
01:02 dalek parrot: Fix an ASSERT_ARGS ommision. Coke++ for the catch
01:02 dalek parrot: review: https://github.com/parrot/parrot/commit/55eec22332
01:03 dalek parrot/coke/rm_pasm: e3466ea | coke++ | tools/build/h2inc.pl:
01:03 dalek parrot/coke/rm_pasm: The libraries were updated, but not the program
01:03 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/e3466ea4b7
01:08 kid51 joined #parrot
01:09 Coke aloha: msg alvis - what is the purpose of docs/binaries/parrot.pod ? it seems /very/ similar to docs/running.pod - can we kill one of these files?
01:09 aloha Coke: OK. I'll deliver the message.
01:09 aloha Coke: Okay.
01:09 Coke ??
01:09 Coke aloha: msg aloha aloha?
01:09 aloha Coke: OK. I'll deliver the message.
01:09 aloha Coke: No clue. Sorry.
01:10 Coke aloha: msg alvis - what?
01:10 aloha Coke: OK. I'll deliver the message.
01:10 aloha Coke: msg alvis - what is the purpose of docs/binaries/parrot.pod ? it seems /very/ similar to docs/running.pod - can we kill one of these files
01:10 Coke aloha: msg alvis - ignore that last one, testing a bug in aloha - still want to know about the pod files, though
01:10 aloha Coke: OK. I'll deliver the message.
01:12 dalek parrot/coke/rm_pasm: 9c6005f | coke++ | src/gc/gc_private.h:
01:12 dalek parrot/coke/rm_pasm: stress.pasm was removed
01:12 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/9c6005fd94
01:39 dalek parrot/whiteknight/fix_pbc_merge: 419bd98 | Whiteknight++ | / (2 files):
01:39 dalek parrot/whiteknight/fix_pbc_merge: update pbc_merge to merge the list of tags in the constants segment
01:39 dalek parrot/whiteknight/fix_pbc_merge: review: https://github.com/parrot/parrot/commit/419bd989d9
01:39 dalek parrot/whiteknight/fix_pbc_merge: 1f7fdf7 | Whiteknight++ | / (2 files):
01:39 dalek parrot/whiteknight/fix_pbc_merge: Fix conflict, remove embed API routine that doesn't belong in master yet
01:39 dalek parrot/whiteknight/fix_pbc_merge: review: https://github.com/parrot/parrot/commit/1f7fdf7f64
01:39 dalek parrot/whiteknight/fix_pbc_merge: 6c75c8d | Whiteknight++ | / (4 files):
01:39 dalek parrot/whiteknight/fix_pbc_merge: Add new routine Parrot_pf_get_annotations_segment. Use it in IMCC to clean up some code. Use it in pbc_merge, along with some other magic, to implement merging annotations. Annotations need a lot more documentation.
01:39 dalek parrot/whiteknight/fix_pbc_merge: review: https://github.com/parrot/parrot/commit/6c75c8d4e7
01:41 dalek parrot: 419bd98 | Whiteknight++ | / (2 files):
01:41 dalek parrot: update pbc_merge to merge the list of tags in the constants segment
01:41 dalek parrot: review: https://github.com/parrot/parrot/commit/419bd989d9
01:42 dalek parrot: 1f7fdf7 | Whiteknight++ | / (2 files):
01:42 dalek parrot: Fix conflict, remove embed API routine that doesn't belong in master yet
01:42 dalek parrot: review: https://github.com/parrot/parrot/commit/1f7fdf7f64
01:42 dalek parrot: 6c75c8d | Whiteknight++ | / (4 files):
01:42 dalek parrot: Add new routine Parrot_pf_get_annotations_segment. Use it in IMCC to clean up some code. Use it in pbc_merge, along with some other magic, to implement merging annotations. Annotations need a lot more documentation.
01:42 dalek parrot: review: https://github.com/parrot/parrot/commit/6c75c8d4e7
01:42 dalek parrot/whiteknight/fix_pbc_merge: 2eaba9f | Whiteknight++ | / (3 files):
01:42 dalek parrot/whiteknight/fix_pbc_merge: Make headerizer so we can build
01:42 dalek parrot/whiteknight/fix_pbc_merge: review: https://github.com/parrot/parrot/commit/2eaba9f367
01:42 whiteknight shit, I didn't intend those commits to end up on master already
01:43 dalek parrot: a40984c | Whiteknight++ | / (3 files):
01:43 dalek parrot: Headerizer to fix build
01:43 dalek parrot: review: https://github.com/parrot/parrot/commit/a40984c452
01:44 whiteknight Whatever, the build seems fixed. I'll check out the codetest ramifications tomorrow morning
01:45 alvis Coke: It, along with the other *.pod files in 'docs/binaries', is the pod page with which to create *roff/man pages. I just haven't had a chance to write the converter yet. Hopefully, I'll get to it in the not too-distant future.
01:48 alvis And, yep, it comes right outta 'docs/running.pod'.
01:59 JimmyZ joined #parrot
02:01 travis-ci joined #parrot
02:01 travis-ci [travis-ci] parrot/parrot#296 (master - 6c75c8d : Whiteknight): The build was broken.
02:01 travis-ci [travis-ci] Change view : https://github.com/parrot/par​rot/compare/55eec22...6c75c8d
02:01 travis-ci [travis-ci] Build details : http://travis-ci.org/parrot/parrot/builds/1255925
02:01 travis-ci left #parrot
02:02 travis-ci joined #parrot
02:02 travis-ci [travis-ci] parrot/parrot#297 (master - a40984c : Whiteknight): The build was fixed.
02:02 travis-ci [travis-ci] Change view : https://github.com/parrot/par​rot/compare/6c75c8d...a40984c
02:02 travis-ci [travis-ci] Build details : http://travis-ci.org/parrot/parrot/builds/1255937
02:02 travis-ci left #parrot
02:08 alvis But, now I think about it, 'docs/binaries/' probably needs a 'README.pod' file to explain its purpose. I'm about 3/4 through with my README -> README.pod branch; I'll make sure to add one to 'docs/binaries'.
02:10 dalek parrot/coke/rm_pasm: a81c28f | jimmy++ | t/perl/testlib/hello:
02:10 dalek parrot/coke/rm_pasm: remove t/pmc/testlib/hello
02:10 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/a81c28fcc3
02:10 benabik ~~
02:19 benabik msg whiteknight The new key methods look fairly useful.  I'll investigate the introspection half in PACT's current debug.winxed sometime soon
02:19 aloha OK. I'll deliver the message.
02:20 JimmyZ any object to remove Befunge?
02:21 benabik JimmyZ: From where?
02:21 JimmyZ examples/pir/befunge
02:22 benabik Is it broken?
02:23 JimmyZ yes
02:25 JimmyZ from the CHANGES file, almost nobody take care of it
02:25 JimmyZ it's toy from 2002
02:26 benabik I'd be tempted to boot it out into it's own repo or something.  I'd suggest e-mailing parrot-dev first.  Give it a day or so before actually axing it.
02:35 JimmyZ hmm
02:35 dalek parrot/coke/rm_pasm: a85adad | jimmy++ | t/compilers/imcc/syn/file.t:
02:35 dalek parrot/coke/rm_pasm: remove pasm test from t/compilers/imcc/syn/file.t
02:35 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/a85adadf14
02:39 benabik I'm slightly hesitant about flat out removing things.  But it's not irreversible.
02:39 dalek parrot/coke/rm_pasm: ba78ca9 | jimmy++ | t/native_pbc/header.t:
02:39 dalek parrot/coke/rm_pasm: remove pasm from t/native_pbc/header.t
02:39 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/ba78ca9e3a
02:40 JimmyZ benabik: there are a already a pir one
02:40 benabik JimmyZ: Sorry, was still talking about befunge
02:40 JimmyZ oh
02:42 JimmyZ benabik: IMO, what parrot is not on focus should be removed, community don't have enough human resources
02:43 JimmyZ benabik: aye, it's not irreversible, we have tags
02:44 JimmyZ someone likes it, download it from old parrot
02:59 * benabik tries git-subtree
03:09 benabik https://github.com/Benabik/parrot/tree/befunge
03:10 JimmyZ git-subtree is great tool
03:16 JimmyZ benabik++
03:22 dalek parrot/coke/rm_pasm: 5f334c5 | jimmy++ | t/native_pbc/string.t:
03:22 dalek parrot/coke/rm_pasm: removed unuseful comment from t/native_pbc/string.t
03:22 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/5f334c5829
03:51 ttbot Parrot 6c75c8d4 MSWin32-x86-multi-thread make error http://tt.taptinder.org/cmdinfo/82432
03:54 crab2313 joined #parrot
05:02 crab2313 joined #parrot
05:57 crab2313 left #parrot
06:01 crab2313 joined #parrot
06:04 crab2313 left #parrot
07:13 fperrad joined #parrot
09:45 JimmyZ joined #parrot
10:21 fperrad joined #parrot
10:31 dalek parrot/coke/rm_pasm: 9ea50eb | jimmy++ | config/auto/signal.pm:
10:31 dalek parrot/coke/rm_pasm: change config/auto/signal.pm to output pir
10:31 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/9ea50ebae5
10:31 dalek parrot/coke/rm_pasm: b8c19b1 | jimmy++ | t/steps/auto/signal-01.t:
10:31 dalek parrot/coke/rm_pasm: fixed test in steps/auto/signal-01.t
10:31 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/b8c19b1db1
10:37 dalek Heuristic branch merge: pushed 103 commits to parrot/whiteknight/rm_pasm2 by zhuomingliang
10:47 PacoAir joined #parrot
10:57 lucian joined #parrot
11:48 crab2313 joined #parrot
11:48 crab23131 joined #parrot
11:54 whiteknight joined #parrot
11:54 kid51 joined #parrot
11:56 whiteknight good morning, #parrot
11:57 crab23131 :)
12:14 dalek rakudo/nom: 973df80 | jnthn++ | src/ (3 files):
12:14 dalek rakudo/nom: Implement temp and let. Passes all of let.t and all the temp.t tests apart from those that depend on TEMP phaser, which is still NYI.
12:14 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/973df80cfa
12:14 dalek rakudo/nom: 3e82557 | jnthn++ | t/spectest.data:
12:14 dalek rakudo/nom: Run let.t and temp.t.
12:14 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/3e82557a98
12:17 dalek rakudo/nom: e5683d3 | jnthn++ | docs/ROADMAP:
12:17 dalek rakudo/nom: Remove completed item from ROADMAP.
12:17 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/e5683d3a5f
12:17 whiteknight hello crab23131, how are you doing today?
12:25 crab23131 I'm new here.
12:26 JimmyZ_ joined #parrot
12:27 masak crab23131: welcome to #parrot.
12:28 crab23131 thanks.
12:31 crab2313 joined #parrot
13:15 dalek rakudo/nom: dd66583 | jnthn++ | docs/ChangeLog:
13:15 dalek rakudo/nom: Updated ChangeLog.
13:16 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/dd66583ba2
13:22 JimmyZ_ joined #parrot
13:24 JimmyZ__ joined #parrot
13:51 whiteknight Is there any way to tell GDB not to load our pretty-printers without having to go to blib/lib/ and delete the damn things?
13:55 JimmyZ don't use GDB that version > 7.3?
13:59 PacoAir joined #parrot
14:17 plobsing joined #parrot
14:28 dalek parrot/whiteknight/gc_finalize: 4b5d336 | jkeenan++ | src/ (2 files):
14:28 dalek parrot/whiteknight/gc_finalize: [codingstd] Add one assert macro.  Insert placeholder documentation in two locations.
14:28 dalek parrot/whiteknight/gc_finalize: review: https://github.com/parrot/parrot/commit/4b5d336a2b
14:32 schmooster joined #parrot
14:35 dalek rakudo/nom: d9fd173 | moritz++ | src/core/Exception.pm:
14:35 dalek rakudo/nom: run END blocks after printing exception
14:35 dalek rakudo/nom: review: https://github.com/rakudo/rakudo/commit/d9fd173279
14:36 kid51 joined #parrot
15:22 JimmyZ_ joined #parrot
15:25 dalek parrot: 1e1d98d | Whiteknight++ | / (2 files):
15:25 dalek parrot: Fix pbc_merge to properly merge debug segments now that we are deduplicating string constants
15:25 dalek parrot: review: https://github.com/parrot/parrot/commit/1e1d98df6d
15:25 whiteknight With that fix, I think pbc_merge FINALLY acts the way it should
15:37 dalek parrot: fd34a70 | jkeenan++ | frontend/pbc_merge/main.c:
15:37 dalek parrot: [codingstd] Fix linelength and c_parens violations.
15:37 dalek parrot: review: https://github.com/parrot/parrot/commit/fd34a70115
15:39 dalek parrot: 947772b | jkeenan++ | frontend/pbc_merge/main.c:
15:39 dalek parrot: [codingstd] Add ASSERT_ARGS in two locations.
15:39 dalek parrot: review: https://github.com/parrot/parrot/commit/947772b80d
15:48 Psyche^ joined #parrot
16:03 dalek parrot: e9e1656 | jkeenan++ | / (2 files):
16:03 dalek parrot: [codingstd] Correct POD-formatting error.  Add placeholder C function docs; one function  still failing for unknown reason.
16:03 dalek parrot: review: https://github.com/parrot/parrot/commit/e9e16567ee
16:12 Coke aloha: msg alvis - why keep 2 copies of the same pod, though? just keep one, updated, that you can use to generate whatever output.
16:12 aloha Coke: OK. I'll deliver the message.
16:15 Coke aloha: msg alvis - I have already had, in that one case, had to update text in /two/ locations. this is staleness waiting to happen.
16:15 aloha Coke: OK. I'll deliver the message.
16:17 whiteknight Coke: I suspect the one in docs/binaries/ is what we want to keep
16:18 whiteknight just for consistancy. We can merge the two docs
16:18 Coke Sure, that's fine. ;)
16:19 * Coke wants an svn output for "git pull --rebase" that shows me what files were changed as part of the pull.
16:19 Coke since it doesn't even tell me how many commits to look back.
16:21 whiteknight kid51++
16:21 Coke oh, it does give me range..
16:22 Coke aloha: jimmyz: what is the replacement for t/perl/testlib/hello ?
16:22 aloha Coke: Okay.
16:22 Coke aloha: msg jimmyz: what is the replacement for t/perl/testlib/hello ?
16:22 aloha Coke: OK. I'll deliver the message.
16:22 aloha Coke: Okay.
16:23 * Coke hopes aloha is smart about the :
16:25 dalek parrot: a6c583c | Whiteknight++ | frontend/pbc_merge/main.c:
16:25 dalek parrot: Add in some missing function-level docs. kid51++ for the catch. Also fix a little bit of formatting so the debugging statement doesn't exceed the line length requirement.
16:25 dalek parrot: review: https://github.com/parrot/parrot/commit/a6c583c301
16:27 alvis Coke: You're not wrong, and I agree with Whiteknight on which one to keep.
16:29 Coke "you're not wrong" sounds so much worse than "you're right". ;)
16:30 kid51 Aha, I was wondering what that comment was about -- debugging.  Thanks.
16:36 alvis Coke: I'm working on revisions to several of our docs in 'docs/', e.g., adding to 'faq.pod', fixing up 'compiler_faq.pod', that sort of thing. When I get it in half-decent shape, I'll post to parrot-dev for comments. I'll include your suggestion to consolidate/merge 'docs/running.pod' with 'docs/binaries/parrot.pod'.
16:40 alvis And, fwiw, I certainly don't mean, "you're not wrong," to imply less correctness. Probably just my pomposity or tendency to hedge coming to the fore. :)
16:41 alvis So ..., just to say I said it, "you're right." :)
16:52 Hunger joined #parrot
17:03 cotto ~~
17:10 dalek parrot: eb50ee8 | alvis++ | docs/index/developer.json:
17:10 dalek parrot: Correcting my earlier failure to include 'release_parrot_github_guide.pod' in the output to the auto-gen'd 'Developers Documentation' page.
17:10 dalek parrot: review: https://github.com/parrot/parrot/commit/eb50ee834e
17:10 dalek parrot: cd71fe1 | alvis++ | / (3 files):
17:10 dalek parrot: Merge branch 'master' of github.com:parrot/parrot
17:10 dalek parrot: review: https://github.com/parrot/parrot/commit/cd71fe1723
17:10 dalek parrot: 2b1029e | alvis++ | docs/project/release_parrot_github_guide.pod:
17:10 dalek parrot: Correct the title so that 'mk_html_docs.pl' can pick it up and properly display it.
17:10 dalek parrot: review: https://github.com/parrot/parrot/commit/2b1029ee25
17:10 dalek parrot: 8e8e085 | alvis++ | frontend/pbc_merge/main.c:
17:10 dalek parrot: Merge branch 'master' of github.com:parrot/parrot
17:11 dalek parrot: review: https://github.com/parrot/parrot/commit/8e8e085bca
17:12 alvis oops! I don't know about that 3d 'un. I was just working on correct some doc-related stuff.
17:13 alvis *correcting
17:14 alvis alvis--
17:17 alvis Hmm, ... Whiteknight working ... not sure what I did, but, someone, please let me know if I broke something.
17:28 Coke # 05/06/2012 - rakudo++ (21745); niecza (93.72%); pugs (41.92%)
17:28 Coke ww
17:31 dalek parrot/coke/rm_pasm: a4d447d | coke++ | .gitignore:
17:31 dalek parrot/coke/rm_pasm: don't ignore pasm files
17:31 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/a4d447d0be
17:31 dalek parrot/coke/rm_pasm: f0a7429 | coke++ | MANIFEST (2 files):
17:31 dalek parrot/coke/rm_pasm: regen MANIFEST
17:31 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/f0a7429dba
17:45 Coke so, the reason I'm getting gzip failures on os x is because stat is telling us the wrong file size.
17:45 Coke boiling down to a test case.
17:49 Coke raw stat is fine (yay), must be some odd thing in the gziphandle test file...
17:51 moritz 6 failing test files remaining in rm_pasm
17:52 Coke (oh, I thought stat was off because the file measurements depnd on the file i'm in the middel of editing. heh.)
17:53 Coke moritz: 7 for me. trying to fix the one that's also failing in trunk.
17:53 moritz Coke: which one?
17:54 moritz I'm looking at  t/perl/Parrot_Test.t and can't make any sense from its errors
17:54 Coke moritz: gziphandle
17:54 moritz Coke: ok, then we won't clash
17:54 Coke moritz: yah, I think I broke that one with an edit to lib/Parrot/Test.pm
17:55 Coke check the log in that area to see if I screwed it up. :(
17:55 moritz Coke: I think jimmyz++ broke it
17:55 nine Good evening, #parrot
17:59 Coke nine: I thought seven eight you!
17:59 dalek parrot/coke/rm_pasm: ac60116 | moritz++ | t/perl/Parrot_Test.t:
17:59 dalek parrot/coke/rm_pasm: fix Parrot_Test.t
17:59 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/ac60116515
17:59 Coke moritz++
18:15 Coke so, this test is wonky.
18:16 Coke the reason it's failing is because it's not reading in enough data from the gzip'd file. read on a gzip()'d handle should get the # of /uncompressed/ bytes - but instead of trying to read exactly that number, the test multiples that # by 2. (why?)
18:16 Coke if I multiply it by (e.g.) 3, it's enough.
18:16 Coke wonder if it's a string encoding issue, and we're dealing with raw bytes.
18:17 Coke doesn't help that the test is using raw registers.
18:17 Coke (and not helpfully named vars).
18:17 Coke I'm going to bump up the constant and add a note that it's a fudge factor. :(
18:19 dalek parrot: a858d47 | coke++ | t/dynpmc/gziphandle.t:
18:19 dalek parrot: up the fudge factor on this test to pass on OS X
18:19 dalek parrot: review: https://github.com/parrot/parrot/commit/a858d47e9a
18:21 dalek Heuristic branch merge: pushed 16 commits to parrot/coke/rm_pasm by coke
18:25 Coke parrot_test still "fails" here -   Parse errors: Bad plan.  You planned 93 tests but ran 96.
18:26 Coke ooh, packfile tests might already be fixed...
18:37 dalek parrot/coke/rm_pasm: 46d231e | coke++ | config/gen/makefiles/root.in:
18:37 dalek parrot/coke/rm_pasm: Add test prereq back.
18:37 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/46d231ed1c
18:37 Coke moritz: down to 3 failing tests..
18:37 Coke (need to rebuild makefile after that push, though.)
18:40 Coke moritz:  the constants one is probably "hard" to fix.
19:03 Coke pbc_disassemble /can/ generate raw PASM.
19:04 * Coke will try it out on master and see if it's roundtrippable.
19:11 contingencyplan joined #parrot
19:14 dalek rakudo/use-arglist: 5162c8b | jnthn++ | src/Perl6/Grammar.pm:
19:14 dalek rakudo/use-arglist: Get arglist thunk to run and return something.
19:14 dalek rakudo/use-arglist: review: https://github.com/rakudo/rakudo/commit/5162c8bde7
20:08 dalek rakudo/use-arglist: 7b98e2a | moritz++ | src/Perl6/Grammar.pm:
20:08 dalek rakudo/use-arglist: import according to arglist
20:08 dalek rakudo/use-arglist:
20:08 dalek rakudo/use-arglist: does not break anything, but does not seem to get arglists working either
20:08 dalek rakudo/use-arglist: review: https://github.com/rakudo/rakudo/commit/7b98e2aa4e
20:11 dalek parrot: cb468ba | Whiteknight++ | ChangeLog:
20:11 dalek parrot: Start a changelog entry for 4.4.0. Mention the pbc_merge fixes.
20:11 dalek parrot: review: https://github.com/parrot/parrot/commit/cb468ba016
20:18 whiteknight msg NotFound Can we get a winxed commandline option to include files, like $include... but from the commandline?
20:18 aloha OK. I'll deliver the message.
20:44 Coke whiteknight: ah, it's not roundtrippable - even with -b, pbc_disassemble doesn't fixup constants. lots of PMC_CONST(5), etc.
21:00 Khisanth joined #parrot
21:03 perlite_ joined #parrot
21:17 Coke so, given that, I think we're ok.
21:39 dalek parrot/coke/rm_pasm: 11e3941 | coke++ | / (2 files):
21:39 dalek parrot/coke/rm_pasm: Fixup pbc_disassemble don't call it PASM
21:39 dalek parrot/coke/rm_pasm:
21:39 dalek parrot/coke/rm_pasm: (it really wasn't - you couldn't roundtrip it.)
21:39 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/11e3941602
22:00 dalek parrot/coke/rm_pasm: 1559b1e | coke++ | t/perl/Parrot_Test.t:
22:00 dalek parrot/coke/rm_pasm: fix test plan
22:00 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/1559b1e585
22:13 kid51 joined #parrot
22:20 Coke moritz++ jimmyz++ # down to one failing test.
22:24 kid51 Testing that branch.
22:25 kid51 Is it the sort of branch whose test results may vary from one platform to another? Or gcc to g++?
22:38 Hunger joined #parrot
23:09 benabik Coke: The gzipfilehandle test stores the file twice, so it should be double in size.  (That one confused me for a bit too)
23:12 Coke I imagine it should be /at least/ double.
23:12 Coke kid51: which branch?
23:13 Coke benabik: if you can add a clarifying comment to replae/update my comment, that would be great.
23:14 benabik I seem to recall having problems with that test but fixing it somehow.
23:14 benabik I'll look into it.
23:18 benabik whiteknight++ # I thought I'd just have to replace pbc_merge in PACT someday
23:18 dalek parrot/coke/rm_pasm: f92c6bd | coke++ | src/pmc/imccompiler.pmc:
23:18 dalek parrot/coke/rm_pasm: remove commented out code
23:18 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/f92c6bdc19
23:18 dalek parrot/coke/rm_pasm: b8fdb94 | coke++ | / (2 files):
23:18 dalek parrot/coke/rm_pasm: Remove PASM as an option from IMCCompiler.
23:18 dalek parrot/coke/rm_pasm: review: https://github.com/parrot/parrot/commit/b8fdb94c9d
23:22 Coke whiteknight: I wonder if your patch would work if we had just my pbc_to_exe patch in b8fdb94c9d
23:22 Coke without that, I couldn't build winxed once I updated imccompiler.
23:23 Coke if the build still works and we keep at one failure, you can try to merge your changes in whiteknight/rm_pasm2 to coke/rm_pasm again.
23:25 benabik Coke: Based on your comment, the compressed file is _larger_ than it's input?
23:26 benabik Coke: On my OS X machine, it's less than a fifth the size.
23:28 kid51 Coke: your rm_pasm branch
23:29 benabik Oh, you're not hitting EOF after reading all the data.  Hm.
23:29 benabik Wonder why I'm not having any problems.
23:31 Coke benabik: no, I'm not hitting eof after reading the amount of data calculated to be the uncompressed amount of data.
23:33 Coke I nitpick only because I'm not sure 2X is accurately "all the data"
23:33 benabik It it.
23:33 benabik It reads in a file, calculates the length of that data, then saves it twice.
23:33 Coke kid51,kid51_at_dinner: I wouldn't expect it to be different, no. not much C has changed.
23:33 benabik The entire process needs comments and variable names.
23:35 benabik Coke: What's your config?  I'm having no problem on 10.7.3 w/ gcc 4.2
23:36 Coke benabik: * 2 + 1 works.
23:36 Coke * 2 does not.
23:36 Coke 10.7.3, gcc 4.2.1
23:36 Coke probably a zlib version issue.
23:37 benabik The test prints the version.  1.2.5 here
23:38 Coke 1.2.6 here.
23:38 benabik Hm.
23:39 whiteknight Coke: I can try to merge rm_pasm2 back into your branch soon. I just don't want to distract from the other work that also needs to get done there
23:39 Coke I'll drop it down to *2+1 and note the version diff.
23:39 Coke whiteknight: 'sfine.
23:39 whiteknight Once all the tests are passing again, I'll have a clear baseline to build on
23:39 benabik Coke: I'd add it just before the read.  I'll comment and name variables.
23:40 benabik (It's not a clear test, IMHO)
23:42 Coke benabik: If you have a preference, I'll let you change it, then. ;)
23:42 benabik I'm not sure why it doubles the data.
23:42 Coke benabik: https://gist.github.com/2625028
23:59 aloha (parrot/parrot) Issues closed : 400 (pbc_merge discards annotations) by Whiteknight : https://github.com/parrot/parrot/issues/400

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

Parrot | source cross referenced