Parrot 4.3.0 "In Which..." | parrot.org | Log: irclog.perlgeek.de/parrot | #parrotsketch meeting Tuesday 19:30 UTC
Set by moderator on 25 April 2012.
dalek rrot/coke/rm_pasm: a7d5324 | coke++ | / (3 files):
remove trailing whitespace
 00:03
00:45 lucian joined
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:49
Coke whiteknight: ok. the docs seem to indicate it does. 00:50
whiteknight I may be confusing it with something else, but I think that what it generates cannot be round-trip compiled 00:51
Coke who is our opengl guy? 00:53
*person
(opengl isn't being detected on my 10.7.3 OS X box.) 00:54
dalek rrot: 55eec22 | Whiteknight++ | src/exit.c:
Fix an ASSERT_ARGS ommision. Coke++ for the catch
 01:02
rrot/coke/rm_pasm: e3466ea | coke++ | tools/build/h2inc.pl:
The libraries were updated, but not the program
 01:03
01:08 kid51 joined
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.
Coke: Okay.
Coke ??
aloha: msg aloha aloha?
aloha Coke: OK. I'll deliver the message.
Coke: No clue. Sorry.
Coke aloha: msg alvis - what? 01:10
aloha Coke: OK. I'll deliver the message.
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
Coke aloha: msg alvis - ignore that last one, testing a bug in aloha - still want to know about the pod files, though
aloha Coke: OK. I'll deliver the message.
dalek rrot/coke/rm_pasm: 9c6005f | coke++ | src/gc/gc_private.h:
stress.pasm was removed
 01:12
rrot/whiteknight/fix_pbc_merge: 419bd98 | Whiteknight++ | / (2 files):
update pbc_merge to merge the list of tags in the constants segment
 01:39
rrot/whiteknight/fix_pbc_merge: 1f7fdf7 | Whiteknight++ | / (2 files):
Fix conflict, remove embed API routine that doesn't belong in master yet
rrot/whiteknight/fix_pbc_merge: 6c75c8d | Whiteknight++ | / (4 files):
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.
rrot: 419bd98 | Whiteknight++ | / (2 files):
update pbc_merge to merge the list of tags in the constants segment
 01:41
rrot: 1f7fdf7 | Whiteknight++ | / (2 files):
Fix conflict, remove embed API routine that doesn't belong in master yet
 01:42
rrot: 6c75c8d | Whiteknight++ | / (4 files):
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.
rrot/whiteknight/fix_pbc_merge: 2eaba9f | Whiteknight++ | / (3 files):
Make headerizer so we can build
whiteknight shit, I didn't intend those commits to end up on master already
dalek rrot: a40984c | Whiteknight++ | / (3 files):
Headerizer to fix build
 01:43
whiteknight Whatever, the build seems fixed. I'll check out the codetest ramifications tomorrow morning 01:44
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:45
And, yep, it comes right outta 'docs/running.pod'. 01:48
01:59 JimmyZ joined 02:01 travis-ci joined
travis-ci [travis-ci] parrot/parrot#296 (master - 6c75c8d : Whiteknight): The build was broken. 02:01
[travis-ci] Change view : github.com/parrot/parrot/compare/5......6c75c8d
[travis-ci] Build details : travis-ci.org/parrot/parrot/builds/1255925
02:01 travis-ci left 02:02 travis-ci joined
travis-ci [travis-ci] parrot/parrot#297 (master - a40984c : Whiteknight): The build was fixed. 02:02
[travis-ci] Change view : github.com/parrot/parrot/compare/6......a40984c
[travis-ci] Build details : travis-ci.org/parrot/parrot/builds/1255937
02:02 travis-ci left
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:08
dalek rrot/coke/rm_pasm: a81c28f | jimmy++ | t/perl/testlib/hello:
remove t/pmc/testlib/hello
 02:10
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.
JimmyZ any object to remove Befunge? 02:20
benabik JimmyZ: From where? 02:21
JimmyZ examples/pir/befunge
benabik Is it broken? 02:22
JimmyZ yes 02:23
from the CHANGES file, almost nobody take care of it 02:25
it's toy from 2002
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:26
JimmyZ hmm 02:35
dalek rrot/coke/rm_pasm: a85adad | jimmy++ | t/compilers/imcc/syn/file.t:
remove pasm test from t/compilers/imcc/syn/file.t
benabik I'm slightly hesitant about flat out removing things. But it's not irreversible. 02:39
dalek rrot/coke/rm_pasm: ba78ca9 | jimmy++ | t/native_pbc/header.t:
remove pasm from t/native_pbc/header.t
JimmyZ benabik: there are a already a pir one 02:40
benabik JimmyZ: Sorry, was still talking about befunge
JimmyZ oh
benabik: IMO, what parrot is not on focus should be removed, community don't have enough human resources 02:42
benabik: aye, it's not irreversible, we have tags 02:43
someone likes it, download it from old parrot 02:44
benabik tries git-subtree 02:59
github.com/Benabik/parrot/tree/befunge 03:09
JimmyZ git-subtree is great tool 03:10
benabik++ 03:16
dalek rrot/coke/rm_pasm: 5f334c5 | jimmy++ | t/native_pbc/string.t:
removed unuseful comment from t/native_pbc/string.t
 03:22
ttbot Parrot 6c75c8d4 MSWin32-x86-multi-thread make error tt.taptinder.org/cmdinfo/82432 03:51
03:54 crab2313 joined 05:02 crab2313 joined 05:57 crab2313 left 06:01 crab2313 joined 06:04 crab2313 left 07:13 fperrad joined 09:45 JimmyZ joined 10:21 fperrad joined
dalek rrot/coke/rm_pasm: 9ea50eb | jimmy++ | config/auto/signal.pm:
change config/auto/signal.pm to output pir
 10:31
rrot/coke/rm_pasm: b8c19b1 | jimmy++ | t/steps/auto/signal-01.t:
fixed test in steps/auto/signal-01.t
Heuristic branch merge: pushed 103 commits to parrot/whiteknight/rm_pasm2 by zhuomingliang 10:37
10:47 PacoAir joined 10:57 lucian joined 11:48 crab2313 joined, crab23131 joined 11:54 whiteknight joined, kid51 joined
whiteknight good morning, #parrot 11:56
crab23131 :) 11:57
dalek kudo/nom: 973df80 | jnthn++ | src/ (3 files):
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
kudo/nom: 3e82557 | jnthn++ | t/spectest.data:
Run let.t and temp.t.
kudo/nom: e5683d3 | jnthn++ | docs/ROADMAP:
Remove completed item from ROADMAP.
 12:17
