Camelia, the Perl 6 bug

IRC log for #parrot, 2010-12-28

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 nwellnhof joined #parrot
00:01 nwellnhof Matt: you should be able to generate coverage reports with the changes in branch nwellnhof/make_cover.
00:01 Matt_ nwellnhof: Thanks!
00:04 dalek parrot/encapsulate-main: 2b0abde | plobsing++ | runtime/parrot/library/PGE/Perl6Grammar.pir:
00:04 dalek parrot/encapsulate-main: mark PGE main subs :main
00:04 dalek parrot/encapsulate-main: review: https://github.com/parrot/parrot/commit/2b0abde560
00:04 dalek parrot/encapsulate-main: 5739ce1 | plobsing++ | lib/Parrot/Test/PGE.pm:
00:04 dalek parrot/encapsulate-main: mark pge testing PIR with :main
00:04 dalek parrot/encapsulate-main: review: https://github.com/parrot/parrot/commit/5739ce1fc8
00:05 Matt_ whiteknight: What about functions like Parrot_api_load_language ?
00:06 Matt_ I think coverage will already increase +10% after commenting out Parrot_api_set_stdhandles and adding tests for get/set_compiler but I want to do a few more
00:08 nwellnhof i'm about to merge nwellnhof/string_unescape now...
00:11 rfw whiteknight: did it finish yet?
00:15 Coke during a bisect, should git st be saying I'm not on a branch?
00:16 dalek parrot: 2ceb0e8 | nwellnhof++ | / (19 files):
00:16 dalek parrot: Merge branch 'master' into nwellnhof/string_unescape
00:16 dalek parrot:
00:16 dalek parrot: Conflicts:
00:16 dalek parrot: DEPRECATED.pod
00:16 dalek parrot: review: https://github.com/parrot/parrot/commit/2ceb0e8bae
00:16 dalek parrot: b8b3c7d | nwellnhof++ | / (16 files):
00:16 dalek parrot: Merge branch 'nwellnhof/string_unescape'
00:16 dalek parrot: review: https://github.com/parrot/parrot/commit/b8b3c7db92
00:17 whiteknight rfw: I'm on a resource-constrained VM for my testing. It moves *very* slowly
00:17 rfw oh
00:17 rfw heh
00:17 rfw i make -j5 in my vm
00:20 dalek TT #1809 closed by nwellnhof++: Remaining string_* functions are deprecated
00:20 dalek TT #1809: http://trac.parrot.org/parrot/ticket/1809
00:24 Matt_ whiteknight: Is there any good way of testing Parrot_api_add_library_search_path ?
00:25 whiteknight good question
00:29 dukeleto tadzik: i was away. did you make progress on that issue?
00:30 rfw whiteknight: in theory my misc.t test should cover the remainder of misc.c shouldn't it
00:30 tadzik dukeleto: nah
00:30 tadzik that seems just weird to me, I don't know where to look
00:31 dukeleto tadzik: what is the current problem? i lost it in the backscroll
00:31 whiteknight rfw, okay it's done. looking at results now
00:32 whiteknight ...no results
00:32 whiteknight wtf
00:32 tadzik dukeleto:
00:33 whiteknight what's the name of the file it's supposed to generate? coverage.html?
00:33 tadzik http://wklej.org/id/445749/
00:33 tadzik repl works, running from file doesn't
00:36 dalek TT #1628 closed by nwellnhof++: Unescape parrot string to parrot string
00:36 dalek TT #1628: http://trac.parrot.org/parrot/ticket/1628
00:41 whiteknight I won't make any more coverage-related GCI tasks if it's going to be such a hassle
00:41 dalek parrot: 20ddb40 | mikehh++ | examples/pir/life.pir:
00:41 dalek parrot: replace concat_s_s/concat_s_sc opcode and tabs
00:41 dalek parrot: review: https://github.com/parrot/parrot/commit/20ddb40e60
00:41 dalek parrot: ee1e105 | mikehh++ | examples/pir/readline.pir:
00:41 dalek parrot: replace concat_s_s opcode and tabs
00:41 dalek parrot: review: https://github.com/parrot/parrot/commit/ee1e10562d
00:42 rfw whiteknight: should be
00:43 rfw i ended up hacking around with lcov and genhtml because make cover did silly things
00:43 Yuki`N Some coverage tasks are easier than others, honestly.
00:43 Yuki`N The hardest part is really setting up the environment for `make cover' to run.
00:44 whiteknight yeah
00:44 Yuki`N I didn't have any trouble personally, but then again I also just installed packages from my distro's repos.
00:44 rfw Yuki`N: can you make cover current master's HEAD
00:44 Yuki`N Um, sure.
00:44 Yuki`N Let me boot up the vm.
00:44 rfw and check if any coverage reports are being generated for src/pmc
00:44 Yuki`N If you don't get any, in my experience.
00:44 nwellnhof rfw: i fixed a a lot of problems in my make_cover branch.
00:44 rfw nwellnhof: ah
00:44 Yuki`N You need to check to make sure that all tests are passing
00:45 Yuki`N And a make distclean sometimes helps.
00:45 whiteknight nwellnhof: I CAN HAZ MERGE NAO PLZ?
00:45 whiteknight PLZKTHXBYE
00:45 rfw 4 points: reimplement parrot in lolcode
00:46 whiteknight IM IN UR BYTECODEZ INTERPRETIN UR BYTES
00:46 tadzik there was a PAST-to-Lolcode compiler once
00:47 whiteknight wtf, does make quickcover cause a complete reconfigure/rebuild?
00:47 whiteknight I need some goddamned alcohol. gcov is angrying up my blood
00:47 Yuki`N There's already a HLL for lolcode on parrot, is there not?
00:47 nwellnhof whiteknight: it has to add some coverage options to gcc.
00:48 nwellnhof ok, i merge make_cover then...
00:48 rfw let's merge nwellnhof/make_cover into my fork
00:49 whiteknight rfw: once he merges, just do a pull on yours
00:49 Yuki`N Do you still need coverage information generated?
00:49 rfw whiteknight: too late \o/
00:50 * rfw watches parrot compile
00:50 whiteknight :)
00:50 Yuki`N Hmm, make distclean doesn't clean out t/src/embed/
00:50 Yuki`N Let me fix.
00:50 whiteknight Yuki`N++
00:50 dalek parrot: c11e91b | nwellnhof++ | config/ (3 files):
00:50 dalek parrot: Merge branch 'nwellnhof/make_cover'
00:50 dalek parrot: review: https://github.com/parrot/parrot/commit/c11e91bcaf
00:51 Tene Yuki`N: Yes, there is lolcode on parrot, but it has not been maintained for quite a while.  I originally wrote it.
00:51 rfw CAN HAZ Tene
00:52 Tene rfw: No, you can not haz tene.  There's only one tene, and I'm currently using it.
00:52 rfw :<
00:52 tadzik it works,$key
00:52 rfw okay now i'm getting 50% coverage
00:52 rfw wtf
00:52 tadzik bah, suprise paste
00:53 Tene rfw: I'll be done with my tene around midnight tonight.  I won't need it again until 9:00 AM, so feel free to stop by and borrow it, if you like.
00:53 rfw Tene: so what is a Tene anyway
00:53 Tene rfw: Me.
00:55 whiteknight blah. make cover fails here. It looks like all the file paths it is looking up are wrong
00:55 whiteknight can't find file src/pmc/src/pmc/continuation.pmc
00:55 whiteknight (and so on for all source files)
00:56 Kapace_ if you need it to work, just get a ubuntu 9.10 vm :P
00:56 whiteknight Kapace_: I'm on Ubuntu 10.10
00:56 Kapace_ same
00:56 rfw Tene: so how do i borrow your you
00:57 nwellnhof whiteknight: are you testing with my changes?
00:57 whiteknight nwellnhof: I'm pulling your changes now
00:58 nwellnhof it should fix the src/pmc/src/pmc/... problem. this was caused by changes in Devel::Cover.
00:58 Tene rfw: I was being silly, and suggesting that I didn't need myself while I was sleeping, and some other nonsense.
00:58 rfw well now you just ruined the joke!
00:59 Tene rfw: I've been very busy lately, but if you want help with lolcode on Parrot, feel free to harass me about it this weekend.
00:59 rfw hah i'm still _very_ new to the wrold of parrot
00:59 dukeleto Tene.new.clone.fork
00:59 whiteknight rfw: We're all new to it
00:59 Tene Every time I've made an offer of help to people here in the past several months, I haven't actually followed through, though, so I wouldn't be very hopeful.
00:59 dukeleto rfw: you are doing quite well for such a newbie ;)
00:59 rfw dukeleto: thanks
01:00 dukeleto rfw: you are rocking GCI, good luck!
01:00 rfw dukeleto: *almost*
01:00 fbrito left #parrot
01:01 rfw okay, coverage is completely ignoring my tests
01:01 Kapace_ rfw: hey, when you got to 100 points, did you unlock any cheat codes or something?
01:01 rfw Kapace_: nope
01:01 rfw i tried up up down down left right left right b a though
01:01 Kapace_ darn..
01:01 Kapace_ lol
01:03 rfw FINALLY
01:03 rfw YES
01:03 rfw I HAVE NO IDEA WHAT I DID BUT IT WORKED
01:05 Yuki`N https://github.com/parrot/parrot/pull/50
01:05 Yuki`N Those are my modifications.
01:05 dalek parrot: 3cb17c1 | mikehh++ | examples/streams/Combiner.pir:
01:05 dalek parrot: replace concat_s_sc opcode
01:05 dalek parrot: review: https://github.com/parrot/parrot/commit/3cb17c1009
01:05 dalek parrot: 128eb76 | mikehh++ | examples/streams/Filter.pir:
01:05 dalek parrot: replace concat_s_sc opcode
01:05 dalek parrot: review: https://github.com/parrot/parrot/commit/128eb763e5
01:05 Yuki`N With the changes that nwellnhof made it looks like they won't merge very well.
01:06 Yuki`N Removing QUICKCOVER_DIRS also affects the clean targets and it doesn't look like you changed those.
01:06 Yuki`N I'll see if I can perhaps merge and then fix that.
01:07 dukeleto rfw: nice contra reference :)
01:07 whiteknight nwellnhof: with your changes I get the same problem
01:07 whiteknight it can't find any of the source files
01:07 rfw dukeleto: sadly i suck at it
01:08 whiteknight no source src/pmc/src/pmc/bigint.pmc, etc
01:08 nwellnhof whiteknight: have you tried make realclean?
01:08 dukeleto rfw: that reminds me of my youth, playing atari and NES games
01:08 whiteknight nwellnhof: I thought so. Let me retry
01:09 nwellnhof make sure that there are no old .gcov files around.
01:09 rfw dukeleto: i enjoy playing retro games myself
01:10 Yuki`N whiteknight, take a look at my pull request. I have some clean-related makefile changes.
01:10 rfw okay my code coverage just went from 81.5% to 31%...
01:10 Yuki`N loooooooool.
01:10 whiteknight Yuki`N: okay, I'll do it in a little bit
01:10 rfw i swear parrot's cover build system is broken
01:11 Yuki`N It's a bit of black magic. :P
01:11 Yuki`N I think the real problem is the tremendous amount of output hides any problems that may exist.
01:11 Yuki`N i.e. failed todo tests print out in their entirety.
01:12 dalek parrot: d6c54c5 | nwellnhof++ | config/gen/makefiles/root.in:
01:12 dalek parrot: Remove reference to QUICKCOVER_DIRS
01:12 dalek parrot: review: https://github.com/parrot/parrot/commit/d6c54c50e3
01:12 mikehh 4 more failures to fix in t/examples/catchall.t, but I will do that later
01:12 Yuki`N nwellnhof, aww you stole my commit.
01:12 * mikehh needs sleep
01:12 Yuki`N nwellnhof, perhaps you can take care of my pull request then, it basically did that + fixed the fact that t/src/embed wasn't getting cleaned.
01:14 Yuki`N rfw, if you give me a copy of the test you wrote I can try it myself.
01:14 Yuki`N Is it on github?
01:14 nwellnhof Yuki`N: the only remaining commit is https://github.com/nol888/parrot/commit/​265e75d20af701e74b94c5a6ea41e64f234a4fdd ?
01:14 rfw Yuki`N: hold on my making -j5 is hanging everything else
01:15 Yuki`N nwellnhof, yes.
01:15 rfw and make doesn't want to die
01:15 Yuki`N Lol I only gave my VM 2 cpu's.
01:16 Yuki`N The i7 is a beast for virtualization.
01:16 rfw i gave mine 4 but i set processor affinities :\
01:16 rfw too bad it doesn't seem to care
01:16 Yuki`N Lol.
01:17 rfw oh tomoyo is the new selinux
01:17 Yuki`N Really?
01:17 rfw seems like it
01:18 rfw developed by ntt (lol japan)
01:18 Yuki`N Well Tomoyo is pretty tsundere.
01:20 rfw okay my vm is conclusively broken
01:20 dukeleto http://tomoyo.sourceforge.jp/\
01:20 dukeleto blarg! http://tomoyo.sourceforge.jp/
01:21 Yuki`N nwellnhof, you broke my make cover. :(
01:22 Yuki`N I have 5 files.
01:23 Yuki`N and lots of gcov files like /.#src#pmc#unmanagedstruct.pmc.gcov
01:23 Yuki`N I'm even more confused now.
01:23 nwellnhof left #parrot
01:24 Yuki`N Hmmm.
01:24 Yuki`N Why is my make cover not working.
01:24 rfw argh
01:24 rfw 38.9% coverage
01:24 rfw why the hell...
01:27 rfw oh, i think my parallelized make is breaking it
01:29 Yuki`N I give up.
01:29 Yuki`N Something with make cover is now broken again.
01:29 Yuki`N Seems to be a gcc version barrier or something.
01:36 nwellnhof joined #parrot
01:38 rfw whiteknight: YES I WIN
01:38 rfw File:src/misc.c
01:38 rfw Coverage:100.0%
01:39 nwellnhof lost my internet connection...
01:39 rfw nwellnhof: make cover doesn't play nice with -jx
01:39 rfw er
01:39 rfw -jX*
01:40 rfw hell, it tries to run tests before configure
01:41 rfw dukeleto, whiteknight or whoever's there: http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129341582405
01:41 nwellnhof rfw: i'll look into that.
01:41 Yuki`N nwellnhof, your fixes break my make cover.
01:42 Yuki`N I get a lot of ".#src#...gcno" files in my root
01:42 Yuki`N And only 5 files appear in the coverage report.
01:42 Yuki`N I rebased without your commit and trying make cover just in case.
01:43 nwellnhof Yuki`N: are there any error messages at the end of make quickcover?
01:43 Yuki`N Nope.
01:45 nwellnhof Yuki`N: are you running make -j?
01:45 Yuki`N nope.
01:48 nwellnhof Yuki`N: what's the output of make quickcover after running the tests?
01:50 kid51 joined #parrot
02:01 dalek parrot: 12e7238 | nwellnhof++ | config/gen/makefiles/root.in:
02:01 dalek parrot: Fix make cover dependencies
02:01 dalek parrot: review: https://github.com/parrot/parrot/commit/12e72382f0
02:02 * dukeleto has weighed in on the great gci-discuss jumpyshoes controversy
02:05 Yuki`N nwellnhof, it's normal
02:05 Yuki`N dukeleto, can you add me another task for increasing src/embed/pmc.c?
02:05 nopaste "kid51" at 192.168.1.3 pasted "make quickcover: broken at d6c54c50e335" (2899 lines) at http://nopaste.snit.ch/27513
02:05 Yuki`N I got it up to 95%, it was about 83% prior.
02:05 Yuki`N The relevant test additions are already in master.
02:05 rfw dukeleto: could you approve this: http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129341582405
02:06 Yuki`N nwellnhof, kid51 has the exact same problem that I have apparently.
02:06 Yuki`N After rebasing out your commit, cover and quickcover work again, so it's definitely something that your commit broke.
02:07 rfw Yuki`N: i wonder why it works here :|
02:07 Yuki`N rfw, what's your gcc version?
02:08 rfw gcc (GCC) 4.5.1 20101125 (prerelease)
02:08 rfw what's yours
02:08 Yuki`N 4.5.1 as well.
02:08 rfw odd
02:08 rfw what's your gcov version
02:08 Yuki`N gcov is part of gcc.
02:09 rfw oh wait
02:09 rfw duh
02:09 rfw yeah
02:09 rfw i'm silly
02:09 Yuki`N You can do make cover now? Weren't you using some hacked-together system before?
02:09 whiteknight dukeleto: what was the jumpyshoes controversy?
02:10 rfw whiteknight: https://groups.google.com/d/topic/​gci-discuss/uPktT0t-gqU/discussion (yes i posted it)
02:11 rfw whiteknight: also, unrelated: http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129341582405
02:12 whiteknight I've looked at jumpyshoes before in the standings. It is *weird* how he is racking up so many points like that
02:12 whiteknight nwellnhof: I made realclean and make quickcover again and this time it works. I must not have realcleaned before as I thought
02:16 whiteknight rfw: I don't see any comments from dukeleto in there
02:16 rfw whiteknight: it has to be reviewed first
02:16 kennym left #parrot
02:16 whiteknight oh, okay
02:16 rfw could you look at my task :D
02:16 rfw (and help me defeat the evil forces of jumpyshoes)
02:17 whiteknight rfw: I can't pull tonight, but I will approve the task
02:18 rfw ah okay
02:18 whiteknight I'll pull it tomorrow
02:18 rfw it should work (provided make cover isn't breaking)
02:18 nwellnhof left #parrot
02:18 whiteknight I trust you. Your work has been high-quality so far
02:18 whiteknight but now I have to go to bed. Goodnight
02:18 rfw heh, thanks
02:18 Yuki`N Night.
02:18 rfw night
02:19 Yuki`N Hmm, waiting on some dfBSD mentors.
02:19 whiteknight left #parrot
02:19 kid51 'make quickcover' is still broken for me at commit 12e72382f0f (in much the same way as previous paste)
02:22 nwellnhof joined #parrot
02:23 AndChat| joined #parrot
02:23 nopaste "kid51" at 192.168.1.3 pasted "make quickover still broken at 12e723" (5612 lines) at http://nopaste.snit.ch/27514
02:23 dalek parrot: 401fcfb | nwellnhof++ | config/gen/makefiles/root.in:
02:23 dalek parrot: Remove gcov -p option
02:23 dalek parrot:
02:23 dalek parrot: This should fix make cover for older versions of Devel::Cover. But
02:23 dalek parrot: some gcov files will now be overwritten.
02:23 dalek parrot: review: https://github.com/parrot/parrot/commit/401fcfb541
02:23 nwellnhof kid51: can you try 401fcfb541?
02:23 kid51 Among other problems with 'make quickcover', I'm getting files named like this in my top-level dir:  compilers#imcc#debug.c.gcov
02:24 kid51 nwellnhof: Are you yourself trying these out before you commit?
02:24 nwellnhof kid51: that was expected. it's caused by the gcov -p option.
02:25 kid51 Even with 'quickcover', the full cycle after a new commit takes most of 15 minutes.
02:26 Matt_ left #parrot
02:26 gg411 left #parrot
02:36 kid51 make quickcover:  still broken at 401fcfb541
02:37 kid51 nwellnhof: Would it be possible to back out your changes and rework them in a branch?
02:41 Matt_ joined #parrot
02:45 nwellnhof kid51: yes, we can do that
02:46 AndChat| left #parrot
02:51 kid51 nwellnhof:  Thanks.
02:51 nwellnhof left #parrot
03:01 mtk left #parrot
03:03 Matt_ left #parrot
03:09 dalek parrot: 7188dd4 | nwellnhof++ | config/ (3 files):
03:09 dalek parrot: Revert make cover changes
03:09 dalek parrot: review: https://github.com/parrot/parrot/commit/7188dd42e8
03:13 kid51 left #parrot
03:23 nwellnhof joined #parrot
03:27 rfw wait what exactly is a roflbbq
03:33 sorear fnord.  you are not cleard for that.
03:51 fbrito joined #parrot
04:01 davidfetter rfw, it's what you have after a long roflcopter ride
04:02 rfw will there be lmaonaise?
04:02 davidfetter of course :)
04:26 dalek TT #1907 created by coke++: Behavior of "exit" opcode changed since 2.11.0
04:26 dalek TT #1907: http://trac.parrot.org/parrot/ticket/1907
04:44 Coke left #parrot
05:07 Coke joined #parrot
05:16 estrabd left #parrot
05:18 nwellnhof_ joined #parrot
05:22 nwellnhof left #parrot
05:22 nwellnhof_ is now known as nwellnhof
05:28 cotto joined #parrot
05:31 cotto ~~
05:33 dd070 joined #parrot
05:35 rurban_ joined #parrot
05:35 dd070 left #parrot
05:35 Myhrlin left #parrot
05:37 rurban left #parrot
05:37 rurban_ is now known as rurban
05:41 nwellnhof left #parrot
05:41 Myhrlin joined #parrot
05:43 dalek parrot: 6dd7f77 | Coke++ | / (2 files):
05:43 dalek parrot: check dependencies on new frontend/ TLD too
05:43 dalek parrot:
05:43 dalek parrot: (and make them all pass)
05:43 dalek parrot: review: https://github.com/parrot/parrot/commit/6dd7f77124
05:43 dalek parrot: 8f1f70d | Coke++ | t/src/checkdepend.t:
05:43 dalek parrot: Don't skip directory that no longer exists.
05:43 dalek parrot: review: https://github.com/parrot/parrot/commit/8f1f70dace
05:43 dalek parrot: 34dea34 | Coke++ | / (2 files):
05:43 dalek parrot: Check deps for src/string/encoding
05:43 dalek parrot:
05:43 dalek parrot: (and make them all pass)
05:43 dalek parrot: review: https://github.com/parrot/parrot/commit/34dea347e0
05:46 dalek parrot: 9b71f41 | nwellnhof++ | / (3 files):
05:46 dalek parrot: Experimental make target quickcover_new
05:46 dalek parrot:
05:46 dalek parrot: New approach to fix make cover on different platforms. The script
05:46 dalek parrot: tools/dev/make_cover.pl is called from make to generate the report.
05:46 dalek parrot: review: https://github.com/parrot/parrot/commit/9b71f41bad
05:46 Coke if I want to create a new upstream branch, ??
05:47 Coke just start with a -b and then push it?
05:51 cotto dukeleto, ping
06:00 cotto coke++ for dependencies cleanup work
06:00 GeJ Merry belated Christmas #parroteers.
06:01 cotto Good tidings to you, to you and your crazy machine.
06:03 GeJ Thanks cotto.
06:05 Coke BWAHAHAHAH
06:06 Coke (apparently init::headers lets me rip out more than I anticipated)
06:13 dalek parrot/checkdepend_pmcs: cd5cbdd | Coke++ | / (14 files):
06:13 dalek parrot/checkdepend_pmcs: remove init::headers config step
06:13 dalek parrot/checkdepend_pmcs:
06:13 dalek parrot/checkdepend_pmcs: hardcode include directory here instead of config step.
06:13 dalek parrot/checkdepend_pmcs: Removing NONGEN headers entirely still passes checkdepend.t
06:13 dalek parrot/checkdepend_pmcs: and the build works at a high -j.
06:13 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/cd5cbddc16
06:20 fbrito can anyone please accept my claim request on this task? :) http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129228630126
06:24 rfw cotto: http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129315789586 <-- this task is invalid: Configure.pl already has --silent implemented
06:25 cotto onde
06:25 rfw at least
06:25 rfw i think it is
06:25 cotto er, fbrito's request was accepted
06:25 cotto rfw, ping kid51 about that.
06:25 rfw ah okay
06:26 cotto I have some concerns about another of his tasks.
06:26 Yuki`N rfw, yeah I already found that.
06:26 rfw msg kid51 http://trac.parrot.org/parrot/ticket/1897 <-- running Configure.pl in --silent mode does, in fact, make it run silenty
06:26 aloha OK. I'll deliver the message.
06:26 rfw Yuki`N: did you tell him?
06:27 rfw i think i'll do  Title: Research embedding Parrot into Internet Explorer 8
06:27 Yuki`N rfw, I brought it up a while back but I don't think anyone paid attention ot it.
06:27 rfw Yuki`N: ah okay
06:27 rfw cotto: http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129052493089 <-- could you approve
06:28 fbrito cotto thank you :). I have also opened a pull request with my work on it
06:29 fbrito cotto but don't worry reviewing it now... I am going to bed
06:29 cotto rfw, done
06:29 rfw thanks
06:29 cotto have fun and don't use wikipedia as your primary resource
06:29 fbrito rfw: nice! a research task :)
06:29 rfw i won't :D
06:29 cotto As a starting point it's great.  Just be sure to dig deeper.
06:37 GeJ Huuuum, tramadol!
06:42 dukeleto rfw: i am excited to see what you come up with :)
06:42 rfw dukeleto: well, i've already found IActiveScript
06:43 cotto looks like the parallel build is much better now.  coke++
06:46 fbrito i am going to bed. good night guys
06:46 Yuki`N dukeleto, can you add me another task for increasing src/embed/pmc.c?
06:46 Yuki`N I got it up to 95%, it was about 83% prior.
06:46 fbrito left #parrot
06:53 dukeleto Yuki`N: you have a link for the previous task?
06:53 Yuki`N left #parrot
06:53 Yuki`N joined #parrot
06:54 Yuki`N http://socghop.appspot.com/gci/task/show/google​/gci2010/parrot_perl_foundations/t129336706640
07:02 dukeleto Yuki`N: http://socghop.appspot.com/gci/task/edit/google/g​ci2010/parrot_perl_foundations/t129351968685?s=0
07:03 Yuki`N Requested claim.
07:05 cotto Yuki`N, accepted
07:05 Yuki`N Submitted work.
07:06 cotto kid speedy
07:07 dukeleto Yuki`N: do you want to make that a pull request against the proper branch?
07:07 dukeleto Yuki`N: then i will approve the task :)
07:07 dukeleto Yuki`N: or is it already committed?
07:07 rfw cotto: http://pastebin.com/RwMH6exx <-- do i need any more content?
07:07 dukeleto Yuki`N: i see that is in your fork
07:07 Yuki`N It's already committed to master.
07:08 dukeleto Yuki`N: gotcha
07:08 dukeleto Yuki`N: things move fast these days
07:08 Yuki`N Yeah. :P GCI.
07:08 Yuki`N Jumpyshoes is batching his tasks it seems.
07:09 dukeleto Yuki`N: i saw your very carefully-worded email to gci-discuss :)
07:09 Yuki`N Yes, indeed.
07:10 rfw i think i was being a bit too blunt with mine
07:10 cotto Yuki`N, done
07:10 dukeleto Yuki`N: task completed. go get another one :)
07:11 cotto dukeleto beat me
07:11 cotto clever gir^H^H^Hduke
07:12 dukeleto cotto: yes, i am quick these days
07:12 Yuki`N :P
07:12 Myhrlin left #parrot
07:13 dukeleto msg fbrito bigint gci task completed
07:13 aloha OK. I'll deliver the message.
07:14 Coke cotto: better?
07:14 dukeleto parrot.git has 5 pull requets right now
07:14 Coke perhaps the encoding stuff was wonky.
07:15 cotto Coke, the parallel build seems much more reliable
07:15 dukeleto cotto: did you apply the git hook stuff already? can that pull request be closed?
07:15 cotto I just ran make -J9 && make reconfigure 16 times and it didn't fail
07:15 cotto dukeleto, yes
07:16 cotto I though I closed that
07:16 dukeleto cotto: https://github.com/parrot/parrot/pull/43
07:16 cotto dukeleto, is there a nice way to review all commits to a github repo?
07:16 rfw cotto: http://pastebin.com/BS2Rkg6r revised abit
07:16 cotto closed
07:17 Coke cotto: it's been failing at -j9 ?
07:17 cotto rfw, you could use a gist
07:17 cotto Coke, occasionally
07:17 Coke if it ever fails, open a ticket so I can fix it. ;)
07:17 rfw oh
07:17 rfw pastebin due to force of habit
07:17 cotto gists are nice for things that get updated frequently
07:17 cotto just my preference, though
07:17 rfw https://gist.github.com/757003 there we go
07:18 rfw yeah i would use gist more if gist.github.com was shorter than pastebin.com
07:19 cotto did you find anything about projects that have tried something similar?
07:20 cotto dukeleto, are you looking through any of the pull requests?
07:20 rfw hm i looked at activestate's perlscript and pythonscript
07:20 rfw i mentioned it somewhere near the top
07:20 cotto so you did
07:21 rfw personally i think using an activex object is better than getting into wsh's activescript
07:21 cotto perhaps I should review when my brain is more functional
07:21 rfw heh okay
07:40 dalek parrot: bad917c | fbrito++ | t/pmc/bigint.t:
07:40 dalek parrot: [t] Add tests to BigInt
07:40 dalek parrot: review: https://github.com/parrot/parrot/commit/bad917cf2a
07:40 dalek parrot: 17c2706 | fbrito++ | t/pmc/bigint.t:
07:40 dalek parrot: [t] Add test to decrease in BigInt
07:40 dalek parrot: review: https://github.com/parrot/parrot/commit/17c2706298
07:40 dalek parrot: 400caf1 | fbrito++ | t/pmc/bigint.t:
07:40 dalek parrot: [t] Add 'add' and 'i_add' tests in BigInt
07:40 dalek parrot: review: https://github.com/parrot/parrot/commit/400caf1bdc
07:40 dalek parrot: 0502e9b | fbrito++ | t/pmc/bigint.t:
07:40 dalek parrot: [t] Add 'multiply' and 'i_multiply' tests in BigInt
07:40 dalek parrot: review: https://github.com/parrot/parrot/commit/0502e9b40a
07:40 dalek parrot: 2de15fa | fbrito++ | t/pmc/bigint.t:
07:40 dalek parrot: [t] Add 'mod' and 'comp' tests to BigInt
07:40 dalek parrot: review: https://github.com/parrot/parrot/commit/2de15fa655
07:42 dukeleto cotto: i have reviewed most of them when i completed the gci tasks they refer to
07:43 dukeleto cotto: do you want to merge a few?
07:43 cotto yes.  I was just going through outstanding pull requests
07:46 cotto are they all ok to merge without reviewing?
07:49 dalek parrot: 6a49fd6 | Yuki`N++ | config/gen/makefiles/root.in:
07:49 dalek parrot: Fix makefile clean target; properly clean generated files in t/src/embed/
07:49 dalek parrot: review: https://github.com/parrot/parrot/commit/6a49fd6f19
07:49 dalek parrot: 2989842 | Yuki`N++ | config/gen/makefiles/root.in:
07:49 dalek parrot: Fix straggling reference to QUICKCOVER_DIRS
07:49 dalek parrot: review: https://github.com/parrot/parrot/commit/2989842c6d
07:51 Yuki`N left #parrot
08:25 theory left #parrot
08:42 dalek parrot/checkdepend_pmcs: fd73b15 | Coke++ | t/src/checkdepend.t:
08:42 dalek parrot/checkdepend_pmcs: Enable testing of src/pmc dependencies.
08:42 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/fd73b154b2
08:42 dalek parrot/checkdepend_pmcs: 05f1002 | Coke++ | config/ (2 files):
08:42 dalek parrot/checkdepend_pmcs: Avoid duplicate deps on PMCs; add minimum deps
08:42 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/05f1002b23
08:42 dalek parrot/checkdepend_pmcs: 47a8ea0 | Coke++ | config/auto/pmc.pm:
08:42 dalek parrot/checkdepend_pmcs: remove debug output
08:42 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/47a8ea038b
08:42 dalek parrot/checkdepend_pmcs: 15b9d2a | Coke++ | config/auto/pmc.pm:
08:42 dalek parrot/checkdepend_pmcs: Minor fixups on calculated dependency paths.
08:42 dalek parrot/checkdepend_pmcs: - don't bother scanning <> includes at all.
08:42 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/15b9d2a036
08:42 dalek parrot/checkdepend_pmcs: a2b22ae | Coke++ | config/auto/pmc.pm:
08:42 dalek parrot/checkdepend_pmcs: every PMC has this as a dep.
08:42 dalek parrot/checkdepend_pmcs: (safe now that we de-dup)
08:42 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/a2b22ae80e
09:02 fperrad joined #parrot
09:21 rfw left #parrot
09:49 cognominal joined #parrot
09:49 dalek parrot/checkdepend_pmcs: a09555a | Coke++ | config/ (2 files):
09:49 dalek parrot/checkdepend_pmcs: move PMC_H_FILES into the pmc dep generator.
09:50 dalek parrot/checkdepend_pmcs: lets us de-dup more elements
09:50 dalek parrot/checkdepend_pmcs: add some more dynamic reqs.
09:50 dalek parrot/checkdepend_pmcs:
09:50 dalek parrot/checkdepend_pmcs: * down to 4 failures in checkdepend.pl
09:50 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/a09555a5f6
09:50 Coke that branch is down to 4 failures in t/src/checkdepend.t (and that's with testing all of src/pmc)
10:00 kennym joined #parrot
10:23 dalek parrot: b743497 | mikehh++ | examples/io/httpd.pir:
10:23 dalek parrot: replace concat_s_s opcode
10:23 dalek parrot: review: https://github.com/parrot/parrot/commit/b743497da2
10:23 dalek parrot: b747d04 | mikehh++ | examples/library/ncurses_life.pir:
10:23 dalek parrot: replace concat_s_s opcode
10:23 dalek parrot: review: https://github.com/parrot/parrot/commit/b747d041aa
10:23 dalek parrot: 0224f9d | mikehh++ | examples/pir/pirric.pir:
10:23 dalek parrot: replace concat_s_s opcode
10:23 dalek parrot: review: https://github.com/parrot/parrot/commit/0224f9d9e5
10:23 dalek parrot: 8765817 | mikehh++ | examples/sdl/lcd/clock.pir:
10:23 dalek parrot: replace concat_s_s opcode
10:23 dalek parrot: review: https://github.com/parrot/parrot/commit/8765817fba
10:27 * Coke wonders why all those were not fixed BEFORE the opcode was removed.
10:28 mikehh Coke: because nobody runs fulltest
10:29 tadzik hmm, any Rubyists out there?
10:29 Coke if you're ripping something out, you should have to.
10:29 mikehh yeah - we should emphasize that in the docs somewhere
10:30 dalek parrot/checkdepend_pmcs: f8671f3 | Coke++ | config/auto/pmc.pm:
10:30 dalek parrot/checkdepend_pmcs: Track cases where our include includes.
10:31 dalek parrot/checkdepend_pmcs: (only 1, so cheat.)
10:31 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/f8671f3af9
10:31 dalek parrot/checkdepend_pmcs: 1f81d6e | Coke++ | config/auto/pmc.pm:
10:31 dalek parrot/checkdepend_pmcs: Fix very old typo
10:31 dalek parrot/checkdepend_pmcs: Since we were never checking dependencies, we never noticed that this code
10:31 dalek parrot/checkdepend_pmcs: and the pmc2c code were doing 2 different things.
10:31 dalek parrot/checkdepend_pmcs:
10:31 dalek parrot/checkdepend_pmcs: - checkdepend.t passes again (now with no .c skips)
10:31 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/1f81d6e6ee
10:31 dalek parrot/checkdepend_pmcs: 19002ba | Coke++ | config/auto/pmc.pm:
10:31 dalek parrot/checkdepend_pmcs: whitespace
10:31 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/19002bac6c
10:31 dalek parrot/checkdepend_pmcs: 987f414 | Coke++ | t/codingstd/pccmethod_deps.t:
10:31 dalek parrot/checkdepend_pmcs: Fix same bug that was cut/pasted from auto::pmc
10:31 dalek parrot/checkdepend_pmcs:
10:31 dalek parrot/checkdepend_pmcs: Add note about merging this dependency check into checkdepend.t
10:31 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/987f4144fb
10:31 dalek parrot/checkdepend_pmcs: b44084d | Coke++ | config/ (2 files):
10:31 dalek parrot/checkdepend_pmcs: Remove now-redundant .mak variable.
10:31 dalek parrot/checkdepend_pmcs: review: https://github.com/parrot/parrot/commit/b44084d18d
10:31 mikehh Coke: have you any thoughts on generating the header/index pages in html_cleanup
10:33 mikehh Coke: we have the docs/index/*.jsom files, just now need to generate the pages from them - we have the rest of the .html files now
10:34 Coke mikehh: haven't given it any thought. Doesn't matter because no one ever runs "make html" anyway. ;)
10:34 Coke there, checkdepend_pmcs is done.
10:36 mikehh Coke: just me and for release docs
10:36 Coke mikehh: yes, no one runs that, see the mailing list. ;)
10:36 mikehh Coke: been following that
10:37 Coke must sleep for an hour or 2. later.
10:38 Coke msg dukeleto I'm not sure Karen had anything in particular to do with your grant getting awarded, FYI.
10:38 aloha OK. I'll deliver the message.
10:43 * mikehh got to go out for a bit - bbl
11:41 tadzik http://wklej.org/id/445971/ -- what can possibly break stuff in lex_lookup()?
11:42 tadzik well, the lex_lookup is ok, but checking the result fails with NPE
11:46 redicaps joined #parrot
12:33 bluescreen joined #parrot
13:09 lidden joined #parrot
13:17 fbrito joined #parrot
13:21 redicaps left #parrot
13:35 rurban_ joined #parrot
13:37 rurban left #parrot
13:38 rurban_ is now known as rurban
13:49 whiteknight joined #parrot
13:49 smash joined #parrot
13:49 smash hello everyone
13:52 whiteknight good morning smash
14:01 mtk joined #parrot
14:03 mtk left #parrot
14:04 mtk joined #parrot
14:04 mtk left #parrot
14:05 mtk joined #parrot
14:29 fbrito hm, how should I test EventHandler PMC set_pmc VTABLE? argument is a ParrotInterpreter...
14:30 whiteknight fbrito: let me look
14:30 fbrito whiteknight: ok, thank you very much :)
14:33 whiteknight fbrito: do you need to test that VTABLE to complete your task?
14:34 fbrito not really :P. I already completed it
14:35 fbrito I was working on this task: http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129228651258
14:36 fbrito and now only invoke and this VTABLE are not covered
14:37 fbrito but according to the EventHandler test file, invoke is broken (seg fault)
14:44 whiteknight EventHandler.invoke is broken? Invoke is like the only thing that damn PMC type does
14:44 whiteknight EventHandler serves no purpose if it can' tbe invoked
14:45 fbrito https://github.com/parrot/parrot/​blob/master/t/pmc/eventhandler.t
14:46 fbrito whiteknight: line 88
14:46 whiteknight damnit
14:46 fbrito I haven't look the problem well, but if you uncomment the test it will seg fault
14:47 whiteknight I think we should deprecate that type. Can you open a ticket on Trac suggesting we deprecate and remove EventHandler PMC?
14:48 fbrito (weird... some times, after reading what I have just wrote, I feel that my english is getting worse and worse :P)
14:48 fbrito whiteknight: ok, sure
14:50 fbrito http://trac.parrot.org/parrot/ticket/1894
14:50 whiteknight fbrito
14:50 whiteknight : your english is very good
14:51 whiteknight I think we can fix EventHandler.invoke for now, if you want
14:52 whiteknight in the invoke VTABLE, try to get e->interp. if that's null, use INTERP instead
14:52 whiteknight I think that should fix the segfault
14:53 fbrito I will give it a try... just a moment
15:02 fbrito (compiling)
15:07 fbrito no success
15:07 dalek parrot: b3af8c9 | Whiteknight++ | / (3 files):
15:07 dalek parrot: fix main.c so that we check the is_error flag and only show an error message and backtrace if we have an error. This fixes TT #1907. Add two tests for the test cause provided by Coke++ to show that it does indeed have an exit code of 3, but that it produces no error output
15:07 dalek parrot: review: https://github.com/parrot/parrot/commit/b3af8c93d7
15:08 fbrito I guess the opportunity to learn gdb has finally come! :D
15:09 whiteknight gdb --args ./parrot t/pmc/eventhandler.t
15:09 whiteknight fbrito: can you nopaste your attempted fix?
15:10 fbrito whiteknight: sure
15:11 Coke whiteknight: thanks. as time permits, I'll rebuild parrot & rakudo and see what happens.
15:13 fbrito whiteknight: http://pastie.org/1411201
15:14 fbrito (I still have to get used to the pastie script in tools folder :S)
15:14 whiteknight Coke++
15:14 whiteknight Coke: I'm very sorry that it took so long to bisect. If I had seen the error report earlier I likely could have narrowed down the cause much more quickly
15:17 Matt_ joined #parrot
15:18 whiteknight fbrito: change "if (e->interp)" to "if (!PMC_IS_NULL(e->interp))"
15:18 whiteknight actually, don't bother
15:18 whiteknight that whole thing is so screwed up
15:23 fbrito seg fault again
15:25 fbrito I have also played with the tests (initialized in another way, changed the invoked sub, and so on) but got no success
15:26 Matt_ whiteknight: How should I go about testing functions like Parrot_api_load_language(Parrot_PMC interp_pmc, Parrot_String lang) ? I wanna finish this task up today
15:27 whiteknight Matt_: comment that function out. It was a prototype
15:27 whiteknight (sorry there are so many of those!)
15:28 Matt_ whiteknight: How about Parrot_api_add_dynext_search_path . Seems to be called from frontend/parrot/main.c
15:29 fbrito ok, I have to go now. When you guys have some time, please accept my claim request on this task: http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129228651258
15:29 Matt_ I dont see anyway of getting the set paths from the Parrot_api_* functions
15:29 whiteknight fbrito: I am testing a fix for EventHandler.invoke now. If it works I will push it for you
15:30 fbrito ok :D
15:30 fbrito I have already increased the code coverage in 17.8% anyway
15:34 whiteknight ok
15:35 fbrito if the task is to increase code coverage in 10% and I do 20%, is it ok to ask to you to create another task?
15:35 whiteknight I'm testing now to make sure I didn't break anything. If that works well I'll push it for you to experiment with
15:35 whiteknight fbrito: submit the work you have now for the first task, and I'll open a new task, keeping track of the extra work you've already done
15:36 fbrito I saw some other gci students asking for that, but I was always too embarrassed to ask :(. I have even made that some tasks ago (increasing code coverage more than 2x)
15:37 whiteknight I can't edit the task in-place, or I would simply bump it up to 20% and 4 points
15:37 whiteknight fbrito: if you drop that task, I'll edit it and you can re-accept it at a higher difficulty
15:38 fbrito done. thank you :).
15:38 whiteknight 4 points is the same as 2 2-point tasks
15:40 Matt_ whiteknight: Any ideas on how to test functions like: Parrot_api_set_warnings(Parrot_PMC interp_pmc, Parrot_Int flags) ?
15:40 Matt_ Maybe getters should be implemented first in a separate task?
15:43 whiteknight Matt_: Search the code for a place that throws a warning message. Write a test that tries to create that condition. Do a c_error_output_like test with the necessary set_warnings code and the expected warning message
15:44 whiteknight grep for Parrot_warn (src/warnings.c)
15:44 Matt_ thanks!
15:44 whiteknight I hope that helps
15:44 whiteknight actually, looking at Parrot_warn I have to ask why it returns 2 on failure, 1 on success. That seems...stupid
15:45 whiteknight Matt_: in your greppings, if you can figure out a reason why it returns those values, I would be very grateful
15:48 fbrito whiteknight: I have to lunch right now... will be back in 40 minutes or so
15:49 fbrito I am starting to get afraid of someone claiming the task :P, so I will claim it and if I manage to increase more than 20% you can create another task, ok?
15:51 fbrito thank you very much for your attention :D
15:51 fbrito Matt_: good luck with your task :)
15:51 fbrito left #parrot
16:00 dalek parrot: b3de476 | Whiteknight++ | src/pmc/eventhandler.pmc:
16:00 dalek parrot: attempted fix for EventHandler.invoke, since it obviously hadn't been updated since before the big PCC refactors several months ago. This VTABLE is untested, though tests are in development
16:00 dalek parrot: review: https://github.com/parrot/parrot/commit/b3de476fe0
16:03 theory joined #parrot
16:09 JimmyZ joined #parrot
16:15 hercynium joined #parrot
16:30 JimmyZ left #parrot
16:36 dngor left #parrot
16:52 Coke whiteknight: oddly, while it fixes my PIR snippet, "exit 3" in rakudo still fails.
16:54 fbrito joined #parrot
16:56 dalek Some 'nwellnhof' person just gave me a 67602 byte commit packet.  They need to be more careful in the future.
16:59 whiteknight Coke: that's extremely weird. do you have the error message text it prints out?
17:00 whiteknight Coke: as basic sanity, that Rakudo is running against the new Parrot version?
17:00 whiteknight er, the version I pushed this morning?
17:01 nopaste "coke" at 192.168.1.3 pasted "latest rakudo/parrot" (5 lines) at http://nopaste.snit.ch/27518
17:01 Coke whiteknight: yup.
17:01 whiteknight okay, I had to ask
17:05 dngor joined #parrot
17:06 whiteknight Coke: I'm looking at the source for &exit in rakudo now. It's a very thin wrapper around PIR exit_i.
17:07 whiteknight Coke: Can you try it with "parrot perl6.pbc ..."
17:07 whiteknight ?
17:08 whiteknight instead of the fakecutable?
17:09 Coke ah, good call. that's it.
17:09 whiteknight okay. my last commit updated main.c, but not pbc_to_exe to match it
17:09 whiteknight lemme fix it
17:10 dalek winxed: r723 | NotFound++ | trunk/ (2 files):
17:10 dalek winxed: fix for parrot deprecated out usages
17:10 dalek winxed: review: http://code.google.com/p/w​inxed/source/detail?r=723
17:15 dalek winxed: r724 | NotFound++ | trunk/winxedst1.winxed:
17:15 dalek winxed: fix string indexing, Issue 15, plobsing++
17:15 dalek winxed: review: http://code.google.com/p/w​inxed/source/detail?r=724
17:18 fbrito whiteknight: hello :). I am still getting seg fault error even after your commit :(
17:20 fbrito and I didn't understand what that "real_next" variable does :s
17:25 dukeleto ~~
17:25 dukeleto fbrito: how goes it?
17:26 fbrito dukeleto: hello :)
17:29 dukeleto fbrito: how is GCI treating you?
17:29 dukeleto fbrito: do you have stuff to work on?
17:29 fbrito dukeleto: I am working on this task: http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129228651258
17:30 fbrito I have already improved it by 17.8%
17:30 dukeleto fbrito: nice work!
17:30 fbrito whiteknight agreed that if I improve it by 20%, he will create another similar task
17:31 fbrito but the most important VTABLE (invoke) is broken
17:31 fbrito he tried to fix it (see last commit on master) but no success
17:31 dukeleto fbrito: interesting!
17:37 fbrito yeah! Increased it by 19.8% :)
17:37 davidfetter fbrito++ :)
17:38 fbrito but invoke is still broken, and as whiteknight, it makes no sense to have a EventHandler that can't invoke
17:38 fbrito as whiteknight said*
17:39 fbrito he was even talking about deprecation
17:40 dalek parrot: 264c681 | NotFound++ | t/pmc/hashiterator.t:
17:40 dalek parrot: test HashIterator with a Hash with non default key type
17:40 dalek parrot: review: https://github.com/parrot/parrot/commit/264c681db1
17:42 dukeleto fbrito: well, you can still write a TODO test for invoke, which will cover it, even if it doesn't work
17:42 dukeleto fbrito: did you create a trac ticket for that problem?
17:44 fbrito dukeleto: there is already one covering that: http://trac.parrot.org/parrot/ticket/1894
17:46 dukeleto fbrito: ok, sounds good
17:47 fbrito I guess I will wait for whiteknight to see if he will try to fix it again
17:47 dukeleto fbrito: don't wait on him, work on something else :)
17:48 fbrito yeah, but now I will have to work for my father, ahhaha
17:50 kennym left #parrot
17:50 whiteknight fbrito: what happens when you use the new invoke?
17:50 fbrito whiteknight: same thing as before. segmentation fault error
17:51 whiteknight fbrito: nopaste your test again? I want to try it myself and see what is going wrong
17:51 fbrito ah, and I wrote a test to catch the exception you wrote (e->code null). It is working fine
17:52 fbrito whiteknight: I have just uncommented what was already written in "t/pmc/eventhandler.t"
17:52 whiteknight oh,okay
17:53 nwellnhof joined #parrot
17:54 tadzik ~~
17:54 fbrito http://pastie.org/1411572
17:54 fbrito I also tried to initialize the EventHandler PMC with a Hash (like the first test on this same file), but still got the same error
17:54 whiteknight ok
17:55 dalek parrot: 3562ee6 | Whiteknight++ | / (2 files):
17:55 dalek parrot: fix pbc_to_exe for Coke++
17:55 dalek parrot: review: https://github.com/parrot/parrot/commit/3562ee6ea7
17:55 dalek parrot: 8effd61 | Whiteknight++ | t/pmc/hashiterator.t:
17:55 dalek parrot: Merge branch 'master' of github.com:parrot/parrot
17:55 dalek parrot: review: https://github.com/parrot/parrot/commit/8effd6102c
17:55 kennym joined #parrot
17:57 Matt_ whiteknight: I tried using Parrot_warn but the warning message never gets printed out
17:58 Matt_ http://pastie.org/private/nnpykbcef0gpxprnne2p0a
17:58 Matt_ Seems like Parrot_api_set_warnings has no effect (something to do with the interp?)
18:01 whiteknight Matt_: in the call to Parrot_warn, try using PARROT_WARNINGS_EXPERIMENTAL_FLAG instead of PARROT_WARNINGS_ALL_FLAG
18:02 Matt_ whiteknight: same issue (which I why I tried *_ALL)
18:05 whiteknight Matt_: ah, I think you have found a bug
18:05 whiteknight Okay, write the test how you think it should work and mark it TODO
18:06 whiteknight and then can you open a ticket on Trac for this?
18:08 Matt_ yup
18:09 fbrito1 joined #parrot
18:10 M_o_C joined #parrot
18:11 Matt_ whiteknight: Should I leave the test in there when submitting the task as it fails?
18:11 whiteknight yes. Mark it TODO
18:13 dukeleto Matt_: call todo() instead of ok() in PIR to mark a TODO test
18:13 Matt_ dukeleto: This is a C-based test
18:13 whiteknight Matt_: grep for "$TODO", that should give you some examples
18:14 fbrito left #parrot
18:17 dukeleto Matt_: if your test is in Perl, then you need to do : local $TODO = "a reason";
18:17 whiteknight msg fbrito: I think your eventhandler test has discovered a nasty GC bug. Awesome
18:17 aloha OK. I'll deliver the message.
18:17 cotto_work ~~
18:18 cotto_work fbrito++ then
18:18 Matt_ dukeleto: Yep - I looked up. Is there an equivalent for C-based tests (that use c_output_is)
18:18 whiteknight Matt_: it's the same, since the test file is written in Perl
18:18 whiteknight the test is C, but the file is perl
18:19 fbrito1 :D
18:19 fbrito1 is now known as fbrito
18:20 dukeleto Matt_: you are writing a Perl test that executes C
18:22 fbrito whiteknight: I think I am done with the task: http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129228651258
18:22 whiteknight fbrito: will look in a bit
18:23 fbrito ok :D thank you
18:23 tadzik hmm, any Rubyists out there?
18:23 tadzik bah, nvm
18:24 tadzik http://wklej.org/id/445971/ -- what can possibly break stuff in lex_lookup()?
18:24 tadzik well, the lex_lookup is ok, but checking the result fails with NPE
18:27 dukeleto tadzik: i see you are fighting the good fight
18:29 tadzik dukeleto: to the last commit
18:40 Kapace_ whiteknight: eventhandler test still fails: http://dpaste.com/291601/
18:40 Kapace_ fbrito: did you get invoke to work on eventhandler?
18:41 fbrito no. according to whiteknight, it may be a "nasty GC bug"
18:41 whiteknight Kapace_: yeah, that event handler test exposes some weird kind of PCC bug
18:41 whiteknight I don't think it's a GC bug anymore
18:41 fbrito ah
18:42 fbrito at the moment, only invoke and set_pmc are not covered by tests
18:43 Kapace_ set_pmc isn't going to be 10% i don't think
18:43 fbrito and 1 line somewhere else
18:43 fbrito Kapace_ https://github.com/parrot/parrot/pull/52 :D
18:44 whiteknight actually, now it's looking like a stupid EventHandler bug
18:44 whiteknight this code sucks
18:45 simcop2387 left #parrot
18:47 Kapace_ fbrito: I had already made tests for mark and get_string_null...
18:48 fbrito oh :o
18:48 fbrito where?
18:48 Kapace_ fbrito: I just left my tests in create_an_event_and_set_attributes
18:48 Kapace_ see "sweep 1" ?
18:49 Kapace_ I had tested bad arguments too
18:49 Kapace_ I just couldn't finish it with invoke being broken
18:50 rfw joined #parrot
18:50 fbrito oh, I really didn't know that you was working on this task
18:50 Kapace_ Tene said that invoking an eventhandler didn't make sense, so I just made the test and commented it
18:50 tadzik dukeleto: any ideas? I don't even know where to seek
18:52 fbrito Kapace_ so you have also claimed a task to improve EventHandler test coverage in 10%?
18:53 Kapace_ fbrito: yes, I closed one with 17%, but I could not get to the next 10% :(
18:53 dukeleto tadzik: not sure. Tene is a good person to ask, or parrot-dev
18:53 tadzik Tene: you around?
18:54 fbrito ahh, I see
18:55 Kapace_ I could really use the points, if you know what I mean :P
18:56 Coke whiteknight: that fixed it.
18:57 whiteknight Coke: awesome. Sorry about the bug
18:57 whiteknight Kapace_, fbrito: the EventHandler PMC really is garbage. I think we should remove it, not improve it's coverage
18:57 whiteknight but that's a personal thing
18:58 Kapace_ whiteknight: yeah, Tene was saying that too
18:58 Kapace_ that nobody uses it etc
18:58 bluescreen nwellnhof: ping
18:59 nwellnhof bluescreen: pong
19:00 dukeleto whiteknight: we need to improve coverage to know how broken it is
19:00 fbrito Kapace_ I was looking an old coverage report
19:00 bluescreen nwellnhof: finally found the issue with TT#1765, but it is not easy to fix it... Basically we're creating interpreters and interp's pmc wraps 2 things attributes and the bare interp
19:00 fbrito Kapace_ that's why I didn't see the improvements you made
19:00 dukeleto whiteknight: and i agree with you, but i still think that GCI students can learn from improving the test coverage of it, and we learn how broken it is
19:01 bluescreen the destroy method of the parrotinterpreter.pmc gets rid of one structure not the other
19:01 Kapace_ fbrito: ah, that make sense
19:02 fbrito Kapace_ ehhe, my bad. the code I wrote is duplicated and useless
19:03 Kapace_ nah, it better organized!
19:03 bluescreen the problem is that multiple interp's pmc point to the same raw interp... so if we destroy the raw interpreter in the parrotinterpreter.pmc we will get double free corruptions as there are 2 interp pmcs sharing same interp
19:04 nwellnhof bluescreen: yes, i tried similar things. it isn't easy to fix.
19:04 Matt_ nwellnhof: I'm once again having trouble running `make cover` with GCC 4.5.1 after the recent changes were merged in
19:04 Matt_ nwellnhof: no source src/src/dynext.c found for src/dynext.c.gcov
19:04 nwellnhof Matt: can you try 'make quickcover_new'?
19:04 bluescreen nwellnhof: tough i don't understand why we share the interp.
19:05 dalek parrot: 76a4452 | Whiteknight++ | src/pmc/eventhandler.pmc:
19:05 dalek parrot: fix EventHandler invoke. 'Fix' is a strong word considering that this PMC is still old garbage
19:05 dalek parrot: review: https://github.com/parrot/parrot/commit/76a4452583
19:05 whiteknight Kapace_, fbrito: That last commit fixes invoke and gets that one test to run on my machine
19:05 bluescreen I'm guessing that in order to fix that we need some sort of redesign
19:05 nwellnhof bluescreen: i don't know that code very well. that's why i gave up after a while...
19:05 bluescreen who should be the expert there?
19:05 bluescreen so I can bug him/her
19:06 nwellnhof whiteknight?
19:07 fbrito whiteknight: yes, but the invoke test was written by Kapace_. If the task is still available, I think he should claim it
19:07 Kapace_ thanks fbrito, whiteknight what do you think?
19:07 dalek parrot/encapsulate-main: 9e74275 | plobsing++ | frontend/pbc_merge/main.c:
19:07 dalek parrot/encapsulate-main: fixup constants after merging bytecode
19:07 dalek parrot/encapsulate-main: review: https://github.com/parrot/parrot/commit/9e742753d0
19:07 dalek parrot/encapsulate-main: 3bac222 | plobsing++ | t/compilers/tge/basic.t:
19:07 dalek parrot/encapsulate-main: falling off the end of a :main sub executes an end op
19:07 dalek parrot/encapsulate-main:
19:07 dalek parrot/encapsulate-main: this exits the test prematurely
19:07 dalek parrot/encapsulate-main: review: https://github.com/parrot/parrot/commit/3bac22218f
19:08 bluescreen whiteknight: are you the "to-go" person when it comes to interp and interp_pmc ?
19:09 whiteknight bluescreen: No. I don't think there is a to-go person for that stuff
19:09 whiteknight I can take a look at it though
19:09 whiteknight I've been through that code many times
19:10 bluescreen ok.. then I'll prepare an email for parrot-dev explaining what the issue is with the interp pmc
19:10 whiteknight okay, that works too
19:10 bluescreen it has a memory leak, but the design makes it hard to fix
19:10 whiteknight where's the leak?
19:11 bluescreen the pmc has 2 things the attributes + raw interp
19:11 bluescreen the destroy method of parrot interpreter pmc only gets rid of the first part
19:11 bluescreen but the trick is that there may be several interp pmcs using the same raw interp
19:12 bluescreen so the fundamental question is freeing the raw interpreter is responsibly of whom?
19:12 Coke original parent interp?
19:13 whiteknight bluescreen: Parrot_destroy
19:14 bluescreen Parrot_destroy calls the GC and that calls destroy in the interp PMC.. at that point if destroy does destroy the raw interp, you get double free corruption
19:14 whiteknight bluescreen: Right, the PMC should not free the interp
19:15 bluescreen Coke: the parent interp has no way to know which are its children interps
19:15 whiteknight Parrot_destroy kills the GC and destroys all PMCs. Then Parrot_destroy frees the interp
19:16 bluescreen for example:      (PIR)         test_interp = new 'ParrotInterpreter'
19:16 bluescreen test_interp is only destroyed by the destroy method of the PMC, Parrot_destroy is never called for that interp
19:17 bluescreen the "on_exit" handler is only called for the interp calling exit
19:17 bluescreen not for the other interps around
19:18 whiteknight ah, okay. I see
19:18 whiteknight fun
19:18 whiteknight as a temp fix, maybe ParrotInterpreter PMC needs a "should I destroy?" flag
19:18 bluescreen child interp doesn't register the on_exit on its parents
19:20 nwellnhof the GC destroys every PMC with the --leak-test option. the problem is probably the order of destruction.
19:22 Tene Kapace_, dukeleto, tadzik, whiteknight: I can't think of any meaningful or useful interpretation of invoke vtable on an EventHandler PMC, and beyond that, the current impl just crashes.  That's not something we want tests for.
19:22 whiteknight ParrotInterpreter.destroy does not call Parrot_destroy
19:22 whiteknight Tene: I fixed the current implementation. It hadn't been updated since the PCC refactors months ago
19:22 Tene whiteknight: Ah.
19:22 Kapace_ Tene: see whiteknight recent commit, he kinda "fixed" it, but yeah, I agree with ^
19:22 whiteknight Tene: I can't think of a meaningful use of the EventHandler PMC
19:22 whiteknight I think we need to deprecate that
19:23 Kapace_ whiteknight: should I still take the task?
19:24 Tene whiteknight: I'm sorry?  I think a task scheduling system is something we do want, as shown by other work on similar projects: threading, async io, etc.
19:24 whiteknight Kapace_: If you think you can complete it. It will be good practice, but ultimately I think we are getting rid of that type
19:24 whiteknight Tene: yes, ultimately. This implementation is not something that we want. That whole system needs a redesign
19:25 Tene whiteknight: Ah, I perhaps misunderstood you.
19:25 Tene Yes, I agree with redesign.
19:25 Kapace_ whiteknight: I already wrote the tests, but I couldn't complete it because invoke was broken, now its fixed, but we think EventHandler shouldn't even exist..
19:25 Tene IMO threading is rather *more* important, so I'd be surprised to see it addressed before threading is.
19:26 Tene Kapace_: It's always better to have tests, so we can *know* when we break things.
19:27 Tene I see tt757 still hasn't been touched in over a year :(
19:28 Patterner left #parrot
19:28 whiteknight Kapace_: Yes, complete the task. Get the points. We'll create something more important for you next
19:28 Kapace_ whiteknight: awesome
19:28 Kapace_ http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129228651258
19:29 whiteknight Tene: I'm waiting on more Lorito developments before working on threading
19:29 whiteknight hopefully threading will be a central concern for the Lorito design
19:29 Tene whiteknight: Sure.  I don't think that anyone's priorities are unreasonable here.
19:30 whiteknight Kapace_: accepted
19:30 whiteknight Tene: I see Tasks and Threads being basically the same thing
19:30 whiteknight a good system for one should include the other
19:32 Tene whiteknight: Agreed.
19:32 Kapace_ whiteknight: how can I make a pull request only for my last commit?
19:33 Kapace_ or should I just make the pull request regardless?
19:33 whiteknight Kapace_:  is there stuff in your fork that you don't want to have included?
19:33 Kapace_ whiteknight: well theres stuff from other tasks..
19:35 whiteknight Kapace_: it would be easier for me to just uncomment that test here instead of trying to pull that commit
19:35 Kapace_ ok cool, just submit a patch then?
19:36 rfw did anyone get round to pulling my changes, lol
19:36 dalek parrot: 74a5256 | Whiteknight++ | t/pmc/eventhandler.t:
19:36 dalek parrot: uncomment out test from Kapace++
19:36 nwellnhof whiteknight: i just found an embed_api regeression. when you call exit with a non-zero status, a backtrace is printed to stderr. this causes a Rakudo spectest failure.
19:36 dalek parrot: review: https://github.com/parrot/parrot/commit/74a5256c7c
19:37 whiteknight Kapace_: Pulled. Done
19:37 whiteknight rfw: let me do that now
19:37 rfw yay \o/
19:38 Kapace_ thanks whiteknight, when your not busy, http://www.google-melange.com/gci/task/show/goog​le/gci2010/parrot_perl_foundations/t129228651258
19:38 Psyche^ joined #parrot
19:38 Psyche^ is now known as Patterner
19:38 whiteknight Kapace_: done
19:39 Kapace_ whiteknight: awesome, thanks again
19:39 estrabd joined #parrot
19:39 dalek parrot/gci_rfw_coverage: dbab727 | (Tony Young)++ | t/pmc/role.t:
19:39 dalek parrot/gci_rfw_coverage: Added a test in role.t to cover make_namespace_autobase.
19:39 dalek parrot/gci_rfw_coverage: review: https://github.com/parrot/parrot/commit/dbab727200
19:39 dalek parrot/gci_rfw_coverage: a17b5b9 | (Tony Young)++ | / (144 files):
19:39 dalek parrot/gci_rfw_coverage: Merge branch 'master' of git://github.com/parrot/parrot
19:39 dalek parrot/gci_rfw_coverage: review: https://github.com/parrot/parrot/commit/a17b5b9320
19:40 dalek parrot/gci_rfw_coverage: 91fe6d6 | (Tony Young)++ | t/pmc/continuation.t:
19:40 dalek parrot/gci_rfw_coverage: Increased coverage of Continuation PMC to 86.60%.
19:40 dalek parrot/gci_rfw_coverage: review: https://github.com/parrot/parrot/commit/91fe6d65b6
19:40 dalek parrot/gci_rfw_coverage: b874a65 | (Tony Young)++ | t/src/misc.t:
19:40 dalek parrot/gci_rfw_coverage: Adding src/misc.c tests.
19:40 dalek parrot/gci_rfw_coverage: review: https://github.com/parrot/parrot/commit/b874a658f3
19:40 dalek parrot/gci_rfw_coverage: 04b407a | (Tony Young)++ | / (4 files):
19:40 dalek parrot/gci_rfw_coverage: Merge branch 'nwellnhof/make_cover' of git://github.com/parrot/parrot
19:40 dalek parrot/gci_rfw_coverage: review: https://github.com/parrot/parrot/commit/04b407ad07
19:40 dalek parrot/gci_rfw_coverage: 458de73 | (Tony Young)++ | t/src/misc.t:
19:40 dalek parrot/gci_rfw_coverage: 100% coverage for src/misc, whoo!
19:40 dalek parrot/gci_rfw_coverage: review: https://github.com/parrot/parrot/commit/458de73602
19:44 rfw i would preferred you rebased my commits together but too late :D
19:44 rfw prefer*
19:45 whiteknight rfw: I like to keep the raw history as-is
19:45 rfw heh
19:45 rfw actually
19:45 rfw i swear that first commit
19:45 rfw dbab727
19:45 rfw was already pulled in
19:47 nwellnhof whiteknight: forget my last message. that was caused by my own changes.
19:48 rfw do i get an entry in CREDITS yet :D
19:48 whiteknight nwellnhof: no, that was a real error. Coke reported it this morning and I fixed it shortly thereafter
19:49 nwellnhof ah, ok.
19:49 cotto_work mikehh: ping
19:49 cotto_work seen mikehh
19:49 aloha mikehh was last seen in #parrot 9 hours 12 mins ago saying "Coke: been following that".
20:06 fbrito1 joined #parrot
20:09 fbrito left #parrot
20:19 kid51 joined #parrot
20:19 mikehh cotto_work: pong
20:20 allison joined #parrot
20:22 cotto_work mikehh: how close to merge-ready is html_cleanup?
20:24 mikehh cotto_work: got to generate the index pages otherwise ok
20:24 mikehh see my #parrotsketch report (submitting in a minute)
20:24 cotto_work ok
20:28 kid51 GCI participants: You are welcome to join our weekly on-line developers meeting in #parrotsketch
20:29 kid51 ps in 1
20:33 kid51 ps START
20:41 kid51 plumage?
20:41 kid51 aloha, plumage?
20:41 aloha kid51: plumage is supposed to be that (I think). but it is mostly built on top of distutils.pbc and setup.pir
20:42 tadzik aloha: forget plumage
20:42 aloha tadzik: I forgot about plumage.
20:43 tadzik plumage is a package manager for Parrot
20:43 tadzik aloha: plumage?
20:43 aloha tadzik: plumage is a package manager for Parrot
20:43 tadzik I like you, aloha
20:57 dalek parrot: d34e58a | Whiteknight++ | / (3 files):
20:57 dalek parrot: small fixes to help the g++ build. still does not complete
20:57 dalek parrot: review: https://github.com/parrot/parrot/commit/d34e58a033
21:01 dalek parrot/encapsulate-main: ac00c83 | plobsing++ | compilers/imcc/ (2 files):
21:01 dalek parrot/encapsulate-main: eliminate seen_main field
21:01 dalek parrot/encapsulate-main: review: https://github.com/parrot/parrot/commit/ac00c839c1
21:03 TonyC left #parrot
21:09 nopaste left #parrot
21:16 hercynium left #parrot
21:19 hercynium joined #parrot
21:21 cognominal left #parrot
21:24 [hercynium] joined #parrot
21:28 whiteknight left #parrot
21:29 hercynium left #parrot
21:29 [hercynium] is now known as hercynium
21:31 mtk left #parrot
21:32 mtk joined #parrot
21:34 bluescreen left #parrot
21:36 rurban_ joined #parrot
21:37 fperrad left #parrot
21:38 rurban left #parrot
21:39 rurban_ is now known as rurban
21:40 simcop2387 joined #parrot
21:47 Yuki`N joined #parrot
21:47 Yuki`N aloha, coverage?
21:47 aloha Yuki`N: coverage is http://cv.perl6.cz or http://tapir2.ro.vutbr.cz/cover/cover-results/
21:48 tadzik aloha: trac?
21:48 aloha tadzik: trac is a bug tracker that just happens to bundle a smallish wiki for your convenience.
21:49 tadzik I remember that having "or is slow, or is really slow, or..."
21:49 Yuki`N :P
21:49 mtk left #parrot
21:51 atrodo tadzik> purl had that in it's definition.
21:52 tadzik ah, good ol' purl
21:55 silug joined #parrot
21:57 Yuki`N karma N
21:57 aloha N has karma of 15.
21:57 Yuki`N lol
21:58 tadzik karma C
21:58 aloha C has karma of 143.
21:58 Yuki`N The ` in my name breaks the karma system.
21:58 Yuki`N karma Yuki`N
21:58 aloha Yuki`N has karma of 0.
21:58 tadzik I see what you did there, C++
21:58 mikehh karma g
21:58 aloha g has karma of 113.
21:58 Yuki`N :P
21:59 Yuki`N g++ file.c
21:59 tadzik yes, that's how it works :)
22:00 Yuki`N msg dukeleto can you take care of the 'silent configure' task on GCI? It's distracting and unfortunately invalid. :(
22:00 aloha OK. I'll deliver the message.
22:00 cotto_work kid51: ping
22:01 mikehh make corevm/make coretest works with g++ now, but frontend/parrot_debugger/main.c fails with make (world)
22:04 kid51 cotto_work: pong
22:04 cotto_work kid51: the gci task to rename makefile targets seems *really* trivial.  Am I missing something?
22:05 kid51 It may seem trivial ... but we've been having so many problems with those two targets that I suspect the task-doer will think it non-trivial.
22:05 kid51 However, feel free to make adjustments
22:06 kid51 I'll defer to others' judgment re GCI tasks, because I haven't yet been mentor.
22:06 kid51 I just care about the corresponding Trac tickets.
22:06 moritz there's nothing wrong with trivial tasks, IMHO
22:06 cotto_work kid51: I don't see how it could be non-trivial.  It's just editing two lines in the makefile template.  Any other problems the student would run into wouldn't be the student's concern.
22:06 dalek TT #1908 created by jkeenan++: Create new testing target: 'make test' + more
22:06 dalek TT #1908: http://trac.parrot.org/parrot/ticket/1908
22:07 kid51 So, please modify the task (including hours needed, if needed)
22:07 kid51 Or, if you think it's extremely trivial, delete it.
22:07 cotto_work ok
22:08 cotto_work I'm just wondering if you see something I don't.
22:08 kid51 If you delete, just msg me about it so that I know I'll handle the TT myself.
22:08 dduncan joined #parrot
22:08 * kid51 away
22:08 kid51 left #parrot
22:10 Yuki`N For 'creating a script creates a trac wiki page', the script doesn't have to actually do the HTTP post and all that, right?
22:10 cotto_work Yuki`N: no
22:10 cotto_work spitting out the markup is fine
22:11 Yuki`N Yeah.
22:11 Yuki`N That makes it easier.
22:27 Yuki`N Well that was rather easy.
22:27 Yuki`N Thank you Perl!
22:29 cotto_work perl ftw
22:29 Yuki`N I'm writing up some POD for good measure.
22:31 cotto_work Yuki`N: good.  I'd ask for some if I didn't see any.
22:31 cotto_work nobody'll use if if they don't know what it does
22:31 cotto_work It'd also be good to start a wiki page with the output of the script.
22:32 Yuki`N Is there a wiki page there?
22:32 Yuki`N Oh well I'll make one I guess.
22:33 cotto_work That's the spirit.
22:33 cotto_work make sure to document the name of the wiki page in the script
22:34 nwellnhof Yuki`N: making a HTTP post would be trivial in Perl. managing the credentials is more of a problem.
22:34 Yuki`N Ah right.
22:34 Yuki`N I'm used to full-fledged programming languages
22:34 simcop2387 left #parrot
22:34 Yuki`N i.e. anything with sockets is usually a pain.
22:35 nwellnhof Yuki`N: have you ever used LWP::UserAgent?
22:35 Yuki`N Nope.
22:35 Yuki`N How do I start a new wiki page?
22:35 Yuki`N Oh wow.
22:36 nwellnhof Yuki`N: give it a try. it will make you appreciate Perl5.
22:37 Yuki`N Hm I cannot edit the wiki apparently.
22:38 nwellnhof Yuki`N: someone has to give you edit permissions. i can't, unfortunately.
22:38 Yuki`N Oh.
22:38 cotto_work Coke: ping
22:38 cotto_work Yuki`N: what's your trac username?
22:38 Yuki`N Yuki`N
22:39 cotto_work If coke pongs, tell him you need a wiki editing bit.
22:39 M_o_C left #parrot
22:39 Yuki`N Ok.
22:39 Yuki`N Thinking of ways to make my output nicer.
22:41 hercynium left #parrot
22:50 cotto_work Yuki`N: the goal is to make it easy to figure out which comments are worth pursuing.  Anything that makes that as easy as possible will help.
22:52 Yuki`N I can try perhaps bolding comments with XXX in them.
22:52 Yuki`N Right now they're already sorted by directory, so it's easy to distinguish between say examples, tests, and actual source.
22:57 cotto_work Yuki`N: does the script handle multi-line comments correctly?
22:57 Yuki`N No, unfortunately.
22:57 Yuki`N It's purely line-based.
23:00 Yuki`N It's alleviated somewhat by an automatically generated github link, but I'm not quite sure how I can try to parse adjacent lines from a file given only perhaps the end of a multi-line comment.
23:00 Yuki`N The perl script itself is quite simple, maybe not even 6 lines of code.
23:01 cotto_work Don't feel like you need to make it perfect before sending a pull request.
23:02 Yuki`N Well I want to at least test it on trac and perhaps make some purely asthetic changes.
23:03 Yuki`N I just wish there was a quick and easy way to get lines xxx-yyy of a text file.
23:03 cotto_work split on a newline and index into the array?
23:05 Yuki`N Maybe.
23:05 Yuki`N Sounds slow though.
23:06 cotto_work The first version can be slow.  You can profile and optimize later.
23:07 dalek parrot/encapsulate-main: 0ee91bb | plobsing++ | compilers/tge/TGE/Compiler.pir:
23:07 dalek parrot/encapsulate-main: tag *every* tge sub :main. emulates old broken pick-first-sub behaviour
23:07 dalek parrot/encapsulate-main: review: https://github.com/parrot/parrot/commit/0ee91bbca1
23:09 dduncan left #parrot
23:10 simcop2387 joined #parrot
23:11 simcop2387 left #parrot
23:13 simcop2387 joined #parrot
23:17 cotto_work plobsing: how backwards-compatible of you
23:23 nwellnhof year end summary:
23:24 Coke cotto_work: pong
23:24 nwellnhof $ git diff --shortstat -M RELEASE_2_0_0..master -- src
23:24 nwellnhof 263 files changed, 76955 insertions(+), 48332 deletions(-)
23:24 Coke (wiki edit bit) what's the UID?
23:24 nwellnhof $ git grep ^ src |wc -l
23:24 nwellnhof 173422
23:26 nwellnhof (skewed by generated files)
23:27 vm joined #parrot
23:32 vm left #parrot
23:33 Yuki`N Coke, account name is Yuki`N, idk uid.
23:35 Coke The user Yuki`N has been granted the permission WIKI_MODIFY.
23:35 Yuki`N Thanks.
23:38 Yuki`N cotto, thanks for your suggestions, multi-line comments work now. :D
23:39 Coke perhaps a C programmer can fix the build for g++ with TT#1909
23:40 Yuki`N Coke, does WIKI_MODIFY also include new page creation?
23:41 Coke no.
23:41 cotto_work Coke: Could I get a trac admin bit so I can make these kinds of changes myself?
23:41 Yuki`N Can you grant that one to me as well? I need to create a new page for my script output.
23:42 Coke The user Yuki`N has been granted the permission WIKI_CREATE.
23:42 Yuki`N Thanks again.
23:42 cotto_work I have a partial bit.
23:42 Coke The user cotto has been added to the group admin.
23:43 cotto_work phenomenal cosmic power
23:43 cotto_work thanks
23:43 Yuki`N In a git merge conflict, the incoming changes are second, yes?
23:52 dalek tracwiki: v180 | Yuki`N++ | WikiStart
23:52 dalek tracwiki: Add hack list wiki page link.
23:52 dalek tracwiki: http://trac.parrot.org/parrot/wiki/W​ikiStart?version=180&amp;action=diff
23:52 dalek TT #1909 created by coke++: g++ build broken
23:52 dalek TT #1909: http://trac.parrot.org/parrot/ticket/1909
23:53 Yuki`N Coke, How do you force usage of g++?
23:54 NotFound Yuki`N: Configure.pl --cc=g++ --cxx=g++ --link=g++ --ld=g++
23:54 Yuki`N http://trac.parrot.org/parrot/wiki/ParrotHackList how does this look?
23:56 Coke NotFound: aren't you our g++ guy?
23:56 Yuki`N g++ build is pretty broken lol.
23:56 Coke (i'm trying to bisect where the build went sour for g++ now.)
23:56 Coke I'm assuming it was the merge back of embed_api (grumble)
23:57 Coke (mainly because I don't know enough C or C++ to fix the damn problem.)
23:57 NotFound Coke: sometimes
23:57 cotto_work Yuki`N: not bad.  It'd be nice if it highlighted "hack" and didn't catch things like "hackers"
23:57 cotto_work It's certainly enough to mark the task complete.
23:57 Coke ooh, looks like it was borked in 2_11_0
23:58 Coke btw, the skeleton needs to be updated to remove the Id$
23:59 Coke (skeleton.vim)
23:59 whiteknight joined #parrot

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

Parrot | source cross referenced