|
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 | ||