whiteknight hello crab23131, how are you doing today?
crab23131 I'm new here. 12:25
12:26 JimmyZ_ joined
masak crab23131: welcome to #parrot. 12:27
crab23131 thanks. 12:28
12:31 crab2313 joined
dalek kudo/nom: dd66583 | jnthn++ | docs/ChangeLog:
Updated ChangeLog.
 13:15
13:22 JimmyZ_ joined 13:24 JimmyZ__ joined
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:51
JimmyZ don't use GDB that version > 7.3? 13:55
13:59 PacoAir joined 14:17 plobsing joined
dalek rrot/whiteknight/gc_finalize: 4b5d336 | jkeenan++ | src/ (2 files):
[codingstd] Add one assert macro. Insert placeholder documentation in two locations.
 14:28
14:32 schmooster joined
dalek kudo/nom: d9fd173 | moritz++ | src/core/Exception.pm:
run END blocks after printing exception
 14:35
14:36 kid51 joined 15:22 JimmyZ_ joined
dalek rrot: 1e1d98d | Whiteknight++ | / (2 files):
Fix pbc_merge to properly merge debug segments now that we are deduplicating string constants
 15:25
whiteknight With that fix, I think pbc_merge FINALLY acts the way it should
dalek rrot: fd34a70 | jkeenan++ | frontend/pbc_merge/main.c:
[codingstd] Fix linelength and c_parens violations.
 15:37
rrot: 947772b | jkeenan++ | frontend/pbc_merge/main.c:
[codingstd] Add ASSERT_ARGS in two locations.
 15:39
