Parrot 2.7.0 "Australian King" Released! | parrot.org Log: irclog.perlgeek.de/parrot/today | Nopaste: nopaste.snit.ch:8001 | close 25 tickets (4 to go), merge branches; profile your favorite PIR for memory leaks with valgrind
Set by moderator on 6 September 2010.
00:01 ascent left 00:09 NotFound left 00:13 ascent joined
dalek TT #1621 closed by jkeenan++: Build errors on Windows 00:18
TT #1621: trac.parrot.org/parrot/ticket/1621
00:20 NotFound joined 00:42 tcurtis joined 00:47 chromatic left 00:49 plobsing2 is now known as plobsing 00:54 Psyche^ joined 00:56 dngor_ joined 00:59 Patterner left, Psyche^ is now known as Patterner 01:00 dngor left 01:04 Infinoid left, Infinoid joined 01:09 aloha left 01:10 bacek left 01:13 dngor_ is now known as dngor 01:16 Paul_the_Greek left 01:29 bluescreen joined 01:45 whiteknight left 01:47 JimmyZ joined
JimmyZ www.parrot.org/scratch/steigender-b...chinesisch . Is it a advertisement? 01:48
GeJ Oh, that's spam alright. 02:01
with an exploit on the top maybe? 02:02
JimmyZ I don't know,heh 02:03
02:06 JimmyZ left 02:21 GodFather joined
GeJ clock ? 02:23
purl GeJ: LAX: Mon 7:23pm PDT / CHI: Mon 9:23pm CDT / NYC: Mon 10:23pm EDT / LON: Tue 3:23am BST / BER: Tue 4:23am CEST / IND: Tue 7:53am IST / TOK: Tue 11:23am JST / SYD: Tue 12:23pm EST /
02:24 GodFather left 02:36 janus left 03:07 theory left
dalek rrot: r48817 | plobsing++ | trunk (3 files):
remove interp.save_func_table field
03:18
03:23 bluescreen left 03:25 bluescreen joined
dalek rrot: r48818 | plobsing++ | trunk/src/runcore/main.c:
remove unused static function notify_func_table
03:35
03:51 janus joined 04:11 patspam left
sorear #1163 spam 04:19
bacek_at_work sorear, I already deleted comment 04:21
dalek kudo: 8cf7fcd | pmichaud++ | src/Perl6/Actions.pm:
Switch 'for' statement modifier to use 'map' internally.
04:29
rrot: r48819 | plobsing++ | trunk (7 files):
allocate evc_func_table lazily and minimally
05:00
05:09 plobsing left 05:41 chromatic joined 06:02 fperrad joined, chromatic left 06:06 uniejo joined
dalek rrot: r48820 | NotFound++ | trunk/src/pmc/stringbuilder.pmc:
missing return in StringBuilder get_string!!!
06:07
06:14 he joined, cotto joined
cotto ~~ 06:14
msg dukeleto trac.parrot.org/parrot/wiki/GitMigration - edit away! 06:31
purl Message for dukeleto stored.
06:32 he left
dalek tracwiki: v7 | cotto++ | GitMigration 06:40
tracwiki: add rough timeline and prereqs for review
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
06:44 he joined 07:12 tcurtis left 08:14 NotFound left 08:15 NotFound joined 08:30 bacek joined
dalek p-rx: 237bf91 | moritz++ | src/Regex/P6Regex/Grammar.pm:
introduce "Quantifier quantifies nothing" error message
08:33
08:33 aloha joined
smash mornin' everyone 08:58
09:07 M_o_C joined 09:17 aloha left 09:18 bacek left 09:28 mikehh left 09:50 fperrad left 09:55 bacek joined 10:01 aloha joined 10:08 jsut joined 10:13 jsut_ left, smash left 10:14 smash joined
bacek aloha, humans 10:31
10:40 fperrad joined 10:58 mikehh joined 11:11 M_o_C left 11:32 contingencyplan left 11:49 M_o_C joined 11:56 mariano__ joined
Coke . 12:00
12:01 bkuhn joined 12:04 dmalcolm joined 12:10 whiteknight joined
Coke Who cleaned up the trac spam? 12:21
Thanks, and : trac.parrot.org/parrot/wiki/TracSpammers . though I suppose it's just more work for us, I do like keeping the list. 12:22
four tickets remaining to hit our goal tomorrow. 12:43
dalek TT #113 closed by rblasch++: [PATCH] sal.h is not included in Visual Studio.NET 2003 12:46
TT #113: trac.parrot.org/parrot/ticket/113
moderator Parrot 2.7.0 "Australian King" Released! | parrot.org Log: irclog.perlgeek.de/parrot/today | Nopaste: nopaste.snit.ch:8001 | close 25 tickets (3 to go), merge branches; profile your favorite PIR for memory leaks with valgrind 12:47
Coke three! three tickets, ha ha ha. 12:51
moritz++ (are you a bot?) 12:52
moritz Coke: hey, rblasch++ closed the ticket, not me :-)
szbalint if the trac timeline would appear on irc that would make the process easier :) 13:02
13:06 aloha left, bacek left
Coke dalek is doing at least part of that, szbalint 13:17
szbalint gah, I'm blind. 13:19
:)
13:35 M_o_C left 13:45 dmalcolm left
dalek TT #1764 reopened by nwellnhof++: Infinite loop in exit handler 13:54
TT #1764: trac.parrot.org/parrot/ticket/1764
13:54 janus left
Coke was that one of the 21 already closed this week? 13:55
moritz I think so
Coke bummer. 13:56
14:00 uniejo left 14:06 Andy joined 14:07 ttbot left 14:09 ttbot joined 14:13 janus joined 14:19 he left 14:23 M_o_C joined 14:33 M_o_C left, M_o_C joined
dalek p-rx: 24b297f | moritz++ | src/Regex/P6Regex/Grammar.pm:
report unrecognized backslash sequences
14:33
mikehh t/op/integer.t and t/pmc/bigint.t FAIL in make corevm/make coretest - error:imcc:loadlib directive could not find library `sys_ops' 14:44
all other tests PASS (pre/post-config, make corevm/make coretest, test, fulltest) at r48820 - Ubuntu 10.04 i386 (g++)
for some reason this does not seem to happen on amd64, but it fails on i386 with gcc/g++ with or without --optimize 14:45
but only in coretest before sys_ops are built 14:46
switching to amd64 to do further testing 14:48
14:49 plobsing_ joined 14:57 mikehh left 15:03 ash_ joined 15:06 luben_work joined 15:09 mikehh joined 15:25 mariano__ left
NotFound mikehh: Is corevm supposed to build dynops? 15:26
15:26 nwellnhof joined 15:32 theory joined
mikehh NotFound: not as far as I am aware, but the tests pass on amd64, so I am confused 15:34
NotFound mikehh: most probably the test is wrong. 15:35
mikehh NotFound: and again it passes make test, just not make corevm/make coretest on i386 15:36
plobsing_ looking at the test, it only uses sys_ops to get MAX_INT and MIN_INT. why aren't these simply pasm include constants? 15:37
15:38 allison joined
mikehh plobsing: 32 bit/64 bit 15:40
moritz that pasm include file could be written at configure time 15:42
plobsing_ 32 bit machines get the constant as 2**32-1, 64 bit as 2**64-1
the op is still necessary when writting cross-platform PBC, but the constant is nifty (esp. in coretest) 15:43
dalek kudo: d84752d | KodiB++ | / (6 files):
Implemented Instants and Durations.

Signed-off-by: pmichaud <pmichaud@pobox.com>
15:45 jsut_ joined
dukeleto 'ello 15:48
mikehh hi dukeleto 15:49
15:50 jsut left
plobsing_ is also surprised that t/op/integer.t (which also uses sys_ops) isn't also failing. 15:51
dukeleto mikehh: howdy 15:52
purl salut, dukeleto.
NotFound The fix is easy: don't use sysinfo 15:53
Even better: deprecate sysinfo
15:53 mj41_ joined
ash_ anyone else ever notice that there is no --without-readline in parrot's config? 15:54
15:55 chromatic joined
ash_ or is it required by something? 15:55
NotFound I'd like better to not detect readline during config and not link it, using nci at runtime instead.
ash_ that could work too, but it currently doesn't check --without-readline, i'll try to patch that 15:56
NotFound wfm 15:57
15:58 mj41 left, mj41_ is now known as mj41
NotFound mikehh: it fails for me with make corevm ; make coretest in amd64 15:59
16:00 theory left, theory joined 16:03 SingAlong joined
SingAlong hi all 16:03
dalek tracwiki: v8 | dukeleto++ | GitMigration 16:04
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
SingAlong is episode-3 of the PCT tutorial fixed? The last time I used it I wasnt able to do the exercises or even run it (the auto-generated test file had problems). 16:05
anyone? 16:07
purl Somewhere, someplace, in some universe, somebody uses whatever you just asked about.
mikehh All tests PASS (pre/post-config, make corevm/make coretest, test, fulltest) at r48820 - Ubuntu 10.04 amd64 (g++) 16:08
SingAlong purl: was that for me? 16:09
purl singalong: no idea
mikehh NotFound: it should AFAICS, but doesn't for me - after make realclean, so I am not sure, let me test further
SingAlong purl: well did it work for you?
purl singalong: no idea
SingAlong purl: ah. thanks 16:10
purl sure thing SingAlong
SingAlong purl: did the episode-3 work for you? (until the end)
purl SingAlong: huh?
NotFound SingAlong: purl is a bot
SingAlong oh crap!
NotFound SingAlong: Did you opened a ticket about that? 16:11
SingAlong NotFound: nope! tcurtis said he'll check that. It was he who figured out that the test file auto-generated had the mistake.
NotFound SingAlong: better open the ticket, people sometimes forget things 16:12
SingAlong NotFound: sure. but now I forgot the error. I guess it's better to post it with the error in detail than just saying that the test file is wrong. So i'll start the tutorial tomorrow and doing it again 16:13
dukeleto SingAlong: thanks, we appreciate it
NotFound SingAlong: good. Maybe is already fixed.
SingAlong dukeleto: np :) love parrot. i almost did my own language (until the tutorial broke) :) 16:14
was thinking about the Parrot PIR BabySteps tutorial i found on the web. but many advised me to go with PCT instead or PIR, now i know why... dealing with registers etc 16:15
NotFound SingAlong: you can write a language both ways, using PCT or writing a custom lexer/parser. 16:16
With PCT you have the tools and the tutorial, with the custom way you must take care of all.
SingAlong NotFound: I used to do a lot of lex and yacc. But can that be used with PCT? or do I have to use NQP. (any docs would help) 16:17
NotFound SingAlong: the PCT way uses perl6 style grammars with nqp. Don't know if you can mix yacc into it without a lot of work. 16:18
SingAlong NotFound: oh just wanted to know if that's easier. I'll skip using yacc if it's a round-about method. 16:19
NotFound You can make a compiler in C with lex/yacc and generate pir, sure.
mikehh NotFound: my bad - it was pulling the files from an installed parrot on amd64
SingAlong NotFound: oh! thats nice! any doc? 16:20
on how to use lex/yacc to generate pir 16:21
dukeleto SingAlong: it is possible, but not recommended.
NotFound SingAlong: just by writing pir code as output.
dalek tracwiki: v9 | dukeleto++ | GitMigration
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
SingAlong dukeleto: not going to do that unless i try hard to figure out whats wrong with the ep-3 :) 16:22
NotFound: yacc and lex only output .c files. how do i force it to generate pir?
NotFound SingAlong: not sure it you can do it without ugly tricks. What I did was to avoid lex and yacc, and using a hand-coded recursive descent parser instead. 16:24
16:27 SingAlong left 16:29 nwellnhof left 16:37 shockwave left 16:43 davidfetter joined
cotto_work good morning, fellow humans 16:46
16:46 tcurtis joined
cotto_work 1001010101, bots 16:46
dalek kudo: 0d6d574 | moritz++ | docs/ChangeLog:
update ChangeLog
16:49
cotto_work dukeleto: ping
16:50 whiteknight left 16:54 jhelwig joined
dalek p-rx: 99fe6e2 | moritz++ | src/stage0/ (3 files):
update bootstrap files
16:54
dukeleto cotto_work: pong 17:14
cotto_work: we should pair on git-related stuff. 17:15
cotto_work +1
purl 1
cotto_work dukeleto: I saw your edits to the git migration page. Are you suggesting using both github and cgit? I like github because it avoides more work for osuosl 17:16
dukeleto cotto_work: cgit is very similar to gitweb. gitweb comes with git
cotto_work: what i am talking about is having the "canonical" git repos on git.parrot.org, and gitweb is a very easy way to access/view them over HTTP 17:17
17:17 whiteknight joined
particle cotto_work: do you have git commit messages working? 17:18
cotto_work Yes. I have a (very slow) demo site set up too.
dukeleto cotto_work: i am thinking that github is where people commit to, but we also have our "canonical" git repo on git.parrot.org, which get synced to the github mirrors via a cron job
17:19 contingencyplan joined
particle cotto_work: saw the demo site, didn't see commit messages. excellent. 17:19
i'd like to have git.parrot.org
dukeleto cotto_work: this will minimize our OSUOSL bandwidth
cotto_work What do you mean by "commit messages"?
particle bandwidth won't be a problem until we exceend 10GB/mo
dukeleto particle: that is the plan. git.parrot.org will be canonical and synced from github via a cronjob
cotto_work They appear when you hover over a commit link. 17:20
particle dukeleto: would you like git.parrot.org to redirect to github?
or to host our own cgit
parrot.org has been designed to host more projects than parrot
eg trac.parrot.org/parrot/... 17:21
however in practice we haven't done that
dukeleto particle: i would like git.parrot.org to be our own cgit/gitweb interface. I don't care which. 17:22
particle: the parrot github org has 2 repos already: parrot.git and pir.git, so I think we should plan on multiple repos
particle: cgit is faster/uses less CPU than gitweb, but has fewer features.
particle: gitweb is probably already installed on parrot.org, just a few conf vars need to be twiddled 17:23
cotto_work, particle: I already have the cronjob to sync github -> git.parrot.org written. It is very simple
particle osuosl prefer cgit 17:24
dukeleto particle: sounds good to me.
particle dukeleto: can you put in a request to support@osuosl.org?
dukeleto particle: sure. what exactly should I ask for? 17:25
particle subject parrot.org: install/configure gitosis/cgit
dukeleto particle: we don't need gitosis
particle oh, no, we don't.
dukeleto particle: no one needs to commit to git.parrot.org
particle right.
dukeleto particle: it will simple act as the canonical mirror
s/simple/simply/ 17:26
particle then explain the github mirroring setup, and ask for a cgit or gitweb setup.
dukeleto particle: ok.
particle and you can find them on #osuosl on freenode to discuss the ticket after it's submitted 17:27
so you might mention your freenode nick if they want to ping you
dukeleto particle: good to know
cotto_work Will git.parrot.org sync both ways or will it be read-only and get changes from github?
17:28 ruoso left
particle prefers two-way, if it's manageable 17:28
dukeleto cotto_work: read-only. it will only pull in changes from github
particle: two-way makes the mirroring a lot more involved
particle yes, i know, i just don't know what the state-of-the-art in git mirroring is 17:29
dukeleto particle: then you have to decide who wins if git.parrot.org diverges from github
cotto_work Then why have it?
particle it's mostly a branding/vanity thing
dukeleto I think many people are concerned that we host our "official" repos on github
particle let's be humble and go with github as canon
17:29 elmex_ joined
dukeleto The perl6 peeps seem to be happy with github as canon 17:30
particle we're free software, we take advantage of free services
cotto_work +1 to particle
moritz the nice things is that everybody has the full repo
dukeleto My $job mirrors all (17) of our github repos to our own servers, in case the bay area falls into the ocean
moritz if githubs decides to shut down their service, it's just a matter of changing URLs 17:31
17:31 elmex left
dukeleto The question boils down to: What is more likely to go down nowadays? parrot.org or github.com ? 17:31
whiteknight There's no reason why git.parrot.org can't redirect to the github page
17:31 elmex_ is now known as elmex
dukeleto whiteknight: agreed. We are decided what is wanted, though. 17:31
Does parrot want to have a read-only mirror on parrot.org ? We already can setup read-only mirrors other places, like git.or.cz and gitorioius 17:32
whiteknight I see no problem with read-only mirrors
dukeleto whiteknight: sure. But do we want to host our own, is the question.
cotto_work -1, mostly because it means more work for osuosl that we can't do ourselves 17:33
whiteknight yes, I should have been more specific: It would be good to host our own read-only mirror, if possible
dukeleto it is a small amount of one-time work. gitweb/cgit needs to get configured, and then a 5 line cronjob needs to run, which is already written.
whiteknight development can always happen offline with git. The more copies we have easily publically available, the more seemlessly we will be able to weather a github outage
cotto_work +0 then 17:34
dukeleto cotto_work: i am not sure that we need to ask osuosl to do it. Can't we do that stuff ourself? We just need them to add the git.parrot.org vhost
whiteknight Assuming that somebody has an up-to-date copy, we can do anything we want while github is down. We could cut a release if we wanted
dukeleto But I hear what y'all are saying. Github as canon is fine with me. 17:35
moritz wfm too
dukeleto whiteknight: we can setup easy read-only mirros on gitorious/git.or.cz, without asking OSUOSL for anything
whiteknight dukeleto: that said, The github feature set is what we want most, not necessarily the github URL
I think we could get our own copy of that code and run our own instance if we absolutely wanted to 17:36
of course we lose out on possibility for mass collaboration
moritz let's ask it the other way: is there are a good reason to host it ourselves?
dukeleto whiteknight: i am only taking about having a read-only mirror on git.parrot.org, not running our own instance of githuby things
moritz apart from NIH, of course :-)
cotto_work moritz++ 17:37
Coke given the amount of work that went into removing the appearance of dependence on perl.org, I have no desire to repeat that with our git server.
dukeleto moritz: I am a newbie. I want to get the official parrot repo. Where do I go?
moritz: git.parrot.org would be my first guess
moritz dukeleto: you start google, and type 'parrot git' 17:38
Coke dukeleto: I'd just google it.
moritz++
moritz dukeleto: and the first hit will either be a parrot wiki page pointing to the github repo
or the github repo itself
having a redirecting subdomain works for me too
(I'd have no problem with a custom hosting solution too, I just think it's more effort to maintain than people assume) 17:39
remember those spiders that follow every link, ask for every changeset, every possible blame at every possible revision?
whiteknight source.parrot.org 17:41
dukeleto moritz: having that as a redirect to github would be fine, though
whiteknight: source is more general, but longer to type ;)
whiteknight the most obvious thing is not always the shortest
17:41 bkuhn left
whiteknight I'm not going to bikeshed here, I'm just pointing it out 17:41
dukeleto We should attempt to paint the smallest bikeshed possible ;)
repo.or.cz/w/parrot.git
17:41 ash_ left, fperrad left, smash left, jan left, GeJ left, Hunger left, atrodo left, sjn left, spinclad left, knewt left, purl left, tcurtis left, theory left, mikehh left, M_o_C left, Andy left, cotto left, Util left, dukeleto left, pjcj left, luben left, PacoLinux left, nopaste left, perlite left, TimToady left, hudnix left, Tene left, KatrinaTheLamia left, elmex left, whiteknight left, jhelwig left, mj41 left, luben_work left, ttbot left, Infinoid left, x3nU left, esskar left, integral left, eternaleye left, p6eval left, bacek_at_work left, wagle left, jjore_ left, dafrito left, particle left, silug left, preflex left, szabgab left, cxreg left, dip left, slavorgn left, Khisanth left, zostay left, snarkyboojum left, athomason left, dzoe left, mikegrb left, frodwith left, Maddingue left, TonyC left, szbalint left, confound left, chromatic left, allison left, plobsing_ left, janus left, NotFound left, bluescreen left, dngor left, Patterner left, ascent left, dalek left, pmichaud left, PerlJam left, AzureSto_ left, s1n left, edenc left, autark left, baest left, rblackwe left, slavorg left, kthakore left, sorear left, estrabd left, jnthn left
davidfetter parrotsketch? 17:41
Coke in several hours. 17:42
little under 3.
davidfetter k
17:42 ttbot joined
davidfetter i guess purl got netsplit or something 17:42
Coke how many tickets left to close before then?
oh. 17:43
missed that. ;)
cotto_work 3 according to the topic, though there's at least one I still need to review
Coke particle - I can't login to trac.parrot.org
betting the overeager cache is back.
Coke tries to find osu. 17:44
17:45 jnthn joined, estrabd joined, sorear joined, slavorg joined, rblackwe joined, baest joined, autark joined, edenc joined, kthakore joined, s1n joined, AzureSto_ joined, PerlJam joined, pmichaud joined, dalek joined, ascent joined, Patterner joined, dngor joined, bluescreen joined, NotFound joined, janus joined, plobsing_ joined, allison joined, chromatic joined, ruoso joined, knewt joined, spinclad joined, sjn joined, atrodo joined, Hunger joined, GeJ joined, jan joined, smash joined, fperrad joined, ash_ joined, elmex joined, whiteknight joined, jhelwig joined, tcurtis joined, theory joined, mj41 joined, mikehh joined, luben_work joined, M_o_C joined, Andy joined, cotto joined, Infinoid joined, x3nU joined, esskar joined, Util joined, dukeleto joined, integral joined, eternaleye joined, pjcj joined, luben joined, PacoLinux joined, p6eval joined, bacek_at_work joined, nopaste joined, wagle joined, jjore_ joined, dafrito joined, perlite joined, particle joined, silug joined, preflex joined, TimToady joined, szabgab joined, hudnix joined, cxreg joined, Tene joined, dip joined, KatrinaTheLamia joined, slavorgn joined, Khisanth joined, zostay joined, purl joined, mikegrb joined, snarkyboojum joined, TonyC joined, dzoe joined, Maddingue joined, frodwith joined, szbalint joined, athomason joined, confound joined 17:46 plobsing_ left, plobsing_ joined
cotto_work It's a good thing I love brittle protocols. 17:46
chromatic Oh, set_bool. No idea. 17:47
dukeleto will be back in a few 17:48
Coke pinged osuosl folks about trac.parrot.org login. 17:50
anyone able to login?
atrodo Coke> I was able to 17:51
cotto_work wfm 17:52
18:09 davidfetter left 18:10 tadzik joined
dalek rrot: r48821 | NotFound++ | trunk/t/pmc/orderedhashiterator.t:
test OrderedHashIterator clone
18:15
dukeleto is back 18:26
dukeleto updated the git migration plan, removed git.parrot.org stuff, for now 18:28
it can be setup at any time, and isn't necessary for the svn -> git switch
mikehh: i added you as a committer to github, as well. 18:31
18:32 nwellnhof joined
dukeleto migrates. be back soon. 18:33
cotto_work clones and forks dukeleto
Coke thanks dukeleto for taking over the git documentation. 18:34
Coke volunteered back in the dark ages.
cotto_work dukeleto++ for git docs
18:38 fperrad left
dalek tracwiki: v10 | dukeleto++ | GitMigration 18:38
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
TT #1774 created by ash++: Adding without-readline support 18:44
TT #1774: trac.parrot.org/parrot/ticket/1774
TT #1749 closed by nwellnhof++: readall method on open FileHandle is inefficient
TT #1749: trac.parrot.org/parrot/ticket/1749
18:46 tadzik left
ash_ i submitted a patch to the trac for --without-readline for the Configure.pl script 18:47
tcurtis nwellnhof: does r48824 create the right encoding for the produced string? When I tried a change that was, iirc, about the same as yours, it failed some tests due to wrong encodings.
nwellnhof tcurtis: yes, see the previous commit 18:48
dalek rrot: r48822 | plobsing++ | trunk/compilers/imcc (3 files):
remove imc_info_t.allocated (was always 0)
18:49
rrot: r48823 | nwellnhof++ | trunk (2 files):
[pmc] Various fixes to StringBuilder PMC

  - Use Parrot_str_clone again, it should be safe now
  - Keep encoding if all strings have the same encoding
rrot: r48824 | nwellnhof++ | trunk/src/pmc/filehandle.pmc:
[pmc] Use StringBuilder in FileHandle.readall
nwellnhof the problem was that concatenating utf8 strings wouldn't necessarily produce an utf8 string 18:50
the string_rep_compatible logic was simply *too* smart 18:51
cotto_work That doesn't sound like "smart". 18:52
nwellnhof it does the right thing. but it can be surprising that the concatenation of utf8 strings produces an ascii string. 18:55
dukeleto is back 18:56
pmichaud nwellnhof: your patch fixes the readall problem, but what about the general problem of concatenating strings?
are long string concatenations always doomed to be slow in Parrot?
nwellnhof pmichaud: that's what StringBuilder is for
pmichaud nwellnhof: you expect a HLL compiler to figure out when there are long sequences of string concatenations and switch to StringBuilder?!
dukeleto cotto_work: no worries. I am enjoying it 18:57
nwellnhof pmichaud: it's a consequence of the immutable string changes. that wasn't my idea but i think immutable strings are a good thing. 18:58
pmichaud I think immutable strings are a good thing also.
My question is about GC.
It's the GC that causes the problem.
the example program I gave (before your patch) resulted in 1700+ mark-and-sweep runs. 18:59
nwellnhof not really, the actual problem is that str_concat has to allocate a new string every time.
chromatic Are you sure my patch to that caused the problems? I can't reproduce it.
nwellnhof that results in a lot of GC pressure, of course.
pmichaud nwellnhof: and lots of large string allocs result in many mark runs, apparently. 19:00
chromatic: I had a fresh checkout -- I can try again, if you wish.
NotFound Conatenating two string to a new string causing the allocation of a new string is no big surprise.
dukeleto cotto_work: what needs doing with github-trac? Are there any time-consuming things that need to be done close to the actual transition?
nwellnhof pmichaud: the old readall code allocated a string of average length (0.5 * filesize) for every line read. 19:01
pmichaud NotFound: but the 1740+ mark runs is way too expensive, especially when we have to mark-the-world.
dalek TT #1771 closed by NotFound++: Cannot hll_map in parrot HLL
TT #1771: trac.parrot.org/parrot/ticket/1771
chromatic I'm happy to argue that running the GC when a fixed-sized pool is full (or too full) is a bug.
That seems like a recipe for quadratic disaster.
pmichaud ...which is what we saw. 19:02
chromatic Exactly.
NotFound Wasn't that fixed?
pmichaud nwellnhof: I'm not arging against your patch, I'm arguing that it doesn't solve the underlying problem (for Rakudo)
chromatic I deleted a lot of code.
cotto_work dukeleto: the code that handles post-receive hooks needs to be tested, but my additions there are trivial.
chromatic There might be a 64-bit problem there, but I'm not sure.
nwellnhof pmichaud: the old code allocated 2.5 GB for reading unixdict.txt if my quick calculation is right 19:03
pmichaud nwellnhof: that doesn't surprise me.
cotto_work Installing the plugin requires the output from git log. Slurping the log into the db shouldn't take more than 30s.
pmichaud does readall still do a line-by-line read?
nwellnhof pmichaud: yes
pmichaud ...why?
tcurtis Could it be rewritten to use Parrot_io_reads like the unopened case? 19:04
nwellnhof pmichaud: i don't know. it probably shouldn't
pmichaud I think it should just do reads in, say, 4k byte chunks until exhaustion.
then for unixdict.txt instead of doing 25000 concatenations we end up doing 50
(or, if it's using StringBuffer, intead of reading in 25000 strings we read only 50) 19:05
chromatic line-by-line reading here is definitely suboptimal.
nwellnhof pmichaud: the funny thing is that readall does that if reads from a filename
whiteknight memory map the file, memcopy into a buffer, viola
chromatic My memcopy if we have mmap? 19:06
pmichaud this case is the one of reading from an already opened filehandle
the filehandle may be something that can't mmap
whiteknight well, if you want to assume that we're not going to close the mmap when we close and destroy the filehandle, that's fine
pmichaud I already switched rakudo's slurp() function to use the readall('filename') approach, that gave us a 30x speedup on slurp()
but that doesn't help with .slurp on already-opened-file 19:07
moderator Parrot 2.7.0 "Australian King" Released! | parrot.org Log: irclog.perlgeek.de/parrot/today | Nopaste: nopaste.snit.ch:8001 | close 25 tickets (2 to go), merge branches; profile your favorite PIR for memory leaks with valgrind 19:07
nwellnhof there are definitely a lot of possibilities for optimizing the IO code 19:07
dukeleto we have 1.5hrs left to close 2 tickets before #ps, to meet our goal 19:08
cotto_work: do we have an instance of the github-trac plugin running somewhere already?
cotto_work dukeleto: mksig.org/trac/wiki/WikiStart (super slow though) 19:09
running on postgres, same as trac.parrot.org
ash_ if someone wants to apply the patch from ticket 1774 that might be a quick ticket to close :P, or if i need to add more to the patch/make a better patch, let me know 19:10
dalek rrot: r48825 | moritz++ | trunk/ext/nqp-rx/src/stage0 (3 files):
[nqp-rx] update bootstrap files from (sha1 99fe6e22); adds improved error reporting for regex parsing
19:12 cotto_work left, cotto_work joined
dukeleto cotto_work: have you contacted the upstream author of the github-trac plugin at all? Is he open to merging your changes? 19:13
cotto_work: and do we care about having those changes merged upstream before converting parrot from svn to git?
cotto_work I haven't contacted him yet. I would prefer to have the changes merged upstream. 19:14
19:15 tadzik joined
dukeleto cotto_work: " * this will require the output from git log, as documented in the README for that plugin " 19:15
19:15 Paul_the_Greek joined
cotto_work I'll send him a message berore #ps. He's active on github (though not on the plugin atm) so he should see it soon. 19:15
*before
dukeleto cotto_work: how long will that step take?
cotto_work++
Paul_the_Greek Afternoon, Parrotheads.
dukeleto Paul_the_Greek: good localtime()
Paul_the_Greek The time of day is to be shifted appropriately in each reader's brain. 19:16
cotto_work merging upstream? It depends on davglass (the current maintainer)'s comments.
Don't consider it a blocker though. 19:18
dukeleto cotto_work: no, how lnog with the "output from git log" step take? 19:20
s/lnog/long/
cotto_work 20 seconds +/- 19:22
actually, it looks more like 3 on my local machine 19:23
luben hello everyone 19:24
what functions are subject of deprecations - those marked PARROT_EXPORT or those marked PARROT_API? 19:25
cotto_work That said, there's a fairly large difference between the number of commits that git log returns and the number of svn revisions we have.
API only 19:26
luben nice
dukeleto cotto_work: awesome. just making sure it is not going to take hours or something 19:31
cotto_work: git log operates on master by default 19:32
cotto_work: what kind of difference are you seeing?
19:32 mj41_ joined
cotto_work 39764 lines containing git-svn-id, whereas we have ~48800 commits. 19:33
dukeleto cotto_work: that could be something to do with branches and tags 19:35
cotto_work: and i don't think svn started on rev 1, since it was converted from CVS
cotto_work: but that is a guess 19:36
cotto_work we have a git-svn-id line for r1
19:36 mj41 left, mj41_ is now known as mj41
dukeleto cotto_work: does github-trac know about branch names? 19:37
cotto_work: what happens if a sha1 prefix is ambiguous?
cotto_work currently it'll pick the first one 19:38
no
I sense that perhaps I'm not as done as I'd thought.
dukeleto cotto_work: i wrote something very similar for a ticket tracking system at my $old_job 19:40
cotto_work: it would accept things like repo:branch or repo:sha1
cotto_work Oh good. Then you'll be familar with the pitfalls.
dukeleto cotto_work: yes, i am familiar with some of the pitfalls 19:41
cotto_work: what is the minumum length sha1 that you accept? i found 5 was reasonable
cotto_work: if not, you will be checking lots of english words that could maybe be a SHA1
cotto_work 5 19:42
Paul_the_Greek Like deadbeef? 19:43
cotto_work It'll check that and ignore it when it doesn't find a matching git id.
dukeleto Paul_the_Greek: yes, but deadbeef is >5 chars :) 19:44
Paul_the_Greek Okay, then just dead and beef. 19:46
cotto_work too short
purl too short is bad as well
dukeleto Paul_the_Greek: yes, that is the point of requiring >= 5 chars. 19:47
cotto_work: i just made the gitmigration wiki page more specific. please add any bullet points that you can think of under the github-trac plugin 19:48
cotto_work Hamburger discussions will not be adversely affected.
Paul_the_Greek Why would you be confused that a sha1 might be a word? 19:49
dukeleto cotto_work: we need a better definition of "well-tested"
cotto_work s/better //
dukeleto Paul_the_Greek: this is the situation. You have a wiki page with 5000 words on it. We only check words >= 5 chars to see if they are SHA1's
Paul_the_Greek Why do you check that at all?
dukeleto Paul_the_Greek: it is expensive to see if something is a SHA1, usually. Unless it is cached
cotto_work It makes me sad that trac extensions don't come with their own built-in tests like perl 5 modules do. 19:50
dukeleto Paul_the_Greek: to autolink SHA1's to the commit that refer to
Paul_the_Greek: like the way trac links r1234 to the changeset of r1234
Paul_the_Greek I don't have to code something around the sha1 to say it's a sha1?
Oh, right, like that.
cotto_work Paul_the_Greek: they can appear anywhere in a body of text.
dalek tracwiki: v11 | dukeleto++ | GitMigration 19:51
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
tracwiki: v12 | dukeleto++ | GitMigration
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
tracwiki: v13 | dukeleto++ | GitMigration
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
Paul_the_Greek So why not have to write at least 'sdecaf' ?
dukeleto Paul_the_Greek: what ?
Paul_the_Greek Why not some indicator like the 'r' ?
cotto_work That's for svn revisions. 19:52
Using it for both would add ambiguity to number-only git ids.\\
Paul_the_Greek Right, but why not something similar for sha1s?
Like an 's'.
cotto_work laziness on the part of the implementer ;) 19:53
tcurtis People are used to just saying the sha1 for git sha1s and it's easier to handle them without a prefix that train everyone to use a prefix, possibly.
dukeleto tcurtis++
Paul_the_Greek Hmm.
dukeleto Paul_the_Greek: don't bikeshed unless you want write the code and train everyone ;) 19:54
cotto_work Allowing git ids directly also seems to be the least surprising thing to me. Other people may be differently surprised.
Paul_the_Greek It's good to live in an age where trying to save the lookup of every word >= 5 letters composed of a--f is considered bikeshedding. 19:56
dukeleto It allows people to cut and paste emails that refer to SHA1's and all kinds of other use cases. It also allows copy+paste into git commands.
Paul_the_Greek: yes, I agree.
Paul_the_Greek I hereby renounce the fact that I grew up in the late '60s and '70s and throw efficiency to the wind.
All hail naked SHA1s! 19:57
dukeleto Paul_the_Greek++
cotto_work Welcome to the 21st century.
We have fast CPUs and we know how to keep them busy.
Paul_the_Greek You mean the century where all my spare cycles go to running dynamic icons? Huzzaaa!
dukeleto ponders making a film called "SHA1's gone wild"
nwellnhof paul: there aren't many words with more than 5 letters consisting only of letters A-F 19:58
Paul_the_Greek Ooh, let's find out how many ...
cotto_work smells a perl script in the making.
luben chromatic, ping
nwellnhof git hashes also have at least 7 characters, AFAIK 19:59
dukeleto nwellnhof: you are wrong about that
nwellnhof: any unique prefix works
nwellnhof: 7 characters will usually be unique for smallish repos, though
The number of characters needed, on average, grows logarithmically with the number of commits in the repo, if anyone cares. 20:00
pmichaud chromatic: I tried that patch from the mailing list again -- same result.
cotto_work i.e. slowly
dukeleto how does parrot-commits happen, currently? 20:01
cotto_work Oh. Good question.
purl Yeah, it is. I'm stumped.
Coke dukeleto: happen?
purl /kick Coke enough with the aybabtu already
Coke presumably a post-commit hook. 20:02
probably right in the trac guts.
dukeleto Coke: just wondering how we will do parrot-commits in the github age. I added it to the migration wiki page
nopaste "nwellnhof" at 192.168.1.3 pasted "grep '^[a-f]*$' unixdict.txt" (66 lines) at nopaste.snit.ch/23252
cotto_work thanks, nerd 20:03
;)
Coke dukeleto: one possible option: don't. 20:04
(just have people sub to the RSS commit feed.)
pmichaud RSS feed seems reasonable
cotto_work I like getting it by email.
pmichaud github allows commits to go to a mailing list 20:05
it's, like, easy.
dukeleto Coke: yep, that is possible. It is simple to get emails sent, but I think it is slightly more challenging to get commit diffs emailed
Coke so do I, but it's not worth any time on my effort to support. ;)
Paul_the_Greek There are only 33 words! What the hell was I worried about?
dukeleto pmichaud: commits yes, commit diffs, not so much
pmichaud: but the link to the diff is in the email, of course
pmichaud ah, yes, it doesn't include the diff.
Paul_the_Greek You could allow four letters and it still doesn't much matter. 20:06
dukeleto facade and decade <-- high class deadbeef cousins
Paul_the_Greek acceded, defaced, effaced
bedded should be useful, too. 20:07
Coke abacab
pmichaud abba
Paul_the_Greek Yagh.
nwellnhof not to forget 0xcafebabe
dalek tracwiki: v14 | dukeleto++ | GitMigration 20:08
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
cotto_work is surprised not to have seen a place called the Ox Cafe.
Paul_the_Greek Or the joint run by the confused brother-in-law: 0cafe 20:09
20:10 whiteknight left
Paul_the_Greek 0xc0ffeebabe 20:10
GeJ Bonjour everyone. 20:11
Paul_the_Greek Howdy GeJ.
20:11 bluescreen left
dukeleto would love if some peeps took a look at github.com/parrot/parrot/blob/leto%...nology.pod and gave feedback 20:13
20:14 bacek joined
dukeleto bacek: GOOD DAY, HUMAN MEAT 20:15
Paul_the_Greek dukeleto: "SHA1 sum" should be "SHA1 hash"? 20:16
moritz it's a "check sum"
GeJ G'Day bacek.
moritz so "sum" isn't all that bad
20:16 shockwave joined
GeJ Servus moritz. 20:16
shockwave pmichaud: ping
moritz hi GeJ 20:17
pmichaud shockwave: pong
Paul_the_Greek moritz: But doesn't it make you think that it is the sum of something?
dukeleto Paul_the_Greek: SHA1 hash seems to be much more common, thanks
moritz Paul_the_Greek: it is
Tene Paul_the_Greek: also notice that the command-line tool used to verify that is named "sha1sum" 20:18
dukeleto Paul_the_Greek: i hear a lot of people use SHA1 sum, but hash seems to be the more common phrase
Paul_the_Greek Standing for "secure hash algorithm" after all.
shockwave pmichaud: Hey. I gave some thought about that field caching thing we were talking about yesterday. Basically,
Tene and, fwiw, the algorith *does* involve quite a few additions. 20:19
chromatic luben, pong
Paul_the_Greek Should have been called 'verifysha1'.
Tene Sure.
20:19 ash_ left
pmichaud maybe it's really "sha1 <sumthing>" :-P 20:19
Paul_the_Greek True, but then we could also call it sha1 xor. 20:20
Tene "sha1sum" is already quite well-established. Using "sum" in that sense is not going to confuse anyone.
Paul_the_Greek: :)
dukeleto is it 10mins to #ps already?
luben chromatic, I have finished the hash refactor to inline hash functions and compare functions.
shockwave pmichaud: The $P0 = assign $P1 ; works really nice, because not only does it stop the need to box a value prior to assign it to a field, which is obviously expense, but it also has the side effect that it works when a field is cached, another method is called and that field is set, and the method returns. But,
chromatic How was it?
Paul_the_Greek Wait, are there XORs? Maybe sha1 shift.
luben 2-3% speedup
it's worth 20:21
chromatic Definitely. 20:22
shockwave Fields of non-scalar types break the whole deal. There are infinite ways in which those fields are can be changed. Every time a function is called, those types of fields have to be invalidated, unless one has a very detailed call-graph as to who changes whom.
pmichaud shockwave: correct.
20:22 aloha joined
luben coretest passes but I have to chase more bugs in make test 20:22
20:23 tadzik1 joined
shockwave pmichaud: Some caching can still be possible. I guess something is better than nothing. And that something may turn out to be substantial in some cases. 20:23
chromatic luben, is this worth doing in a public branch? 20:24
luben yes, I will try to create one in the svn
20:25 cognominal left, cognominal joined
luben here is my working repo if you are interested in the code: luben.spnet.net/gitweb/?p=parrot/.git;a=summary 20:26
dukeleto luben: do you have a gitub id? 20:27
20:27 tadzik left
luben let me see 20:28
pmichaud have to run here... bbiaw
tadzik1 how is threading in Parrot going? I think I remember someone working on it for GSoC, is it soon-to-be-working in Parrot?
chromatic #ps in 1 20:29
tcurtis Chandon implemented green threads, but I think actual parallel execution is blocking on deciding how to approach data sharing and synchronization. 20:30
luben I have just created github account
chromatic wants to change all of the PackFile functions to use STRINGs, not the random blobs of memory C thinks are strings. 20:32
dalek kudo: 19a8b71 | pmichaud++ | src/core/Any-list.pm:
Add Match workarounds for .[] and .{} to (temporarily) allow match objects to do something sane. Addresses RT #75868.
20:39
dukeleto luben: what is your github username? 20:40
luben: have you sent a CLA?
chromatic luben sent his CLA and has his commit bit. 20:41
luben dukeleto, username: luben, CLA: yes,
dukeleto luben: have you gotten a commit bit to svn yet? just wondering 20:42
luben chromatic, the refactor passes make test again.
dukeleto, yes, since yesterday
chromatic Fantastic.
dukeleto luben: awesome. 20:43
20:44 tadzik1 is now known as tadzik
dukeleto luben: ok, you have a commit bit on github now 20:44
luben dukeleto, thanks 20:46
NotFound Someone fluent with blizkost source here? 20:51
20:53 M_o_C left
tcurtis ns_func_cleanup was merged in r47678. 20:55
mikehh t/op/integer.t and t/pmc/bigint.t FAIL in make corevm/make coretest - error:imcc:loadlib directive could not find library `sys_ops'
all other tests PASS (pre/post-config, make corevm/make coretest, test, fulltest) at r48825 - Ubuntu 10.04 amd64 (g++ with --optimize)
moderator Parrot 2.7.0 "Australian King" Released! | parrot.org Log: irclog.perlgeek.de/parrot/today | Nopaste: nopaste.snit.ch:8001 | close 25 tickets; merge branches; review Git conversion plan 20:56
tcurtis msg whiteknight ns_func_cleanup appears to have been merged into trunk at r47678. Can it be deleted? 20:56
purl Message for whiteknight stored.
aloha OK. I'll deliver the message.
dukeleto cotto_work: i just added some import stuff to the migration wiki 20:58
s/import/important/
dalek rrot: r48826 | tcurtis++ | branches/pct_multi_support:
Delete branch pct_multi_support, which was merged into trunk at r46785.
cotto_work so you did 20:59
tcurtis purl, msg tcurtis Remember to address a specific bot with msgs until purl goes bye-bye.
purl Message for tcurtis stored.
dalek tracwiki: v15 | dukeleto++ | GitMigration
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
dukeleto cotto_work: we need to make sure basic things in tools/ know about git
cotto_work: and the release manager guide is important to have done as well 21:00
cotto_work very good catch 21:02
dukeleto++
21:02 dafrito left
Coke an svn branch to work on the git-support would be a good idea. 21:03
21:04 dafrito joined
dukeleto Coke: i am working in git branches, because they will be able to get merged after the switch. Not so much for svn branches. 21:05
Coke hokay 21:07
dukeleto Coke: but your point is taken
cotto_work We won't be able to merge current svn branches after the git migration?
That sounds important.
dukeleto cotto_work: they are all already git branches
cotto_work and those can be merged fine? ok
dukeleto cotto_work: yeah, i guess you are right. svn branches are all in git, so what I just said was wrong 21:08
chromatic Whew.
dukeleto meant to say "i am going to work in git branches, because I loath svn"
cotto_work ditto
nwellnhof tcurtis: just the "definitely lost" reports for ./perl6 -e -1 21:09
21:15 whiteknight joined
dalek rrot: r48827 | chromatic++ | trunk/src/packfile.c:
[pf] Fixed some compiler warnings.
21:15
tcurtis nwellnhof: so, leave out the errors it reports?
chromatic Please do. 21:16
nopaste "tcurtis" at 192.168.1.3 pasted "valgrind --leak-check=full ./perl6 -e -1" (48 lines) at nopaste.snit.ch/23253 21:17
chromatic Only two leaks there then.
tcurtis was treating lots of leaks in terms of the amount of memory leaked. 21:19
That second leak would account for most of the leaking that showed up in the spectest run.
chromatic I'm not seeing those leaks. 21:20
The only leak I see is the weird Valgrind/dlerror leak.
Paul_the_Greek Is github.com/parrot our repository? 21:21
cotto_work dukeleto: we should also make sure that the first person to cut the release after our git migration will be knowledgeable enough to deal with any holes in the release manager guide. 21:22
nwellnhof i'm not seeing any leaks for 'perl6 -e -1' either
on 32bit debian
cotto_work github.com/parrot is our organization 21:23
dukeleto cotto_work: yes, 1st release manager after the switch should be very confident with git
Paul_the_Greek cotto_work: Should we all have an account there? 21:24
cotto_work yes 21:25
Paul_the_Greek What would my password be? Why do I only see 5 members?
cotto_work sign up for github and dukeleto will add you to the Parrot org once he knows your github username
nwellnhof dukeleto: how can i find out if i have a commit bit on guthub?
Paul_the_Greek So I just create one of the free accounts? 21:26
cotto_work yup 21:27
pay your $0 and you're good to go
21:27 tadzik left
tcurtis Valgrind says that it is searching for pointers to the not-freed blocks. Is there any reason why, if Rakudo did skip global destruction, that wouldn't find any memory that would have been freed if Rakudo hadn't skipped global destruction? Is Parrot doing some pointer magic that will confuse valgrind? 21:28
Paul_the_Greek msg dukeleto My account name on github is Paul-C-Anagnostopoulos. (Note that doesn't exactly match my Parrot name.) 21:29
purl Message for dukeleto stored.
aloha OK. I'll deliver the message.
nwellnhof tcurtis: afaik, the memory not freed by global destruction would only show up under 'possibly lost' 21:30
dukeleto nwellnhof: when you look at github.com/parrot/parrot, if you are a committer, you will see a SSH clone URL
nwellnhof tcurtis: the real memory leaks are the ones 'definitely lost'. they should show up regardless of global destruction. 21:31
dukeleto nwellnhof: i just added you
Paul_the_Greek Oh, dukeleto, I just messaged you.
21:31 kid51 joined
dukeleto Paul_the_Greek: you have been added 21:31
nwellnhof dukeleto: i see it on github.com/parrot
thanks
dukeleto nwellnhof: no worries 21:32
nwellnhof dukeleto++ # for all the git work
Paul_the_Greek Do you think there will be any confusion due to different usernames on Parrot/svn and git?
dukeleto Paul_the_Greek: we may need to fiddle with AUTHORS for karma to work correctly 21:33
tcurtis chromatic: gc_threshold_tuning appears to have been merged at r48485, can it be deleted now?
chromatic Go ahead. 21:34
Paul_the_Greek I was thinking that things with "Paul C. Anagnostopoulos" in the svn repository won't match up with "Paul-C-Anagnostopoulos" in git.
kid51 tcurtis: My approach is: delete branches immediately after merging. That way, you're not tempted to do further revisions in branch. 21:35
21:35 perlite left
nwellnhof tcurtis: yes, gc_threshold_tuning can be deleted 21:37
tcurtis kid51: seems sensible, but I'm hesitant to delete branches, even if they are merged into trunk and haven't since been worked on, without making sure that there's not some reason that they were kept around. 21:38
dalek rrot: r48828 | tcurtis++ | branches/gc_threshold_tuning:
Delete branch gc_threshold tuning, which was merged in r48485.
21:39
21:40 perlite joined 21:42 ash_ joined 21:45 ash_ left
chromatic Hm, the NCI thunk generator seems to have a memory leak. 21:50
Or at least generates potentially leaky code.
21:51 Paul_the_Greek left
dalek tracwiki: v16 | dukeleto++ | GitMigration 21:54
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
chromatic See, for example, the Parrot_str_to_cstring calls in pcf_p_tiB3P. 21:56
If something updates the char pointer in place in the called function, kablammo.
kthakore chromatic: or more like 'sqish' .... I thought you were talking about a mem leak 22:01
chromatic That too. 22:03
kthakore so sqish_blammo
chromatic: oh hai btw
chromatic hello
kthakore chromatic: how is life? 22:04
chromatic Busy! Giving a talk tomorrow at PDX.pm on Modern Perl.
kthakore ooh! Good Luck!
Not that you need it! 22:05
tcurtis dl.dropbox.com/u/1277662/reallycleanbts is the backtraces from the Rakudo spectest run under valgrind if anyone else wants to look at them. That file is 705K and contains only the backtraces and the line in front of each backtrace that describes how much was lost.
kthakore tcurtis: how much is the total lost?
chromatic Is that 64-bit Parrot trying to load a 32-bit PBC? 22:07
dukeleto THE HORRRO 22:08
THE HORROR, even
dukeleto slaps forehead
tcurtis I don't think so. I did build gen/core.pir on my 32-bit OS X and transfer it to the 64-bit linux vm, but I'm fairly certain I did all compilation to pbc in the VM. 22:10
dalek tracwiki: v17 | cotto++ | GitMigration 22:11
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
22:12 ash_ joined
chromatic Aha. 22:13
NotFound Done my first pull request in github. I'm near the "master of git" status now ;)
tcurtis kthakore: average of about 132k lost in main, iirc. The total amount definitely lost varies from about 125k to nearly 170k, at a glance. (k in this message is metric, not binary). 22:14
plobsing_ coverage? 22:16
purl coverage is cv.perl6.cz or tapir2.ro.vutbr.cz/cover/cover-results/
dukeleto msg cotto_work how much do we care about our svn -> git author map?
purl Message for cotto_work stored.
aloha OK. I'll deliver the message.
dukeleto NotFound++ # your git-fu is strong, grasshopper 22:19
NotFound I'm ruminating about doing a simple gui library using perl5 Gtk2, with the idea of making the backend pluggable later. 22:25
22:26 cotto_work2 joined
cotto_work2 dukeleto: I don't care as long as there's a reasonable way to manually map between them. 22:27
ttbot Parrot trunk/ r48832 MSWin32-x86-multi-thread make error tt.taptinder.org/file/cmdout/386067.txt ( tt.taptinder.org//buildstatus/pr-Pa.../rp-trunk/ ) 22:29
kid51 afk
22:30 tcurtis left, cotto_work left
nwellnhof i just benchmarked the charset_massacre branch... 22:35
rakudo build: 3m16s after, 3m 17s before
22:35 dalek left
chromatic +1 22:36
purl 1
nwellnhof callgrind perl6 -e -1: 1.619B Ir after, 1.625B Ir before
22:37 dalek joined
whiteknight is starting to get excited about the github switchover 22:45
cotto_work2 good time to get excited 22:46
dalek tracwiki: v18 | cotto++ | GitMigration 22:47
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
nwellnhof i'm ready to merge charset_massacre right now. all tests PASS, with and without optimize on debian 32bit. 22:55
cotto_work2 Do it.
22:55 cotto_work2 is now known as cotto_work
cotto_work Yeah. What cotto_work2 said. 22:56
nwellnhof incoming...
purl duck!
chromatic Ah, IMCC tailcalls leak SymReg structures.
nwellnhof git-svn++ 22:57
ii makes svn almost fun to work with 22:58
dukeleto nwellnhof: you have not yet begun to feel the pain, grasshopper 23:00
nwellnhof i said "almost fun" 23:01
ttbot Parrot trunk/ r48833 MSWin32-x86-multi-thread make error tt.taptinder.org/file/cmdout/386136.txt ( tt.taptinder.org//buildstatus/pr-Pa.../rp-trunk/ ) 23:05
dalek rrot: r48833 | nwellnhof++ | trunk (66 files):
Merge branch charset_massacre
dukeleto nwellnhof: don't expect "make fulltest" to act the same under a git-svn clone, FYI 23:06
nwellnhof: svn-specific tests don't work properly with a git-svn clone
nwellnhof what does "make fulltest" do? 23:08
kid51 make fulltest runs the regular make test through alternate runcores; runs tests of examples, coding standards; etc. 23:14
It's the most thorough regularly run result.
The release manager must get it to PASS before each release. 23:15
Others amongst us run it periodically.
23:15 patspam joined
kid51 So, if anyone wants to construct another make target that averts svn-specific tests, I welcome that. 23:16
But we'll eventually have to write git-specific tests as well (my hunch)
23:17 kid51 is now known as kid51_at_dinner
dukeleto nwellnhof: i meant that you can't truse the output of "make fulltest" with a git-svn clone 23:17
I don't know that we will need git-specific tests. There are no svn properties in git. 23:18
We will need to remove svn-specific tests in a branch, and have it ready to merge in when we switch to git. I've added this to the migration checklist
The more things we can do before the switch, and have ready, the better. 23:19
NotFound src/extra_nci_thunks.c contains control codes
src/extra_nci_thunks.c:2857: error: stray ā€˜\\260’ in program 23:20
mikehh NotFound: It says that it is a generated file? 23:21
NotFound mikehh: it is
23:21 bluescreen joined
NotFound Trying make corevm 23:22
mikehh NotFound: there are changes in r48831 that look like they are from testing (also fails codetest) 23:23
NotFound corevm builds. Let's hope coretest says something interesting
No, only the expeced integer and bigint 23:25
mikehh NotFound: see line 2672/2674 23:27
NotFound mikehh: were? 23:28
purl or used for plural past tense which this sentence obviously it.
cotto_work forget were
purl cotto_work: I forgot were
mikehh NotFound: src/nci/extra_thunks.c 23:29
NotFound Oh, nice, source files without svn properties...
cotto_work probably generated
NotFound cotto_work: yes, generated by human things. 23:30
cotto_work * !!!!!!! DO NOT EDIT THIS FILE !!!!!!! 23:31
looks machine-generated to me
NotFound cotto_work: I talked about codetest results 23:32
The shit in the generated file is probably the result of a bug, doesn't happen on i386 23:33
Great optimization work, boys.
ttbot Parrot trunk/ r48834 MSWin32-x86-multi-thread make error tt.taptinder.org/file/cmdout/386163.txt ( tt.taptinder.org//buildstatus/pr-Pa.../rp-trunk/ ) 23:35
dalek rrot: r48834 | nwellnhof++ | trunk/src/string/encoding (5 files):
[str] Use str_copy in to_encoding
23:36
rrot: r48835 | mikehh++ | trunk/src/string/encoding (6 files):
add svn properties
tracwiki: v19 | dukeleto++ | GitMigration 23:40
tracwiki: trac.parrot.org/parrot/wiki/GitMigr...ction=diff
NotFound Keeps failing in amd64 with --optimize, it builds and pass most tests without --optimize 23:44
ttbot Parrot trunk/ r48835 MSWin32-x86-multi-thread make error tt.taptinder.org/file/cmdout/386185.txt ( tt.taptinder.org//buildstatus/pr-Pa.../rp-trunk/ )
mikehh NotFound: I think that the file needs to be regenerated 23:48
NotFound in pcre.t output there are some "Freeing t_..." messages 23:49
chromatic Hm, that's probably my fault. 23:50
r48831?
NotFound r48836
chromatic Is r48831 the culprit? 23:51
23:52 dngor_ joined
NotFound ack "Freeing " 23:52
src/nci/extra_thunks.c
luben on amd64 I can't even finish the build with --optimize=-O3
NotFound 2672: if (!STRING_IS_NULL(ts_0)) { fprintf( stderr, "Freeing t_0\\n" ); Parrot_str_free_cstring(t_0); }
2674:if (!STRING_IS_NULL(ts_2)) { fprintf( stderr, "Freeing t_2\\n" ); Parrot_str_free_cstring(t_2); }
dalek rrot: r48836 | mikehh++ | trunk/src/string/encoding/ascii.c:
fix codetest failure - line length
23:53
NotFound luben: I can't even without the =-O3
23:53 dngor left
rrot: r48837 | nwellnhof++ | trunk (26 files):
[str] Switch most users of string_make to Parrot_str_new_init
rrot: r48838 | chromatic++ | trunk/compilers/imcc (3 files):
[IMCC] Plugged tailcall compilation memory leak.
ttbot Parrot trunk/ r48836 MSWin32-x86-multi-thread make error tt.taptinder.org/file/cmdout/386214.txt ( tt.taptinder.org//buildstatus/pr-Pa.../rp-trunk/ )
luben it's not charset masacre merge fault - it was the same before
and I spent 2 hours debugging my hash reorganization before I found that trunc merge was causing the failing test 23:55
NotFound chromatic: trac.parrot.org/parrot/changeset/48831/ shows it, looks like the commit included the generated file from a previous test
chromatic Testing a patch now.
luben NotFound, It depends on the compiler, with gcc-4.5 it build and fails on make test, with gcc-4.4 it does not build 23:56
ttbot Parrot trunk/ r48837 MSWin32-x86-multi-thread make error tt.taptinder.org/file/cmdout/386220.txt ( tt.taptinder.org//buildstatus/pr-Pa.../rp-trunk/ ) 23:57
whiteknight you guys are on fire today 23:58
NotFound Let's fire missiles!