Parrot 2.11.0 Released | parrot.org | Log: irclog.perlgeek.de/parrot/today | Onward and upward with Google Code-In | Please test rakudo with bleeding edge parrot! | merge html_cleanup and embed_api2
Set by moderator on 21 December 2010.
00:00 Yuki`N left
whiteknight I have a few in 00:01
we need dukeleto to approve them 00:02
Yuki'N: There are a few tasks in the queue still that you could do. Some of them should be pretty quick and easy, even if they aren't super fun 00:12
sorear gci isn't over yet?
cotto_work It ends on the 10th 00:13
whiteknight GCI is like the gift that keeps giving 00:14
We have 76 tasks completed already 00:15
Next year I think we need to have at least 100 tasks set up
our slowness at creating and managing tasks has been the biggest limiting reagent here
cotto_work Bah. Getting make cover to fail quicky with nmake is not easy. 00:16
whiteknight nmake sucks at failing 00:19
cotto_work Ah. windows' echo treats everything as its argument, not just the quoted stuff
whiteknight s/ at failing//
cotto_work If I just put "exit 1", it stops correctly 00:21
whiteknight Who's on the January release, cotto? 00:24
cotto_work yup
whiteknight Okay 00:29
I would definitely like to see PLATFORMS be updated this month
and lots of smoke reports submitted
aloha: coverage? 00:38
aloha whiteknight: coverage is cv.perl6.cz or tapir2.ro.vutbr.cz/cover/cover-results/
whiteknight how is it that Parrot_pcc_build_sig_object_from_varargs has zero test coverage? 00:40
Parrot_pcc_parse_signature_string too. There are a few functions in src/call/args.c that I feel like either should be covered, or are probably dead code 00:41
Parrot_pcc_allocate_empty_context is probably dead if it isn't covered 00:42
Parrot_pcc_invoke_sub_from_c_args. That one has probably been completely replaced by Parrot_ext_call 00:44
so many of these functions SHOULD NOT be labeled PARROT_EXPORT
bluescreen whiteknight: ping 00:45
whiteknight pong
bluescreen one question
whiteknight sure
bluescreen all example in t/src/embed do this: "GET_INIT_STRUCT(initargs);" thats calls calloc
whiteknight right 00:46
bluescreen should we give user a FREE_INIT_STRUCT macro?
whiteknight what do you mean?
bluescreen i mean, if that nevers get freed we're in troubles
cotto_work bluescreen: ping
bluescreen the other option is let the user call free himself
cotto_work: pong
cotto_work bluescreen: can you make a note in #1899 as to which commit fixed the issue? 00:47
bluescreen sure... that was mi comment in the commit,
cotto_work: do you mean like a comment in track saying "Commit XXXXX fixed the issue"? 00:48
cotto_work bluescreen: yes.
bluescreen cotto_work: done.. I'm used to do the opposite put the ticket in the commit message 00:49
whiteknight src/dynext.c:make_string_pmc is basically just a box operation. It's the same as the code in the box_p_s op, but it doesn't support HLL mapped types
bluescreen whiteknight: In the same way we gave users a GET_INIT_STRUCT we should give them a FREE_INIT_STRUCT.. I'd wouldn't let them use "free" directly to get rid of "initargs" 00:52
whiteknight oh, right. I see what you are saying 00:53
yes, that probably causes a leak, doesn't it?
bluescreen yes.. well not a big problem in the case you exit immediately
but if you are to be creating and destructing interps it can be a problem
whiteknight actually better yet, we should probably change GET_INIT_STRUCT to take a structure, not a structure pointer
allocate it on the stack, don't use calloc 00:54
bluescreen yeah much better, and that would be the only struct we ever publish outside parrot
00:55 kennym left
whiteknight we should probably rename that macro to something better like PARROT_INIT_ARGS() or something 00:57
i didn't put a lot of thought into that macro, obviously
bluescreen he he... I like much better PARROT_INIT_ARGS() 00:59
dalek rrot: f7512cd | cotto++ | lib/Parrot/Pmc2c/PCCMETHOD.pm:
[pmc2c] remove some unused generated code
01:01
01:04 perlite joined 01:09 macroz left
bluescreen one question: what prevents gabarge collector from destroying string, hashes you create in an embedded application ? for example in main.c (of my app ) pmc = Parrot_api_pmc_new(...); 01:11
it won't be referenced from any other pmc... so in theory mark and sweep should sweep it, isn't it? 01:12
cotto_work either they're on the stack where the stack-walking code will find them or they're manually added to the root set
whiteknight bluescreen: the init args define a "stacktop" pointer. GC traces the system stack up until this location 01:15
if the embedding application sets their stack top high enough, the GC will trace into embedding code
also, there is a function somewhere that allows us to register a PMC for GC 01:16
src/pmc.c:Parrot_pmc_gc_register and Parrot_pmc_gc_unregister 01:17
Those might make good routines to expose through the new API
cotto_work +1
whiteknight Although I feel like it' ahuge waste to add two new functions to the API for such an arcane kind of thing 01:18
cotto: You took a look at the new draft of PDD10. Besides the spelling/grammar fixes, what did you think about it?
cotto_work whiteknight: I'd like to read through it a couple times and mull it over a bit. So far I've only skimmed through it. 01:19
whiteknight okay, what do you think of the style of it?
I was trying not to descend to function-level documentation
It mostly documents what we have in the branch now. There are very few points in there which are TODO 01:22
cotto_work I do like the overall organization. It's approachable. 01:24
registration is important. Most users may not end up needing it, but for the ones that do the only alternative is to repeatedly call VTABLE_mark on the PMC manually or accept semi-random failures. 01:28
that's my first thought on it, at any rate 01:29
sorear the alternative is to have a global Hash PMC of "tracked C globals", like Lua does
cotto_work sorear: that's basically what the root set is now 01:30
sorear is there a Parrot API that I can use to save and restore PMC* values by name? 01:31
cotto_work sorear: how would that be used?
sorear Parrot_pmc_gc_register is going to lead to problems if we want the collector to be able to move stuff
cotto_work: void run_script_button() { Parrot_call_sub(interp, Parrot_get_cglobal(interp, "script")); } 01:32
01:34 Yuki`N joined
cotto_work by "save and restore", do you mean full freeze/thaw or just referencing? 01:36
whiteknight: do you plan on keeping the limitation that interps after the first have to have a parent? 01:38
plobsing sorear: Parrot_pmc_gc_register is only a problem if the handles we provide embedders are guarranteed to be pointers. Nothing I have read makes that guarrantee. 01:43
cotto_work plobsing: interesting 01:44
sorear plobsing: for some reason I thought you were going to typedef PMC *Parrot_PMC; 01:45
ha.
ah.
cotto_work: referencing.
cotto_work whiteknight: the docs don't specify how :immediate and friends are handled
plobsing sorear: that's the way it works now (and I agree that the current implementation won't work with a collector that moves stuff indiscriminantly), I'm just saying it doesn't have to work that way. 01:47
cotto_work whiteknight: I'm not sure how I feel about Parrot_api_disassemble_bytecode. 01:52
whiteknight cotto_work: I don't understand the limitation that interps after the first must have a parent, and I am going to ignore it 02:00
we'll treat that like a bug if we have to
cotto_work: :immediate is the province of the compiler. :load and :init should probably be covered by docs. I haven't wanted to discuss them because plobsing and I are trying to change them 02:01
cotto_work ok
whiteknight Parrot_api_disassemble_bytecode only exists to implement pbc_disassemble.exe
I don't like it at all, but until we separate out that functionality into the packfiles or whatever other tool has providence over it, it has to be there 02:02
Kristaba Hi whiteknight 02:03
whiteknight hello Kristaba
Kristaba I would do some tasks for Parrot, but I'll go to see my family for a week at the other end of France (more than ten hours of car) 02:04
whiteknight ouch. 10 hours is hard
Kristaba So, have you something to do that doesn't require an internet access? :p
Yeah, France with its small country road :D 02:05
(never mind if it's not really a GCI task)
Or, if you have an idea of something that can allow me to prepare a future task 02:06
whiteknight Kristaba: I don't have any off the top of my head. When do you leave?
Kristaba In 5 hours :x 02:07
Oh, some days ago, you tell me about a C# API, or something like 02:08
whiteknight Kristaba: oh, right. github.com/Whiteknight/parrotsharp 02:09
I do it in MonoDevelop, which is free
if you want to create wrappers for a few more API functions in that, I'll make tasks for whatever you do and count it for GCI 02:10
a lot of it is still a prototype, so you can play and experiment with things if you want 02:12
Kristaba Perfect, that will keep me busy for the road ;)
whiteknight awesome. I don't have any unit tests for that either. I need to set those up, or you can play with that if you want 02:14
whatever you want to do. The project is young and needs everything :)
Yuki`N whiteknight, ping 02:15
I'm still up for some API work, if you found anything.
sorear is curious about this "the far side of France has no internet access" thing
Yuki`N As repetitive as it sounds. 02:16
Kristaba whiteknight: Yeah, I don't promise anything, but I'll play with for sure :p
Well, about 3h a.m here., I go bed :)
whiteknight goodnight
02:17 mtk joined
Kristaba So, good night and good Christmas everyone 02:17
02:17 Kristaba left
cotto_work whiteknight: I like the pdd10 update so far. 02:20
nice work
whiteknight thanks. That's my christmas present to you
cotto_work whiteknight: did you see that pauseless gc article? 02:21
whiteknight cotto_work: yeah, read it enthusiastically
Yuki`N Hmmm :/
cotto_work does it sound like something Parrot could do post-Lorito? 02:22
whiteknight makes some extremely aggressive use of virtual memory, I would be *very* surprised if we could implement that in a way that works on *nix, solaris, and windows
and I highly doubt we could use it on an embedded device like android
(which is where we want pauseless GC in the first place) 02:23
cotto_work: in short, I guess I really can't tell you how hard it would be, I would need to take a look at some of their code to see what all their techniques were
sorear "pauseless GC" is actually pretty easy if you don't care about constant factors 02:24
sorear goes to find an article 02:25
plobsing Azul, the people who made the pauseless GC in question, require either a custom hypervized OS or a custom Linux kernel on which to run their VM.
whiteknight right, I highly doubt we could use it
and even if linux got a permanent kernel improvement and that got shipped to all major distros, we still wouldn't have it on Windows or solaris 02:26
02:27 mtk left
cotto_work That's unfortunate. Maybe if the patch makes it into Linux we could have a specialized gc to take advantage of it, but it doesn't sound like a portable solution. 02:28
time to go home 02:30
whiteknight time for me to go to bed. See you tomorrow 02:31
02:31 whiteknight left 02:32 mtk joined 02:35 mtk left 02:41 mtk joined 02:45 kid51 joined 03:04 Yuki`N left 03:22 AzureSto_ left
cotto ~.~ 03:23
03:25 AzureStone joined 03:27 AzureStone left 03:29 AzureStone joined
Coke aloha: seen mj41? 03:39
aloha Coke: mj41 was last seen in #parrot 23 days 18 hours ago saying "dukeleto: Ok. Will look at it too as fast temporary solution.".
Coke aloha: tt? 03:40
aloha: taptinder?
aloha Coke: taptinder is continues integration tool - taptinder.org . For Parrot project running on tt.taptinder.org/ and reporting build failures to #parrot channel as ttbot.
Coke msg mj41 - anything I can do to get tt.taptinder.org/ working again? 03:41
aloha OK. I'll deliver the message.
03:50 particle left 03:52 particle joined 03:55 bluescreen left 04:06 bluescreen joined 04:09 Coke left
dalek rrot: d519bc8 | cotto++ | config/auto/coverage.pm:
[configure] use $(PERL) instead of trying to figure out portable shell code for windows and *nix
04:09
04:09 Coke joined
cotto seen fbrito 04:15
aloha fbrito was last seen in #parrot 1 days 8 hours ago leaving the channel.
cotto accidentally a word 04:26
04:35 kid51 left 05:12 ilbot2 joined
moderator Parrot 2.11.0 Released | parrot.org | Log: irclog.perlgeek.de/parrot/today | Onward and upward with Google Code-In | Please test rakudo with bleeding edge parrot! | merge html_cleanup and embed_api2
05:14 snarkyboojum joined 05:15 Infinoid joined, estrabd joined 05:34 rurban_ joined 05:36 rurban left 05:37 rurban_ is now known as rurban
dalek tracwiki: v7 | cotto++ | Platforms/Windows 05:43
tracwiki: fix up the msvc parts a bit
tracwiki: trac.parrot.org/parrot/wiki/Platfor...ction=diff
06:09 confound joined
dalek rrot: 8d253b8 | Coke++ | t/codingstd/perlcritic.t:
Mention env var in POD
06:29
rrot: 4b96417 | Coke++ | t (2 files):
Make checkdepend a real codingstd test.

  * eliminate Test::Differences dependency
  * eliminate App::Ack dependency
  * Rip out any code that dealt with PIR -
   Results of this were always ignored anyway, and we can put it back when