15:48 Psyche^ joined
dalek rrot: e9e1656 | jkeenan++ | / (2 files):
[codingstd] Correct POD-formatting error. Add placeholder C function docs; one function still failing for unknown reason.
 16:03
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.
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.
whiteknight Coke: I suspect the one in docs/binaries/ is what we want to keep 16:17
just for consistancy. We can merge the two docs 16:18
Coke Sure, that's fine. ;)
Coke wants an svn output for "git pull --rebase" that shows me what files were changed as part of the pull. 16:19
since it doesn't even tell me how many commits to look back.
whiteknight kid51++ 16:21
Coke oh, it does give me range..
aloha: jimmyz: what is the replacement for t/perl/testlib/hello ? 16:22
aloha Coke: Okay.
Coke aloha: msg jimmyz: what is the replacement for t/perl/testlib/hello ?
aloha Coke: OK. I'll deliver the message.
Coke: Okay.
Coke hopes aloha is smart about the : 16:23
dalek rrot: a6c583c | Whiteknight++ | frontend/pbc_merge/main.c:
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
alvis Coke: You're not wrong, and I agree with Whiteknight on which one to keep. 16:27
Coke "you're not wrong" sounds so much worse than "you're right". ;) 16:29
kid51 Aha, I was wondering what that comment was about -- debugging. Thanks. 16:30
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:36
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:40
So ..., just to say I said it, "you're right." :) 16:41
16:52 Hunger joined
cotto ~~ 17:03
dalek rrot: eb50ee8 | alvis++ | docs/index/developer.json:
Correcting my earlier failure to include 'release_parrot_github_guide.pod' in the output to the auto-gen'd 'Developers Documentation' page.
 17:10
rrot: cd71fe1 | alvis++ | / (3 files):
Merge branch 'master' of github.com:parrot/parrot
rrot: 2b1029e | alvis++ | docs/project/release_parrot_github_guide.pod:
Correct the title so that 'mk_html_docs.pl' can pick it up and properly display it.
rrot: 8e8e085 | alvis++ | frontend/pbc_merge/main.c:
Merge branch 'master' of github.com:parrot/parrot
alvis oops! I don't know about that 3d 'un. I was just working on correct some doc-related stuff. 17:12
*correcting 17:13
alvis-- 17:14
Hmm, ... Whiteknight working ... not sure what I did, but, someone, please let me know if I broke something. 17:17
Coke # 05/06/2012 - rakudo++ (21745); niecza (93.72%); pugs (41.92%) 17:28
ww
dalek rrot/coke/rm_pasm: a4d447d | coke++ | .gitignore:
don't ignore pasm files
 17:31
