Priorities for this week: MappedByteArray increase coverage & write demo, Google Code-In | Post closed tickets in your report. | Note: This channel is for our Tuesday status meetings (at 20:30 UTC); you probably want #parrot instead. | irclog: irclog.perlgeek.de/parrotsketch/today
Set by moderator on 23 December 2010.
02:27 kid51 joined
kid51 has blizzard issues; will be trying to keep his car from getting towed; may not be at #ps today 02:28
03:13 kid51 left 04:44 Coke left 05:07 Coke joined 05:28 cotto joined 12:33 bluescreen joined 17:41 NotFound joined
dukeleto will not be able to make #ps today 19:53
20:00 whiteknight joined
whiteknight WHAT I DID: 20:02
* Got embed_api merged. cotto++ for the help
* Fixed a handful of issues relating to embed_api
* Trying to keep up with GCI tasks.
* Holiday related stuff
WHAT I WILL DO:
* Still haven't fixed my dev laptop. Need to do that ASAP.
* Continue on with GCI.
WHAT I AM BLOCKING ON:
* Laptop broken
* Sick kid
* Not enough time.
Coke - on master & checkdepend_pmcs, got t/std/codingstd.t looking at all .c file deps. Please review branch and mergeback to trunk. 20:15
- poked whiteknight++ into some bugfixes for rakudo.
*EOR
er, t/src/checkdepend.t # odd mistype.
20:18 kid51 joined
kid51 kid51 additional report 20:19
* Followed up on ? of who can approve/publish GCI tasks
* Created 3 GCI tasks (which I think are still unclaimed)
* Had no time for TT #855 20:21
WILL TRY TO DO:
* TT #855 renewed effort
* Blogging re teams and roadmaps
BLOCKING ON:
* Blizzard travails: Car was not towed but is ... injured :-( 20:22
EOR
cotto_work *did:
- keep up (kinda) with gci
- lots of code review
- coke++ for the dependency cleanups, restoring my faith in make -j
- nwellnhof++ for making our coverage tests less ugly
- more of the same
- packfile planning and hacking
*blockers:
- none
*eor
20:26 nwellnhof joined, allison joined
mikehh What I did since my last report: 20:27
* building and testing parrot on amd64/i386, with gcc/g++
* last couple of days g++ does not build
* some fixes
* replace concat_s_s/concat_s_sc in examples
* html_cleanup needs to generate index pages (docs/index/*.json has the necessary info)
What I intend to do in the next week:
* testing and fixing
* hopefully complete the index/header pages in html_cleanup
.eor
nwellnhof what i did 20:28
- merge string_unescape
- work on my other branches
- misc fixes
plans
- merge unicode_io
eor
20:29 davidfetter joined
Util Note: Poor connectivity; traveling 20:29
# Done:
20:29 Kapace_ joined
Util Nothing; on holiday this week. 20:29
# Blockers:
'Tis the season...
# 7-day ticket report: 20:30
2 closed: done
12 closed: fixed
10 new
1 reopened
.end
cotto_work good morning 20:32
Util Hello
whiteknight hello
kid51 Hello; q1q
mikehh hi there 20:33
kid51 Are any of our customary discussion leaders here?
cotto_work?
cotto_work hi
kid51 u want to start off? 20:34
cotto_work How'd we do on last week's goals?
kid51 1. Goals were recorded in #parrot, but not in #parrotsketch
2. (I don't know how to set the topic in #parrotsketch)
mikehh missed out on merge for html_cleanup, got distracted by g++ issues and examples_tests failures
kid51 3. Lots of GCI-related activity.
4. We were temporarily blocked on GCI due to absence of an "organization administrator" to approve/publish issues. I believe that has been resolved. 20:35
cotto_work yes
I made a note of the goals at the end of last #ps
keeping up with gci seems to have been ok 20:36
kid51 5. Since many of the GCI participants are on Xmas break, they seem to be working/on channel round the clock :-)
cotto_work I'm an org admin and dukeleto is back, so we shouldn't be blocking on that.
NotFound Late hello
kid51 nwellnhof merged one branch (per earlier report) 20:37
nwellnhof and we merged embed_api
whiteknight ...and dealt with the fallout 20:38
q2q
cotto_work html_cleanup needs a bit of work. mikehh, do you think it can be merged this week?
mikehh hopefully, it's just the index page generation 20:39
cotto_work let's make that a goal then
Did anyone look at distutils or plumage to try and increase their bus number? 20:40
whiteknight I moved plumage to github
parrot/plumage
cotto_work great
mikehh frontend/parrot/main.c fails to build with g++ (gcc ok) and attempts I have made to fix it have failed
whiteknight so everybody reading this is now a committer on it
20:40 atrodo joined
NotFound cotto_work: I took some look, but I'm not fluent witn nqp 20:40
whiteknight mikehh: I'm buidling with g++ now and will have that issue fixed soon 20:41
mikehh whiteknight: thanks
whiteknight cotto_work: I suggest plumage be the second priority
cotto_work wfm
whiteknight: anything specific about it? 20:42
kid51 q3q
whiteknight make it work perfectly, make it able to submit smoke reports
smoke reports for it would be a huge help
cotto_work any volunteers for that?
NotFound I had a plumage failure in machine at work with an updated ubuntu. I think the problem is the usage of a git command not pesent in old versions. 20:43
s/updated/outdated
kid51 NotFound: quite possible; my Debian box has a 1.5-ish git. There are certain commands I have to do differently from 1.6 and 1.7. 20:44
How are smoke reports for Plumage done?
Do they go to Smolder?
NotFound I'll look at it in more detail and fill a ticket.
mikehh they used to, probably needs fixing 20:45
there was a build target to test and smolder test
cotto_work ok. Anything else before we move to questions? 20:46
kid51: you had some questions 20:47
kid51 Yes.
1. I'm getting very inconsistent results with 'make quickcover'. I know I had it working reasonably well, but others have been modifying it since. 20:48
Could i ask that if you touch the coverage targets, you do so in a branch and make sure you test both 'make quickcover' and 'make cover'?
Discussion? Otherwise, I'll move on to #2. 20:49
nwellnhof kid51: i tracked it down to changes in Devel::Cover. version 0.68 and above break our current system.
kid51 nwellnhof: Can you post about that to parrot-dev? 20:50
nwellnhof yes, of course
kid51 Thanks.
#2: 3.0 release next month -- 3 weeks from today; anything special we have to do?
whiteknight PANIC
cotto_work DON'T PANIC
whiteknight those are the two options 20:51
kid51 Is there anything we have promised for 3.0 that we will not be able to deliver on?
whiteknight I don't think we've made many promises about it, no
kid51 Or is the .0 release fundamentally no different from the .11?
whiteknight so long as it's stable and no slower, I think we can accept it and support it
mikehh cotto_work: you are the release manager are you not? 20:52
cotto_work yes
kid51 Any deprecations we have to get in?
mikehh q1q
cotto_work I think we have plenty already, but now would be a good time if anyone thinks of one.
bluescreen old embed api ? 20:53
whiteknight bluescreen: yes
I would like to deprecate the functions in src/embed.c in general. Some of those functions can remain but should be moved to better homes
and with moving may come renaming, etc 20:54
kid51 Does Rakudo have any special needs as we go to 3.0?
cotto_work It sounds like doing an embed api review should be a goal for this week.
whiteknight cotto_work: yes, a review of those functions in general is a good idea
cotto_work I submitted a patch to Rakudo for the removal of concat_s_s, so they're ready for that. 20:55
whiteknight some are superfluous now, some can be improved/moved/renamed
cotto_work I'm not sure about other HLLs
20:55 plobsing joined
kid51 #3: Are any of our GCI students here right now? Want to say anything to the assembled masses of developers? 20:56
Kapace_ Make more tasks!
:P
perferably non-test coverage tasks, since make cover keeps blowing up
mikehh how long has GCI got to run? 20:57
cotto_work mikehh: it ends on the 10th
Kapace_ 12 more days
whiteknight Kapace_: noted
kid51 Kapace_: Although 'make cover' does blow up, and that's a PITA, these tasks have focused us more on code coverage than ever before in the history of the Parrot Project 20:58
Util We just discussed GCI on #phasers for Perl 6. Masak intends to submit/mentor some Perl6 tasks.
cotto_work Great! I'll be happy to see Perl 6 get some gci love.
nwellnhof Kapace_: you can try 'make quickcover_new' if 'make cover' blows up. 20:59
kid51 END of my questions
Kapace_ nwellnhof: yes, I have tried that seems to work :)
cotto_work whiteknight: you had some questions
whiteknight yes 21:00
We need to expose Parrot_pmc_gc_register and Parrot_pmc_gc_unregister through the API to help embedders protect their PMCs from GC 21:01
any opinions on the best way to do that, or should I just put together a thin wrapper?
mikehh how much added complexity does it add 21:02
s/add/bring/ 21:03
plobsing the only concern I know of was that it puts expectations on GC that might be undesirable
whiteknight there are two functions I want to expose. At the simplest, that's two small API functions
plobsing but I think there may be workarounds
whiteknight plobsing: exactly. We could call it something like "pin/unpin" to hide details
the API does provide a mechanism for specifying a stacktop for GC stackwalking, so most embedding data should be safe 21:04
but there are rare instances when we would like to keep a reference alive even when it doesn't appear to libparrot that it's still reachable
mikehh you are effectively telling the gc don't deal with this memory item - I will 21:05
plobsing like anonymous callback subs held in head-allocated structures
s/head/heap/
whiteknight mikehh: What registering does is puts the PMC into an array that is marked explicitly by the GC
so yes, it's like explicitly saying to keep a PMC alive until I say otherwise
nwellnhof couldn't we abstract the API completely away from PMCs? 21:06
whiteknight nwellnhof: And use what instead?
mikehh so register says keep it and unregister says ok get rid of it
whiteknight mikehh: exactly
nwellnhof whiteknight: some wrappers
whiteknight nwellnhof: We could explore that option. I'm not sure I see a benefit in it right now 21:07
nwellnhof whiteknight: the benefit would be a nicer API. it's more work though.
plobsing nwellnhof: how would the wrappers handle the lifetime of objects? 21:08
Util Can we give methods to the PMCs that call reg/unreg on themselves? 21:10
plobsing wouldn't we still need to have explicit lieftime-management functions in the API?
whiteknight Util: We could put a method on default.pmc, yes
Util: Calling a PMC method from an embedding C application is not trivial though
okay, let's move this discussion to #parrot or the list, since I think we could talk about it all day 21:11
My second question: I would like to deprecate and eventually remove the EventHandler PMC.
Until this morning, it had been unusably broken for several months. Attempting to invoke it caused a segfault. It was completely untested 21:12
I've put in a fix this morning, but the PMC is still unused, isn't pretty, and is part of a subsystem that is going to need major work in the coming months
I suggest we remove it now, while we know that nobody is making effective use of it 21:13
or, deprecate it now
nwellnhof it it's broken it's kind of deprecated already. 21:14
whiteknight I would agree, but I won't argue that point
I would like to remove it, deprecate it otherwise
mikehh was it always broken, or just recently 21:15
whiteknight mikehh: Certainly it was never updated to work since the PCC refactor. So it's been months without so much as a bug report
I can't say if it worked before that
(untested, unused)
cotto_work If it's that broken, nobody wins anything by keeping it around. 21:16
nwellnhof i can't image it has any users, then.
cotto_work fire at will
mikehh just curious, some of the older HLL haven't been tested recently, eg cardinal
whiteknight cotto_work: thanks. Exactly what I wanted to hear
cotto_work Is that the last of your questions, whiteknight? 21:17
Tene mikehh: Cardinal recently received a small amount of work towards making it build successfully, at least, although it does not pass tests any more.
mikehh Tene: maybe if we get plumage going again we can work on cardinal and some others 21:18
cotto_work mikehh: yes. I think part of the drive to get plumage usable is that it'll let us more easily test HLLs and libraries automatedly. 21:19
Tene whiteknight: You're wrong here. EventHandler has worked just fine for a long time. What was broken was the invoke vtable, which wasn't used because it's meaningless. EventHandlers aren't vtable_invoked.
whiteknight Tene: Then maybe I don't understand what they are. What purpose do they serve if not invoked as a handler? 21:21
Tene whiteknight: The real eventhandler functionality, addhandler/schedule, has not stopped working at any time, afaict.
whiteknight cotto_work: yes, that's it
Tene whiteknight: it's like an exceptionhandler.
whiteknight Tene: exception handlers are invoked
Tene whiteknight: not vtable_invoked, no.
Hmm... could be I'm full of shit here, not entirely sure. 21:22
Still, the functionality of eventhandlers *has* continued to work just fine, as demonstrated by tests.
whiteknight Tene: src/exceptions.c:232 21:23
it is VTABLE_invoked
there's no other way to move control flow there
Tene you addhandler an eventhandler, and then schedule a task, and it works fine.
whiteknight Tene: there are very very few tests for EventHandler. Code coverage until this morning was at like 17%
Tene: addhandler may be doing something weird. I need to look at it 21:24
NotFound whiteknight: we have lots of PMC with very low coverage, that says nothing.
Tene whiteknight: t/pmc/scheduler.t 21:25
whiteknight NotFound: no, the coverage by itself says nothing
but if the PMC is throwing segfaults and nobody notices, we can't prove that it works 21:26
Tene Throwing segfaults for an unsupported operation that's not used by anything, or necessary.
NotFound whiteknight: I've been fixing segfault conditions in several importante PMCs that nobody noticed.
whiteknight Okay. I'll hold off on EventHandler for more study, but I still think we don't need it and it should be deprecated 21:27
cotto_work That sounds like a good idea.
whiteknight I'll look at it and raise the issue again next week if necessary 21:28
whiteknight has to sign out now. Later
cotto_work mikehh: you had a question
21:28 whiteknight left
mikehh mor like an observation really - I think it is important to run fulltest and perhaps test rakudo (maybe plumage) before merging a branch 21:29
NotFound Example of very low coverage: multisub.pmc 29% 21:30
kid51 make fulltest before branch merges+++++
mikehh for example removal of concat_s_s/concat_s_sc caused problems with rakudo and examples_tests
NotFound And for winxed
cotto_work I agree. We need to be more proactive about ensuring that we don't break HLLs, especially Rakudo and Partcl.
kid51 If your branch touches any file starting with 'src/', you really need to do 'make fulltest'
cotto_work The plumage work should make that simpler.
mikehh don't forget winxed 21:31
NotFound But that thing was already deprecated and announced and there was an easy update path, no big issue.
plobsing branch merges shouldn't break core's fulltest. breaking HLLs should be fine if they have a reasonable upgrade path. 21:32
Tene considers make_hlltests
21:34 bluescreen left
nwellnhof examples is the most critical part of make fulltest. maybe they should be moved to normal tests. 21:34
cotto_work I think they were taken out because of how long they took to run. 21:35
plobsing nwellnhof: do that, and I'll only run coretest most of the time
mikehh also codetest
I think we might want to separate out the perlcritic test in codetest as they are the longest running part 21:36
nwellnhof the problem with make fulltest is that it tests all the runcores IIRC. that takes ages. 21:38
mikehh at the moment just testb, testf and testr 21:40
nwellnhof make examples_tests takes about a minute.
mikehh used to be another 4 or so
so does codetest without perlcritic
kid51 'make not-quite-fulltest', anyone? ;-) 21:41
cotto_work So it sounds like we need more emphasis on examples_tests 21:42
Tene make did-i-break-it
nwellnhof i'm for adding examples to the standard test target.
kid51 If people can tell me what they want, I'll create an intermediate target
'make test' + 'make examples_tests' + 'make codetest (except perlcritic)' -- does that sound okay? 21:43
(I don't want 'make test' to take any longer than it currently does.) 21:44
mikehh good target, don't know what to call it though
kid51 I'll think of something
cotto_work kid51: sounds fine. serial examples_tests takes under a minute on my machine, parallel takes 38s. 21:45
mikehh but still keep fulltest as it is
cotto_work that's not excessive
nwellnhof i'd rather make that the default target and have other targets for quicker tests.
cotto_work Can someone start a parrot-dev thread to discuss this further?
kid51 nwellnhof: I recommend let's try an intermediate target and see how we like it -- later we can ponder making it the default for 'make test' 21:46
Tene The default should be more comprehensive, yes. We've had problems with people not running the right tests, and a proliferation of small test targets that people need to remember doesn't actually help anyone.
I'm not really clear on what the utility of trimming down the 'test' target is.
kid51 Tene: I don't think we're talking of trimming 'make test' 21:47
mikehh time to run the tests
kid51 Tene: We're talking of a new target that would include 'make test' plus others -- but much less than 'make fulltest'
Tene If the examples tests are important to run, why not add them to 'make test'?
kid51 They will only run quickly on boxes with high resources.
Also, as I stated above, I recommend trying out the new target before substituting it for 'make test' 21:48
Tene I really feel like I'm missing something here, but I certainly haven't been active enough to legitimately complain, so I'll just assume you guys have good reasons that I just haven't been present enough to notice. :) 21:50
kid51 I will open up a Trac ticket for this.
cotto_work kid51: thanks.
Are there any other questions? 21:51
NotFound Maye instead of adding make targets we should kill some, and use parrot programs to run some of that things.
cotto_work NotFound: that's an interesting idea. Can you mention that in kid51's ticket? 21:52
NotFound More complex Makefile --> more portability problems
mikehh and test with C++ as well as C
cotto_work Here are the goals I have for the week: 21:53
GOAL 1: more gci tasks (everyone)
GOAL 2: merge html_cleanup (mikehh)
GOAL 3: get plumage working, increase its bus number (?)
GOAL 4: review current embedding api, decide what to keep (whiteknight)
GOAL 5: study EventHandler, decide what parts are useful and what's worthwhile (whiteknight)
GOAL 6: run examples_tests and test Rakudo, Partcl and winxed before merging significant changes into master (everyone)
NotFound But... plumage is working, isn't it?
kid51 cotto_work++ on goals
cotto_work NotFound: If it is, that'll be a very easy goal.
Either way, we need more people who understand it. 21:54
mikehh also bus number increase
NotFound Last week it was broken for metedata with utf8 chars, but I fixed that, and was a parrot problem, not a plumage one.
mikehh NotFound++ 21:55
NotFound The only problem I know is the one I commented before, git targets with old git versions. 21:56
kid51 Anything else before we wrap? 21:58
cotto_work That's a wrap. 21:59
moderator Priorities for this week: irclog.perlgeek.de/parrotsketch/201...#i_3126985 | Post closed tickets in your report. | Note: This channel is for our Tuesday status meetings (at 20:30 UTC); you probably want #parrot instead. | irclog: irclog.perlgeek.de/parrotsketch/today 22:06
22:08 dduncan joined, kid51 left 22:12 NotFound left 22:14 plobsing left 22:18 atrodo left 22:29 davidfetter left, nwellnhof left 23:09 dduncan left