we have code that will pass.
rrot: d70323e | Coke++ | src/dynpmc/ (2 files):
Fix last failure on t/codingstd/checkdepend.pl

  (I think I did this differently in a branch. This way is better.)
rrot: 08e9d21 | Coke++ | MANIFEST:
track file rename
06:30
cotto coke++ 06:32
NotFound exit 0 \\ 06:41
/usr/bin/perl Configure.pl --ccflags=" -fprofile-arcs -ftest-coverage" \\
--linkflags="-fprofile-arcs -ftest-coverage" --ldflags="-fprofile-arcs -ftest-coverage"
/bin/sh: line 0: exit: too many arguments
tapir2.ro.vutbr.cz/cover/logs/last-run.txt
dalek rrot: f50b101 | Coke++ | t/codingstd/checkdepend.t:
remove trailing whitespace.
06:42
Coke cotto: I just got sick of having to run it by hand once a week. ;)
next time I will try to remember to reconfig & run codetest before committing. 06:43
s/committing/pushing/
cotto NotFound, I thought I tested that. 06:44
my mistake
NotFound Is a shame, I was expecting to see exporter.pmc at 100% 06:46
cotto It's an easy fix. I'm running make cover now to make sure there isn't anything else. 06:49
06:58 bacek joined
cotto still broken 07:01
hi bacek 07:02
thanks for getting aloha back up
07:03 cognominal left
bacek aloha, cotto 07:07
07:34 theory left
dalek rrot: 14aa06c | cotto++ | config/gen/makefiles/root.in:
[cover] get make cover working again
08:23
mj41 Coke: Please join irc://irc.freenode.org/taptinder and look at github.com/mj41/TapTinder/issues 08:35
08:45 mikehh_ is now known as mikehh
mikehh opbots, names 08:45
08:57 slavorgn left 09:26 fperrad joined 09:32 rfw left
dalek rrot: a6f2157 | (Gerd Pokorra)++ | docs/project/release_manager_guide.pod:
add me to take release 3.2
10:01
11:14 bluescreen left 11:23 cognominal joined 12:07 contingencyplan left 12:19 bluescreen joined 12:25 slavorgn joined
mikehh Coke: checkdepend.t does not work before make, and has a failure after 12:38
Coke: or out it this way - it has multiple failures - Tests: 97 Failed: 95 before make, after configure and Tests: 121 Failed: 1 after make 12:42
s/out/put/
all files in include/pmc/* seem to be generated and are required by the test - it needs to go somewhere else - not make codetest which can be run after configure 12:45
13:34 rurban_ joined, whiteknight joined 13:36 rurban left, rurban_ is now known as rurban
whiteknight good morning, #parrot 13:38
13:45 smash joined
smash hello everyone 13:45
whiteknight good morning, smash 13:47
13:48 plobsing_ joined
whiteknight msg cotto: I'm ready to start the embed_api4 merge any time. Let me know if you have any questions or concerns before I push the button. 13:50
aloha OK. I'll deliver the message.
13:51 kennym joined 13:52 plobsing left
whiteknight when we get that branch merged and the coverage bot starts giving us statistics for it, I will start setting up GCI tasks to increase coverage for it 13:52
Coke mikehh: the test doesn't fail if Makefile isn't present, it skips everything. 14:11
That said, feel free to move it to another directory.
(I don't see any targets that are appropriate.) 14:12
any suggestions? 14:13
mikehh Coke: don't know - have to think about that - Configure generates the makefile, but you have to make to generate some of the dependencies 14:15
in testing I usually run Configure with --test, make manifest_tests, make codetests, make corevm/make coretest, make world/make test, then fulltest 14:17
all after a make realclean and git pull --rebase 14:19
maybe in the t/src directory 14:20
14:22 JimmyZ joined
JimmyZ good evening, parrot 14:25
mikehh hi JimmyZ 14:27
JimmyZ hello mikehh
mikehh JimmyZ: how's things in your part of the world? 14:28
JimmyZ everything goes well 14:30
except looking for a job 14:32
GCI work for chinese students may be unpractical. 14:33
14:47 JimmyZ left 15:25 Matt_ joined 15:41 jan joined
dalek TT #1901 created by Matt221++: Parrot_io_get_last_file_position should be removed in the next deprecation ... 15:50
TT #1901: trac.parrot.org/parrot/ticket/1901
whiteknight I strongly suspected that function was not being used 15:51
Matt_ whiteknight: Submitted the task for review :) 15:52
whiteknight: github.com/parrot/parrot/pull/38 15:53
whiteknight Matt_: awesome. Let me pull it and I'll close it 15:54
Matt_ thanks!
16:06 bluescreen left
plobsing_ aloha: msg NotFound assign (=:) is an unacceptable substitute for real closure variables. Any chance I could use unary * for fetch/push update? Or get 'volatile' for the generation of pessimistic code? 16:13
aloha plobsing_: OK. I'll deliver the message.
plobsing_: Okay.
NotFound plobsing_: unary * ? 16:14
plobsing_ *x; // pointer-ish syntax
NotFound I see... let me think about it...
16:21 Matt- joined 16:26 Matt_ left 16:33 Matt- left, Matt_ joined
NotFound plobsing_: volatile seems to be a cleaner approach, I'll look at that way. 16:34
plobsing_ thanks 16:35
dalek rrot/gci_deprecate_io_position: 8b88b11 | (Matt Rajca)++ | / (3 files):
Removed Parrot_io_get_last_file_position for deprecation
16:36
whiteknight Matt_: pulled. I've closed the task. Thanks 16:39
Matt_ whiteknight: Ill take this too: www.google-melange.com/gci/task/sho...5777080#c1 16:44
16:45 dmalcolm joined
whiteknight Matt_: approved. Good luck! 16:53
17:09 whiteknight left
cotto msg whiteknight Merge whenever you're ready. 17:14
aloha OK. I'll deliver the message.
17:32 theory joined
cotto_work ~~ 17:33
dalek m-eta-wink-kzd: 2839fce | plobsing++ | src/Makefile:
add rules for generating pbc
17:37
m-eta-wink-kzd: a773923 | plobsing++ | src/ometac.winxed:
add stage1 winxed compiler
m-eta-wink-kzd: 9087d9f | plobsing++ | src/ (2 files):
change build to accomodate stage1 compiler
17:39 ambs joined
dalek TT #1902 created by Matt221++: Parrot_freeze_size should be removed for deprecation 17:48
TT #1902: trac.parrot.org/parrot/ticket/1902
17:57 zaur joined 18:04 whiteknight joined
cotto_work hio whiteknight 18:05
whiteknight hey cotto_work
you're happy with the embed_api4 branch? 18:06
cotto_work yes
whiteknight okay then. I'll merge soon
cotto_work whiteknight: do you see any value in including Visual Studio instructions in Platforms/Windows? All that's necessary is the Windows SDK. 18:07
18:07 particle1 left
cotto_work trac.parrot.org/parrot/wiki/Platforms/Windows 18:08
whiteknight cotto_work: I was looking at that page yesterday too. No, VisualStudio isn't necessary but it is a way to get the SDK
cotto_work are the instructions worth keeping or can we assume that anyone who has visual studio will know how to use it?
I'd like to remove any unnecessary instructions from the page. There's more there now than I think is necessary. 18:09
whiteknight are you on windows? 18:10
for any of your developing?
18:10 particle joined
cotto_work I have a windows machine set up with the msvc toolchain and I'm going to get the mingw toolchain working later today. 18:10
It's not as painful as I was hoping.
er, afraid of
whiteknight oh, okay. I have the msvc toolchain here at work 18:11
I don't see any reason to talk about VisualStudio except to mention that it's a vector to obtain the SDK
cotto_work wfm
whiteknight Matt_: I'm running a merge and tests here locally. I'll get your pull request after that 18:14
msg plobsing Can you take a quick look at #1902? A GCI student has identified it as dead code, and I want a second set of eyes on it before we move to deprecate/delete 18:15
aloha OK. I'll deliver the message.
whiteknight we wouldn't need to worry about deprecation at all if functions like these weren't marked PARROT_EXPORT 18:16
stupid PARROT_EXPORT 18:17
I would like to put in a blanket deprecation notice for a cull of PARROT_EXPORT functions that are not used, not usable, or that simply should not be exported 18:18
We have too damn many functions being exported
plobsing_ whiteknight: done 18:24
as I understood it, PARROT_EXPORT didn't mark something as "part of the api", it just made it visible for parrot core tools (which were special at the time). 18:32
18:34 fperrad left 18:36 fperrad joined
whiteknight plobsing_: I like that interpretation of things much more than the alternative 18:38
cotto_work Note that "PARROT_API" is spelled differently from "PARROT_EXPORT". 18:39
plobsing_ also, IIRC, PARROT_EXPORT is sometimes needed to make the windows build work for some reason. 18:41
and then there's the issue of ops libraries and C++ forcing me to extern a symbol 18:42
cotto_work I have a working window build environment now, so we can test what breaks it pretty easily.
18:44 Matt- joined
cotto_work whiteknight: are you set up to build on Windows? 18:46
18:50 Matt_ left, Matt- left, Matt_ joined 18:53 dd070 joined
dalek tracwiki: v8 | cotto++ | Platforms/Windows 18:57
tracwiki: rework instructions on getting the mscv and mingw environments set up
tracwiki: trac.parrot.org/parrot/wiki/Platfor...ction=diff
19:04 lucian joined 19:06 nwellnhof joined
nwellnhof cotto: why did you remove the instructions for Microsoft Visual C++ 2010 Express from the wiki page? 19:07
cotto_work nwellnhof: they're not necessary. The Windows SDK includes the compiler. 19:08
I want it to be as straightforward as possible to start hacking on Parrot on windows. 19:09
nwellnhof: does that strike you as sensible? 19:10
nwellnhof yes, if the SDK contains everything needed. the Visual C++ Express install might be a bit leaner, though. 19:11
cotto_work I suspect that the SDK is lighter.
not certain though
nwellnhof the SDK seems to contain the .net stuff, too. 19:12
cotto_work I had to install it separately.
nwellnhof but Visual C++ Express installs all kinds of other stuff, too. 19:13
19:15 ambs left
whiteknight Visual C++ Express definitely contains the .net stuff 19:17
kennym hi, who's in charge of this task? www.google-melange.com/gci/task/sho...9031964617 19:19
cotto_work kennym: any Parrot mentor can deal with any task 19:20
19:27 Patterner left, theory left 19:37 Psyche^ joined, Psyche^ is now known as Patterner 19:38 Khisanth left
lucian github.com/appcelerator/kroll similar to mozilla's XPCOM i think 19:56
lucian has to go
19:59 contingencyplan joined, lucian left 20:05 sorear left 20:12 sorear joined 20:13 he_ joined 20:14 he__ left, Kapace_ left 20:22 simcop2387_ joined
whiteknight has somebody played with dependencies or the makefile recently or anything? 20:22
I'm getting a lot of failures in t/codingstd/checkdepend.t here locally where I'm merging embed_api4 into master
and a lot of these are files I that have not been changed in the branch 20:23
cotto_work that test was only added yesterday
20:24 simcop2387_ left
whiteknight okay, can I ignore it for now? 20:24
cotto_work probably good to fix what can be fixed 20:25
dalek m-eta-wink-kzd: 24b594b | plobsing++ | src/Ωη.coda.winxed:
re-open classes the hard way (winxed does not support this with syntactic sugar yet)
20:32
m-eta-wink-kzd: f135bf8 | plobsing++ | src/Makefile:
track more generated files
m-eta-wink-kzd: d50db48 | plobsing++ | src/ometa-base.winxed:
use quotes to avoid pseudo-constructor
m-eta-wink-kzd: 2ec1b38 | plobsing++ | src/ometa-base.winxed:
lookup class for instantiation - String.new doesn't work
20:33
whiteknight cotto_work: is that test passing in master? 20:35
cotto_work whiteknight: no. there's one failure 20:36
20:38 bluescreen joined, rfw joined
whiteknight blah. 20:42
well, I have no idea why this test is failing, or how to fix it 20:43
merge is going to have to wait till tomorrow, at least 20:44
dalek rrot/embed_api5: 4e02ead | Whiteknight++ | / (53 files):
fix some conflicts in branch merge
20:49
rrot/embed_api5: c92d340 | Whiteknight++ | / (3 files):
fix a few codetest failures
rrot/embed_api5: 95a4a78 | Whiteknight++ | src/embed/pmc.c:
fix more codestd problems
cotto_work I'll take a look at it. 20:55
whiteknight thanks. I can't make heads nor tails of it 20:56
Matt_ whiteknight: When you get a chance to, please review: github.com/parrot/parrot/pull/39 plobsing_ already approved the change on the trac ticket page 20:57
20:58 bluescreen left
dalek rrot: 8780440 | plobsing++ | config/gen/makefiles/root.in:
fix checkdepend.t
20:59
plobsing_ why does embed_api keep incrementing? 21:00
whiteknight plobsing: because I keep merging from master, and I don't want to merge in-place 21:01
plobsing_ why not? 21:02
21:02 plobsing_ is now known as plobsing
dalek rrot/gci_removed_freeze_size: 0aee73a | (Matt Rajca)++ | / (2 files):
Removed Parrot_freeze_size for deprecation
21:02
whiteknight Matt_: done
plobsing it's not like you are rewriting history or anything. 21:03
whiteknight plobsing_: is it a problem that I'm incrementing it?
cotto_work It's a curious git workflow.
plobsing not really a technical problem, no
it does make it harder to keep track of what the latest embed_api branch is 21:04
whiteknight I'm experimenting with it
plobsing that and you're not cleaning up your defunct branches
whiteknight I've cleaned up some. I wasn't deleting branches that people said they still had outstanding commits on 21:05
dalek rrot: 1819260 | cotto++ | / (7 files):
various minor fixes to make msvc less sad
rrot: 658e577 | cotto++ | src/pmc/orderedhash.pmc:
[pmc] remove some unneeded code from OrderedHash
atrodo "less sad" Brilliant 21:07
plobsing whiteknight: you can delete branches on which people have unpushed changes. they just have to merge those into the latest branch (which they have to do anyways).
and it gives a good indication of "hey, you're working on a defunct branch. stop doing that" 21:08
rfw hey whiteknight 21:12
or whoever
www.google-melange.com/gci/task/sho...9295757244 <-- looking to claim
21:12 plobsing is now known as whoever
whoever hi rfw! 21:12
21:12 whoever is now known as plobsing
rfw hi whoever, formerly known as plobsing! 21:12
where are the namepsace tests, by the way 21:13
cotto_work t/pmc/namespace.t 21:14
rfw ah got it
thanks
plobsing > find t | grep namespace # there's a couple of places
Matt_ whiteknight: can you close the task: www.google-melange.com/gci/task/sho...7080#c3002 21:21
cotto_work I wish melange didn't have one url that worked and another that didn't.
whiteknight rfw: Accepted. Good luck 21:22
rfw whiteknight: thanks
whiteknight Matt_: Accepted and closed. Thanks
cotto_work notanumber.net/archives/54/underhan...-redaction
21:23 simcop2387 left
rfw whiteknight: i don't quite understand what make_namespace_autobase does 21:23
i don't think it's deprecated though, since i can see it in the code
whiteknight rfw: you see it used somewhere? that's disappointing 21:25
rfw whiteknight: yeah
whiteknight rfw: Where is it used from?
rfw src/pmc/role.pmc: _namespace = Parrot_ns_make_namespace_autobase(interp, _namespace);
whiteknight stupid role PMC
rfw: Add tests to t/pmc/role.t to try and execute that function
rfw ah okay 21:26
whiteknight doesn't matter what it does, just get it covered for now
(understanding it would be good too, but for now I want it to be covered at least a little)
once we know that we are properly exercising it, we can fix it, replace it, delete it, whatever and know that we aren't breaking anything
rfw okay 21:27
dalek rrot/embed_api5: 8780440 | plobsing++ | config/gen/makefiles/root.in:
fix checkdepend.t
21:28
rrot/embed_api5: 1819260 | cotto++ | / (7 files):
various minor fixes to make msvc less sad
rrot/embed_api5: 658e577 | cotto++ | src/pmc/orderedhash.pmc:
[pmc] remove some unneeded code from OrderedHash
rrot/embed_api5: 1a76c9b | Whiteknight++ | / (9 files):
Merge branch 'master' of github.com:parrot/parrot
whiteknight checkdepend.t is still broken on that branch 21:29
rfw aahhhhh oh god what
parrot didn't compile
error: Parrot_DynOp_core_2_10_1 undeclared
whiteknight: could you take a look at that 21:30
plobsing rfw: you need to reconfigure after parrot revision number changes
rfw plobsing: i did
plobsing make realclean?
rfw i did make clean before 21:31
cotto_work whiteknight: I'd say go ahead and merge. I'm getting inconsistent results with checkdepends and I'm not entirely convinced that your code is wrong.
or just use plobsing's fix
rfw oh works now
probably not clean enough
whiteknight cotto_work: too late now. I'm heading home. Will do it tomorrow 21:32
later
21:32 whiteknight left
cotto_work wfm 21:32
21:33 dd070 left 21:34 simcop2387 joined, rurban_ joined, dmalcolm left 21:36 rurban left 21:37 rurban_ is now known as rurban
rfw wh 21:37
argh
cotto_work whargh
anyone think it'd be a bad idea for me to merge embed_api5? 21:42
rfw yay 21:45
managed to cover autobase
21:47 smash left
rfw anyone available: github.com/parrot/parrot/pull/40 and www.google-melange.com/gci/task/sho...9295757244 21:49
plobsing cotto_work: I was going to attempt to fix all the checkdepends tests to allow for just that. If you are willing to let those tests slide, all the better. 21:50
21:53 jan_ joined
plobsing vtable_set_pmc on FPA can cause it to change size. that seems wrong to me. 21:54
cotto_work merginated 21:55
dalek rrot: 1a76c9b | Whiteknight++ | / (9 files):
Merge branch 'master' of github.com:parrot/parrot
21:56 jan left, jan_ is now known as jan
cotto_work msg whiteknight I went ahead and merged embed_api5. If you're happy, please delete the old embed_api* branches. 22:07
aloha OK. I'll deliver the message.
Matt_ I'm getting a couldn't read 'src/extend_vtable.c': No such file or directory at lib/Parrot/Headerizer/Functions.pm line 124. error when running make headerizer. Is this a known issue? 22:08
dalek rrot: 99ba321 | plobsing++ | lib/Parrot/Distribution.pm:
remove some references to files that no longer exist
plobsing Matt_: (1) I don't have that issue. (2) that file *should* exist. 22:14
you should run 'make' before running the headerizer?
s/you/maybe you/
rfw could someone look over my role.t test case to cover make_namespace_autobase? 22:15
cotto_work rfw: merging 22:23
rfw cotto_work: thanks
could you approve the task too
cotto_work sure 22:24
dalek rrot: c8e0332 | (Tony Young)++ | t/pmc/role.t:
Added a test in role.t to cover make_namespace_autobase.
22:26
rfw got it, thanks 22:28
22:37 Khisanth joined
nwellnhof msg moritz thanks for applying my Rakudo patches 22:55
aloha OK. I'll deliver the message.
22:57 kid51 joined, fperrad left 23:00 contingencyplan left
kid51 If I "nice" a command-line program, and if that program calls several other programs (e.g., 'make' 'make test') are those other programs "nice-d" as well? 23:05
cotto_work: ping 23:06
23:06 Khisanth left
cotto_work kid51: pong 23:06
kid51 The conditions you msg-ed me about -- they're still broken?
cotto_work kid51: can you remind me what I msg'd you about? 23:07
kid51 make cover not failing gracefully when Devel::Cover not installed
cotto_work ah
I think that's fixed now.
kid51 Yeah, I saw some commits to that today. I will review. 23:08
cotto_work It should also be portable.
kid51 (I expected problems like that.)
cotto_work It's a bit hacky. If you can think of a more elegant way to fail quickly I'm open to it. 23:09
nwellnhof kid51: regarading your "nice" question: yes, i tihnk so.
kid51 nwellnhof: Thanks.
cotto_work: Yes, I didn't know how to do an IF block in a Makefile. (and don't, really) 23:10
Matt_ cotto_work: Does a Parrot_api_pmc_box_string exist in src/embed/api.c ? I grep'ed all the embed_api* branches and got no results yet: 23:12
"Update the function Parrot_api_pmc_box_string in src/embed/api.c to use your new function"
cotto_work Matt_: src/embed/pmc.c 23:14
Matt_ cotto_work: thanks! 23:16
23:27 Khisanth joined
Matt_ cotto_work: Can you do a quick review of github.com/parrot/parrot/pull/41 23:29
and github.com/parrot/parrot/pull/42 (on the embed_api2 branch) 23:30
cotto_work Matt_: I just merged embed_api5 23:31
23:33 Khisanth left 23:41 Yuki`N joined
Matt_ cotto_work: what do you want me to do? 23:41
cotto_work Matt_: you don't need to do anything. I 23:48
'll just apply the diffs manually.
Yuki`N dukeleto, ping 23:50
nopaste "kid51" at 192.168.1.3 pasted "[codingstd] checkdepends: Possible solution" (112 lines) at nopaste.snit.ch/27430 23:57
NotFound plobsing: ping 23:59
kid51 Coke, whiteknight: That patch may solve that codingstd failure. With it I get a PASS on 'make test' and 'make codetest'
cotto_work kid51: go for it if it doesn't break anything
dalek nxed: r722 | NotFound++ | trunk/winxedst1.winxed:
initial support for volatile in lexical var
kid51 Cf: smolder.parrot.org/app/projects/rep...tails/1833