rrot/coke/rm_pasm: f0a7429 | coke++ | MANIFEST (2 files):
regen MANIFEST
Coke so, the reason I'm getting gzip failures on os x is because stat is telling us the wrong file size. 17:45
boiling down to a test case.
raw stat is fine (yay), must be some odd thing in the gziphandle test file... 17:49
moritz 6 failing test files remaining in rm_pasm 17:51
Coke (oh, I thought stat was off because the file measurements depnd on the file i'm in the middel of editing. heh.) 17:52
moritz: 7 for me. trying to fix the one that's also failing in trunk. 17:53
moritz Coke: which one?
I'm looking at t/perl/Parrot_Test.t and can't make any sense from its errors 17:54
Coke moritz: gziphandle
moritz Coke: ok, then we won't clash
Coke moritz: yah, I think I broke that one with an edit to lib/Parrot/Test.pm
check the log in that area to see if I screwed it up. :( 17:55
moritz Coke: I think jimmyz++ broke it
nine Good evening, #parrot
Coke nine: I thought seven eight you! 17:59
dalek rrot/coke/rm_pasm: ac60116 | moritz++ | t/perl/Parrot_Test.t:
fix Parrot_Test.t
Coke moritz++
so, this test is wonky. 18:15
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
if I multiply it by (e.g.) 3, it's enough.
wonder if it's a string encoding issue, and we're dealing with raw bytes.
doesn't help that the test is using raw registers. 18:17
(and not helpfully named vars).
I'm going to bump up the constant and add a note that it's a fudge factor. :(
dalek rrot: a858d47 | coke++ | t/dynpmc/gziphandle.t:
up the fudge factor on this test to pass on OS X
 18:19
Heuristic branch merge: pushed 16 commits to parrot/coke/rm_pasm by coke 18:21
Coke parrot_test still "fails" here - Parse errors: Bad plan. You planned 93 tests but ran 96. 18:25
ooh, packfile tests might already be fixed... 18:26
dalek rrot/coke/rm_pasm: 46d231e | coke++ | config/gen/makefiles/root.in:
Add test prereq back.
 18:37
Coke moritz: down to 3 failing tests..
(need to rebuild makefile after that push, though.)
moritz: the constants one is probably "hard" to fix. 18:40
pbc_disassemble /can/ generate raw PASM. 19:03
Coke will try it out on master and see if it's roundtrippable. 19:04
19:11 contingencyplan joined
dalek kudo/use-arglist: 5162c8b | jnthn++ | src/Perl6/Grammar.pm:
Get arglist thunk to run and return something.
 19:14
kudo/use-arglist: 7b98e2a | moritz++ | src/Perl6/Grammar.pm:
import according to arglist

does not break anything, but does not seem to get arglists working either		 20:08
rrot: cb468ba | Whiteknight++ | ChangeLog:
Start a changelog entry for 4.4.0. Mention the pbc_merge fixes.
 20:11
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.
Coke whiteknight: ah, it's not roundtrippable - even with -b, pbc_disassemble doesn't fixup constants. lots of PMC_CONST(5), etc. 20:44
21:00 Khisanth joined 21:03 perlite_ joined
Coke so, given that, I think we're ok. 21:17
dalek rrot/coke/rm_pasm: 11e3941 | coke++ | / (2 files):
Fixup pbc_disassemble don't call it PASM

  (it really wasn't - you couldn't roundtrip it.)		 21:39
rrot/coke/rm_pasm: 1559b1e | coke++ | t/perl/Parrot_Test.t:
fix test plan
 22:00
22:13 kid51 joined
Coke moritz++ jimmyz++ # down to one failing test. 22:20
kid51 Testing that branch. 22:24
Is it the sort of branch whose test results may vary from one platform to another? Or gcc to g++? 22:25
22:38 Hunger joined
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:09
Coke I imagine it should be /at least/ double. 23:12
kid51: which branch?
benabik: if you can add a clarifying comment to replae/update my comment, that would be great. 23:13
benabik I seem to recall having problems with that test but fixing it somehow. 23:14
I'll look into it.
whiteknight++ # I thought I'd just have to replace pbc_merge in PACT someday 23:18
dalek rrot/coke/rm_pasm: f92c6bd | coke++ | src/pmc/imccompiler.pmc:
remove commented out code
rrot/coke/rm_pasm: b8fdb94 | coke++ | / (2 files):
Remove PASM as an option from IMCCompiler.
Coke whiteknight: I wonder if your patch would work if we had just my pbc_to_exe patch in b8fdb94c9d 23:22
without that, I couldn't build winxed once I updated imccompiler.
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:23
benabik Coke: Based on your comment, the compressed file is _larger_ than it's input? 23:25
Coke: On my OS X machine, it's less than a fifth the size. 23:26
kid51 Coke: your rm_pasm branch 23:28
benabik Oh, you're not hitting EOF after reading all the data. Hm. 23:29
Wonder why I'm not having any problems.
Coke benabik: no, I'm not hitting eof after reading the amount of data calculated to be the uncompressed amount of data. 23:31
I nitpick only because I'm not sure 2X is accurately "all the data" 23:33
benabik It it.
It reads in a file, calculates the length of that data, then saves it twice.
Coke kid51,kid51_at_dinner: I wouldn't expect it to be different, no. not much C has changed.
benabik The entire process needs comments and variable names.
Coke: What's your config? I'm having no problem on 10.7.3 w/ gcc 4.2 23:35
Coke benabik: * 2 + 1 works. 23:36
* 2 does not.
10.7.3, gcc 4.2.1
probably a zlib version issue.
benabik The test prints the version. 1.2.5 here 23:37
Coke 1.2.6 here. 23:38
benabik Hm.
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.
whiteknight: 'sfine.
whiteknight Once all the tests are passing again, I'll have a clear baseline to build on
benabik Coke: I'd add it just before the read. I'll comment and name variables.
(It's not a clear test, IMHO) 23:40
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.
Coke benabik: gist.github.com/2625028
aloha (parrot/parrot) Issues closed : 400 (pbc_merge discards annotations) by Whiteknight : github.com/parrot/parrot/issues/400 23:59

Powered by Cro and the Raku® Programming Language.

Please report any issues / comments / feature requests as an issue on App::Raku::Log.

Thank you!