|
Parrot 1.0 Released | parrot.org | Thanks to everyone who helped get 1.0 out the door! Set by moderator on 18 March 2009. |
|||
|
00:01
mikehh joined
|
|||
| dan | What does this mean when I try to use target=parse: "load_bytecode" couldn't find file 'library/Data/Dumper.pir' | 00:01 | |
|
00:09
AndyA joined
|
|||
| kid51 | From: 'make help': | 00:14 | |
| Installation: | |||
| install: Install under '/usr/local' on Unix systems | |||
| install-dev: Same as 'install' but also install support for development. | |||
| dan | kid51: It worked, thanks. | 00:15 | |
|
00:37
Whiteknight joined
01:05
tetragon_ joined
01:20
NotFound joined
|
|||
| NotFound | hi | 01:20 | |
| simontwo | hello | 01:34 | |
|
01:34
dukeleto joined
|
|||
| megazooch | ighto | 03:02 | |
| :O | |||
| dukeleto | there does not seem to be a parrot built-in for getting the polar representation of a Complex, am I missing something or is on purpose? | 03:08 | |
| Infinoid | If you feel that's something useful, you can always submit a patch. :) (I don't deal with Complexes much, so I wouldn't know) | 03:12 | |
|
03:35
janus joined
03:51
TiMBuS joined
04:05
Counterspell joined
|
|||
| dukeleto | Infinoid: would I be adding a new method to the Complex pmc vtable? | 04:34 | |
| cotto | dukeleto, we're trying to refer to VTABLE entries as functions rather than methods. If you see documentation referring to VTABLE methods, it's a bug. | 04:54 | |
| dukeleto | cotto: good to know | 04:55 | |
| cotto | For what you want, you'd add a METHOD to the Complex PMC, similar to Complex's ln. | 04:56 | |
| There should be plenty of similar code there to give you the general idea. | 04:57 | ||
| Hmmm. Apparently our documentation is quite buggy. | 05:01 | ||
|
05:02
msmatsko joined
|
|||
| cotto | Also, don't forget to add tests when you submit the patch. | 05:08 | |
| 'vtable entries' is fine too. | 05:13 | ||
| though not preferred | 05:14 | ||
|
05:44
japhb joined
07:15
uniejo joined
07:29
contingencyplan joined
|
|||
| cotto | (make headerizer)++ | 08:20 | |
| dalek | rrot: r37644 | cotto++ | trunk (7 files): [hash] change Parrot_new_INTVAL_hash to parrot_new_intval_hash, make it return a Hash* |
08:46 | |
| mikehh | t/codingstd/trailing_space.t failure - I have a patch - should I post it | 08:51 | |
|
08:53
masak joined
|
|||
| cotto | I think I just fixed it with that commit. | 08:53 | |
| irony++ | |||
| mikehh, if that commit didn't catch it, feel free to nopaste | 08:54 | ||
| dalek | rrot: r37645 | cotto++ | trunk/src/hash.c: [hash] move similar functions together, reformat code (no functional changes) |
08:55 | |
| mikehh | cotto: I am about to extract all the compiler warnings from make world - do you think that a worthwhile project? | 09:43 | |
| cotto | mikehh, I don't think it'd hurt. We generally try to avoid compiler warnings. | 09:59 | |
| I don't think you'll find very many. | 10:00 | ||
| either way, it's long past bed time. night | 10:01 | ||
| mikehh | I've got about a dozen or so with gcc 4.3.2 on Kubuntu Intrepid i386 | ||
| night - it's monrning for me | 10:02 | ||
| s/monrning/morning/ | 10:03 | ||
| dalek | rrot: r37646 | moritz++ | trunk/src/ops/experimental.ops: [cage] try to clean up POD errors from experimental.ops |
10:10 | |
|
10:23
bacek joined
10:40
Piper joined
|
|||
| Piper | Hi there. I am Piper. I am now publicly logging this channel. If you don't want to be logged, please leave now. | 10:40 | |
|
10:47
Piper joined
|
|||
| Piper | Hi there. I am Piper. I am now publicly logging this channel. If you don't want to be logged, please leave now. | 10:47 | |
| masak | Piper: for a bot, you're awfully talkative. >:-( | 10:50 | |
| and a bit rude, too. | |||
| moritz | especially for a bot that's supposed to listen, not to talk | ||
| masak | oh well. it's early in bot history still. | ||
|
10:51
cognominal joined
10:52
alvar joined
11:05
clunker3 joined
11:23
fperrad joined
11:41
ruoso joined
12:15
bacek joined
12:47
alvar joined
13:03
gryphon joined
13:16
PerlJam joined
|
|||
| Coke | ~~ | 13:26 | |
| warnings: trac.parrot.org/parrot/wiki/BuildWarnings | 13:29 | ||
| RT down, presumably due to the perl.org server migration. | 13:36 | ||
| moritz | but the RT pages now load faster than ever before :-) | ||
|
13:43
riffraff joined
|
|||
| Coke | "you funny, doctah jones." | 13:51 | |
| Infinoid | Coke: That's for unfixable warnings? Or just a random place to stick the fixable stuff? | 13:54 | |
| Coke | all of the above? | 13:55 | |
| Infinoid | (I've got some pthreads and gdbm warnings in my system headers which have been around for a long time, and aren't parrot's fault.) | ||
| Coke | I think certainly for the unfixable. I think the other ones are debatable, but having a central page for them makes sense to me. | 13:56 | |
| (especially since warnings aren't failures.) | |||
| (except when they are elsewhere) | |||
| Infinoid | and the yacc/lex warnings! yay. | 14:02 | |
| mikehh | I get 27 warnings in 13 files - Kubuntu Intrepid i386 gcc version 4.3.2 (Ubuntu 4.3.2-1ubuntu12) | 14:04 | |
| Infinoid | Coke: I'm not really sure how you intend to format that file to keep it manageable, but my warnings are up, please move things around however you like. | 14:05 | |
|
14:05
Andy joined
|
|||
| dalek | tracwiki: v2 | Infinoid++ | BuildWarnings | 14:06 | |
| tracwiki: Add the warnings I regularly see (and ignore). | |||
| tracwiki: trac.parrot.org/parrot/wiki/BuildW...ction=diff | |||
| shorten | dalek's url is at xrl.us/bek9rs | ||
| Infinoid | Some of those items might already have tickets. | 14:11 | |
| dalek | rrot: r37647 | fperrad++ | trunk/lib/Parrot/Distribution.pm: [tools] fix RT #43691 |
14:28 | |
| Coke | Infinoid: added the ones I have on top of that. | 14:40 | |
| I'd put in the RT, but rt down. | |||
| dalek | tracwiki: v3 | coke++ | BuildWarnings | 14:42 | |
| tracwiki: trac.parrot.org/parrot/wiki/BuildW...ction=diff | |||
| shorten | dalek's url is at xrl.us/bek9ui | ||
| Coke | ... I just had a terrible idea to rename TWIP. | ||
| pmichaud | good morning #parrot | 14:46 | |
| 13:51 <Coke> "you funny, doctah jones." | 14:47 | ||
| Odd, we watched that episode just two nights ago... :-| | |||
| dalek | tracwiki: v4 | mikehh++ | BuildWarnings | 14:51 | |
| tracwiki: trac.parrot.org/parrot/wiki/BuildW...ction=diff | |||
| shorten | dalek's url is at xrl.us/bek978 | ||
| Coke | pmichaud: heh. | 14:58 | |
| timbunce++ (I can now run Devel::NYTProf against pmc2c.pl) | 14:59 | ||
| sadly, no obvious "FIXME"s leap out. | |||
|
15:01
ruoso joined
|
|||
| particle | io bound, then? | 15:02 | |
| Coke | possibly. | 15:03 | |
| davidfetter | seen pmichaud | 15:05 | |
| purl | pmichaud was last seen on #parrot 17 minutes and 49 seconds ago, saying: Odd, we watched that episode just two nights ago... :-| | ||
| davidfetter | pmichaud, is anybody working on getting rakudo working with a pre-installed parrot? | ||
| Coke | I was under the impression that already worked. | 15:06 | |
| (and was used as the basis of a patch to partcl which I haven't applied) | |||
|
15:13
PacoLinux joined
|
|||
| szabgab | I've started to write this Inline::Rakudo module that would load perl6.pbc and then allow the user to execute perl 6 code from perl 5 | 16:13 | |
| the problem is that when I call load_bytecode '$ENV{RAKUDO_DIR}/perl6.pbc' | |||
| it complains about not finding PCT.pbc | 16:14 | ||
| unless I happen to be in the parrot directory | |||
| can I tell in any way parrot to look for the PCT.pbc and other files in a specific directory? | 16:15 | ||
| Coke | are you running against an installed parrot? | 16:27 | |
| szabgab | not, one that was built by Rakudo using --gen-parrot | 16:28 | |
| BTW here is the code: github.com/szabgab/perl6-in-perl5 | |||
|
16:29
rg joined
16:53
magnachef_ joined
|
|||
| pmichaud | davidfetter: there have been several folks who have wokred on getting Rakudo to work with an installed parrot, but I don't know the current status of those efforts. | 17:02 | |
| afaik it doesn't 'already work' | |||
| szabgab: Parrot should already be smart enough to know where to locate PCT.pbc -- if not, that sounds like a bug in parrot. | 17:03 | ||
| davidfetter | pmichaud, k. btw, is rakudo in a state where it's worth packaging it? | ||
| (ports, debs, RPMs...) | |||
| pmichaud | davidfetter: short answer: yes | 17:04 | |
| longer answer: we definitely want to know what we need in order to package it. | |||
| (i.e., if there are any major stumbling blocks to packaging it) | |||
| davidfetter | well, getting it working with an installed parrot would be a big one | ||
| so you can list parrot (or parrot-dev, or whatever) as a dependency | 17:05 | ||
| pmichaud | I haven't seen (haven't looked for) documentation that describes how dynpmcs and dynops are to be generated now. Rakudo is still using the old methods. | ||
| davidfetter | hrm. that'd be an "upstream" issue, i suppose | 17:06 | |
| !dynpmc | |||
| hrm | |||
| pmichaud | allison submitted a patch towards the end of february that was supposed to address it, but that patch was before I did the big refactor of Rakudo's configure/build process | 17:07 | |
| I haven't had time to review the patch and figure out exactly what it was attempting to do. | |||
| (and the patch is based on the old build environment, so it's now a fair bit out of date.) | 17:08 | ||
| davidfetter | dynops? | ||
| purl | dynops is, like, definitely borked on windows. i forgot about that | ||
| davidfetter | hrm | ||
| szabgab | pmichaud, or the way rakudo is building it with the --gen-parrot flag, or the way I stup things | ||
| davidfetter | dynpmcs? | ||
| pmichaud | szabgab: when rakudo builds parrot with the --gen-parrot flag, it's just doing a standard developer build. | 17:09 | |
| it's not doing any thing out of the ordinary as far as building parrot goes. | |||
| szabgab | maybe it should :-) | ||
| though I guess the standard parrot build should work too | 17:10 | ||
| jonathan | rakudobug? | 17:13 | |
| purl | it has been said that rakudobug is mailto:rakudobug@perl.org | ||
| sjn | pmichaud: did you make a hotel reservation in Oslo? | 17:14 | |
| pmichaud: only hotel name I have registered here is "Best Western" which is kinda silly since there are probably 15-20 of them in Oslo | 17:15 | ||
| pmichaud | sjn: yes, I made a reservation -- I'm at the BW Anker | 17:16 | |
| (also listed at www.perlworkshop.no/npw2009/wiki?no...StaysWhere ) | |||
| jonathan: where are you staying while in Oslo? | 17:17 | ||
| jonathan | pmichaud: I *think* the same one...I was going to check that with batman | ||
| I know it's reserved. | 17:18 | ||
|
17:25
n8willis joined,
Khisanth joined
17:26
NordQ joined
|
|||
| jonathan | pmichaud: Best Western Anker Hotel | 17:28 | |
| sjn | pmichaud: great | 17:33 | |
| Coke misses RT. | 17:42 | ||
| pmichaud | jonathan: maybe add yourself to www.perlworkshop.no/npw2009/wiki?no...StaysWhere ? | 17:43 | |
| jonathan | pmichaud: Will do. | 17:44 | |
|
18:07
Woody4286 joined
|
|||
| dalek | rrot: r37648 | allison++ | trunk/docs/pdds/pdd22_io.pod: [pdd] Documenting finalized network I/O decision. |
18:26 | |
| cotto | seen wknight8111 | 18:27 | |
| purl | wknight8111 was last seen on #parrot 5 days, 23 hours, 14 minutes and 38 seconds ago, saying: is too much of a wuss to quit his job [Mar 17 19:11:41 2009] | ||
| cotto | mikehh, ping | 18:28 | |
|
18:39
barney joined
18:41
protorom joined
|
|||
| cotto | hi barney | 18:41 | |
| barney | hi cotto | 18:48 | |
| mikehh | cotto: hi | ||
| barney is away for dinner | 18:53 | ||
| cotto | mikehh, in the page on compiler warnings you mention one for some OS X functions that already has an rt. | 18:55 | |
| The page should have a link to the rt. | |||
| mikehh | I think that was Coke on the OS X stuff | 18:56 | |
| I just added my gcc 4.3.2 bits | 18:57 | ||
| cotto | Right you are. I'll bug Coke about it. | 18:59 | |
| Coke, ping | |||
| Coke | cotto: pong | 19:06 | |
| cotto: RT is DOWN, or I would have linked to the rt. =-) | |||
| s/is/was/ | 19:07 | ||
| ... and in the text, it says, " (There's an RT for this one)" sheesh. =-) | 19:08 | ||
| cotto | no excuses now ;) | ||
| Coke | fixed | 19:11 | |
| dalek | tracwiki: v5 | coke++ | BuildWarnings | 19:13 | |
| tracwiki: trac.parrot.org/parrot/wiki/BuildW...ction=diff | |||
| shorten | dalek's url is at xrl.us/bembft | ||
| cotto | thanks | 19:14 | |
|
19:17
Tene joined
|
|||
| mikehh | one of the longstanding warnings in src/thread.c - suggest braces around empty body in do 'statement' is: CLEANUP_POP(1); | 19:23 | |
|
19:23
contingencyplan joined
|
|||
| mikehh | so far I havewn't been able to find this macro | 19:23 | |
| Coke | use ack. | 19:25 | |
| ack -a CLEANUP_POP => | 19:26 | ||
| include/parrot/thr_pthread.h ; include/parrot/thr_windows.h | |||
| and include/parrot/thread.h | |||
| mikehh | ah - I was looking in the parrot includes not the c library | 19:27 | |
| Andy | better still, ack for cleanup_pop, which will be case-insensitive | ||
| particle | or make tags-vi && vim -t CLEANUP_POP | ||
|
19:29
desertmax joined
|
|||
| dalek | lectus: b482db0 | (Bernhard Schmalhofer)++ | t/pmc/boolean.t: add tests for stringification and numification |
19:33 | |
| shorten | dalek's url is at xrl.us/bembiy | ||
| dalek | lectus: 41012b4 | (Bernhard Schmalhofer)++ | t/pmc/boolean.t: include/ is in the searchpath |
||
| shorten | dalek's url is at xrl.us/bembi2 | ||
| Coke | particle: does that work when it has multiple definitions? | 19:34 | |
| Andy: wouldn't you need ack -i ? | |||
| Andy | nope | ||
| ack assumes -i if you have no caps | |||
| see --smart-case | |||
|
19:34
protorom left
|
|||
| Coke | hurm. my initial reaction is 'ew". | 19:34 | |
| with mismatched quotes, even. | |||
| doesn't seem to be present in my ancient 1.80 | 19:35 | ||
| Andy | so throw a --no-smart-case in your .ackrc | ||
| Coke | defines are case sensitive anyway, aren't they? | 19:36 | |
| Andy | sure | ||
| but who knows what generated it. | |||
| Just in general -i is good I've found | |||
| particle | it brings you to the first, then you navigate to the others with :ts 2 or whatever | 19:37 | |
| Coke | Andy,particle: nifty | ||
| Coke updates his ack | |||
| mikehh | # define CLEANUP_POP(a) pthread_cleanup_pop(a) | 19:45 | |
| dalek | pp: c85f726 | (Bernhard Schmalhofer)++ | (4 files): Merge branch 'master' of git@github.com:bschmalhofer/pipp |
||
| shorten | dalek's url is at xrl.us/bembkd | ||
| dalek | pp: 812a445 | (Bernhard Schmalhofer)++ | t/pmc/ (3 files): include is in the search path |
||
| shorten | dalek's url is at xrl.us/bembkj | ||
| Coke | wierd. I have pod simple 3.05 installed. I install 3.07 ; in cpan, i type 'r' ... shows me I have 3.05 installed and that 3.07 is available. | ||
| mikehh | Coke: I have had this problem before - check your install directories | 19:54 | |
| Coke | "they exist!" =-) | ||
| There are two copies of Pod::Simple; I presume one is the core, one is my upgrade. | |||
| annoying that perl+cpan aren't DTRT. | 19:55 | ||
| mikehh | you will probably find that the new version was installed but in a different directory - lower in the search heirarchy | ||
| barney | is get_string() still the VTABLE for stringification ? | 19:57 | |
| Coke | barney: SFAIK. | 19:58 | |
| barney | In Eclectus I seem to get a different stringification than implemented in get_string() | 20:00 | |
| dalek | lectus: 4a8ed6d | (Bernhard Schmalhofer)++ | (2 files): add implementation and test of name() |
||
| shorten | dalek's url is at xrl.us/bembor | ||
| Coke | barney: code sample? | 20:03 | |
| dalek | rrot: r37649 | NotFound++ | trunk/src/parrot_debugger.c: [debugger] allow running parrot_debugger without a file to debug |
20:04 | |
| barney | github.com/bschmalhofer/pipp/blob/8.../boolean.t | ||
| shorten | barney's url is at xrl.us/bembpm | ||
| NotFound | barney: I fixed a thing like that in ecmascript some weeks ago. | 20:06 | |
| dalek | rrot: r37650 | cotto++ | trunk/src (5 files): [hash] use get/set_pointer instead of PMC_struct_val on Hash-like PMCs |
20:08 | |
| barney | get_string() in ecmascript/src/classes/Boolean.pir ? | 20:09 | |
| NotFound | barney: Yes | 20:10 | |
| barney | (np preserved svn history)-- | 20:12 | |
| (non preserved svn history)-- | |||
| found it. POD wasn't protected by C-comments | 20:13 | ||
| NotFound | barney: trac.parrot.org/languages/browser/...oolean.pir | 20:15 | |
| shorten | NotFound's url is at xrl.us/bembrf | ||
| dalek | lectus: 79b14fd | (Bernhard Schmalhofer)++ | (2 files): fix parsing of *.pmc file, by adding a C startcomment |
20:18 | |
| shorten | dalek's url is at xrl.us/bembrs | ||
| barney | Eclectus is (mostly) back, smolder.plusthree.com/app/public_pr...ails/19325 | 20:19 | |
| shorten | barney's url is at xrl.us/bembru | ||
| dalek | rrot: r37651 | allison++ | trunk/ports/cpan (2 files): [ports] Adding CPAN release instructions as a separate guide. |
20:21 | |
|
20:23
allison joined
|
|||
| dalek | rrot: r37652 | allison++ | trunk/ports/cpan/pause_guide.pod: [ports] Moving PAUSE documentation into CPAN ports directory. |
20:25 | |
| Coke | svn:up::git: ? | 20:28 | |
| pull? | |||
| purl | KABLAMM! or English slang for snogging/kissing/taking someone home | ||
| allison | purl: forget pull | ||
| purl | allison: I forgot pull | ||
| dalek | rrot: r37653 | allison++ | trunk/MANIFEST: [ports] Moving PAUSE guide to CPAN ports directory. |
20:29 | |
| Tene | Coke: pull is okay. I usually prefer git fetch; git rebase origin; | 20:30 | |
| allison | Coke: I've been using 'git pull' (there seem to be several different equivalents, depending on how you have your local git set up) | ||
| Tene | In the case that you don't have extra commits on the local tree, they're identical. | ||
| Coke | origin? | ||
| purl | i heard origin was lib/Parrot/Config.pm | ||
|
20:31
particle joined
|
|||
| Coke | origin is also a made up religion from the Stargate universe. | 20:31 | |
| purl | okay, Coke. | ||
|
20:32
ujwalic joined
|
|||
| allison | Tene: how do I wipe out all local commits and reset to the foreign master? "git reset --hard"? | 20:34 | |
| Infinoid | git reset --hard origin | 20:35 | |
| or origin/master, if you want to be ultraspecific | |||
| Coke | I for one am glad we didn't switch parrot over to git just yet. ^_^ | 20:36 | |
| Infinoid | I think it would have done parrot more good than it did rakudo, to be honest | ||
| allison now wonders what "HEAD is now at 1c263b0 implement Match.chunks" means... | 20:37 | ||
| Coke | Infinoid: long term, no doubt. | ||
| allison | short-term it would have foobar'd us for the 1.0 release | ||
| Infinoid | we have branch merging problems *now*, that is my only reason for caring about git | ||
| Coke | allison: let me pre-empty the git folks: | ||
| now that we've released 1.0 ... can we switch to git? =-) | |||
| allison | no :) | ||
| Coke | ... pre-empty? there's an odd slip. | 20:38 | |
| mikehh | agreed | ||
| allison | ("pre-empty": but gives it a nice round feel) | 20:39 | |
| barney | Shouldn't string_chr() be Parrot_str_chr() ? | ||
| allison | lots of reasons, but the bottom line is that we've taken too many hits from infrastructure changes in the past 6 months, and need to be focusing on other things | 20:40 | |
| Infinoid | ...or at least focusing on fixing the issues with the new infrastructure we already have | ||
| mikehh | at least in svn you can refer to r37645 or whatever, with git -- wow | ||
| allison | most of our failed merges are user-error, not SVN's fault, and git provides even fewer safety nets | ||
| gravity | mikehh: You only need the first part of the git hash to refer to a change, so it's not really any worse than an svn rev in practice. | 20:42 | |
| allison | (I do, though, completely think git was the right choice for Rakudo and Cardinal, and several other languages. It's all about appealing to a particular language community.) | ||
| mikehh | don't agree - at least svn is sequential | ||
| barney | For PHP I should have switched to CVS | ||
| Infinoid | I usually don't bother with the hashes, we have named branches for that. | ||
| Tene | mikehh: I always copy/paste revisions anyway. the format of the revision id isn't really relevant. | 20:43 | |
| It's a double-click regardless | |||
| Infinoid | the svn merging issues I'm referring to were when using the proper tools in the proper way, it still fails with an obscure error message. I've seen this happen with two branches so far, repeatably, and I don't think it was user error. | 20:44 | |
| allison | barney: string_chr is included in the deprecation list of "functions not named "Parrot_<subsystem>", but it may be going away entirely (it doesn't make much sense for non-ASCII character sets) | ||
| Infinoid | I think svn has some issues with files changed in branch and renamed in trunk | ||
| Coke | yes, that's a known svn issue. :| | 20:45 | |
| mikehh | I mean I have rakudo 1c263b0 and before that it was something like 44e1496 | ||
| allison | Infinoid: yes, if you rename a file in a branch, and then keep the branch around a long time, any subsequent changes to that file in trunk won't be merged in when you merge the branch into trunk | ||
| Infinoid: the solution, do file renaming as a short-lived branch, merge it into trunk, then start the larger refactors in a second branch | 20:46 | ||
| Infinoid | it's not that the changes won't be merged. It's that svn aborts the merge and leves the checkout in a trashed state | ||
| But overall, you're right | |||
| allison | Infinoid: I've successfully done it several times | ||
| Coke | Infinoid: IME, it just throws out the merges, and leaves it in a workable state that is missing the changes. | 20:47 | |
| very (*#&$ annoying. | |||
| allison | Infinoid: and, an aborted merge just needs a recursive revert to be restored to a completely clean state | ||
| Infinoid | Ok. maybe your version of svn is spiffier than mine then | ||
| Coke | Apparently truMerge works around that issue. | ||
| latest rakudo fails to build for me. | 20:48 | ||
| Unable to open filehandle from path 'src/gen_actions.pir' | |||
| (guessxing that's the PCT.pbc issue) | |||
| allison | Coke: interesting, hadn't looked at truMerge before, will look at it more | ||
| dalek | lectus: 98f7075 | (Bernhard Schmalhofer)++ | src/pmc/eclectuscharacter.pmc: fix get_string() in EclectusCharacter |
||
| shorten | dalek's url is at xrl.us/bembv2 | ||
| Coke | allison: was bitten by this bug at work, found that along with the bug report at svn. | 20:49 | |
| barney | allison: In Scheme there is a character class that is basically in Int, for printing it I need to convert the Int to a String | ||
| Coke | (nearly every file in our repository is named wrong.) | ||
| (and I can't fix it on a branch. or really while we have branches) | 20:50 | ||
| allison | Coke: at the end of the day, we just have to bite the bullet and do it | ||
| Coke | convert to git? ok. =-) | ||
| allison | Coke: when I do file renaming in a branch, I just check the commit log of that file since the branch was made, and make sure I merge in any trunk changes before merging the branch back to trunk | 20:51 | |
| Infinoid | Uh, no. | 20:52 | |
| I have personally resolved two broken branch merges where the branch author had merged trunk changes into the branch and then everything was looking great... and then svn croaked merging the branch back into trunk. | |||
| (And that has nothing to do with renaming, as far as I know) | |||
| allison | I've resolved some pretty hairy ones too | 20:53 | |
| Coke | Infinoid: yup, that sounds different. | ||
| Infinoid | I rather like pmichaud's approach of *never* merging trunk changes into the branch, which keeps the history linear | ||
| allison | the worst are when the branch developer merges in trunk changes a piece at a time | ||
| Infinoid | But this is all just working around a half-broken tool, which I still don't like. | ||
| allison | Infinoid: pmichaud's approach has problems too | ||
| Infinoid: git has problems too | 20:54 | ||
| Coke | (a piece at a time) but how else would you do that if you're going to keep the branch for longer than a <too short amount of time> ? | ||
| allison | Coke: you do a complete merge from trunk a particular revision | ||
| Infinoid | allison: git matches the branch workflow better. | ||
| barney | Eclectus is back, smolder.plusthree.com/app/public_pr...ails/19326 | ||
| shorten | barney's url is at xrl.us/bembw5 | ||
| mikehh | Ubuntu is using bzr | 20:55 | |
| Infinoid | Thing is, if you merge the trunk changes into the branch, that's the only necessary merge. Pushing that back into trunk doesn't merge again | ||
| So it's really frustrating that the branch author had already *done* the merge, but svn refused to pull the results of that back into trunk | |||
| allison | Coke: that's clean, and keeps it up to date in documented chunks (that is, the next time you do a merge from trunk, you do it from the revision of the last merge to the current HEAD, and don't try to repeat earlier merges) | ||
| Infinoid | So from my perspective, git doesn't have that problem. It may have different problems, I suppose | 20:56 | |
| Coke | allison: ok. I thought that's what you meeant by a piece at a time. | ||
| allison | mikehh: I like bzr, and would put it on the list of considerations in a couple of years when we think about infrastructure changes again | ||
| mikehh | :-} | 20:57 | |
| allison | Coke: ah, ok, I meant "taking changes from one file here and there, and applying them to trunk" | ||
| I mean "applying them to the branch" | |||
| Coke | barney: ping. | 20:59 | |
| dalek | rrot: r37654 | barney++ | trunk (3 files): [dynpmc] Removing the unused Pair PMC |
||
| mikehh | I think git is a lot like C - iy assumes you know what you are doing - which is fine for kernel hackers but I am not sure if it is that safe | ||
| Coke | barney: there is no long a deprecation notice for the Pair PMC - so we can't rip it out until 1.5 | 21:00 | |
| "no longer" | |||
| mikehh | s/iy/it/ | ||
| Coke | msg barney there is no long a deprecation notice for the Pair PMC - so we can't rip it out until 1.5 | ||
| purl | Message for barney stored. | ||
| barney | That's also valid for undocumented dynpmc's ? | 21:01 | |
| Coke | msg barney (also, removing a PMC invalidated PBC_COMPAT, which you didn't update there. for future reference) | ||
| purl | Message for barney stored. | ||
| Coke | oh, right, PBC_COMPAT prolly doesn't care about dynpmcs. | ||
| but yes, it needs a deprecation notice. | |||
| unless it's covered by another notice in there... | 21:02 | ||
| (doesn't seem to be.) | |||
| Infinoid | mikehh: How isn't it safe? If you screwed up your tree, just don't commit :) | ||
| mikehh | Infinoid: but what happens if you do - commit O mean | 21:03 | |
| this keyboard is driving me nuts | |||
| Infinoid | mikehh: there's always reset for the local case and revert for the remote case, I suppose | 21:04 | |
| The kernel folks don't really have that problem. Almost all their contributors don't have commit access, they have a great email review process | |||
| gravity | Speaking of which, is there a mailing list where commit diffs go to, or just irc? | 21:05 | |
| Infinoid | lists.parrot.org/pipermail/parrot-c...hread.html | ||
| shorten | Infinoid's url is at xrl.us/bembzm | 21:06 | |
| gravity | Ah, thank you! | ||
| mikehh | Infinoid: I was involved in a project using git and there was this clown - (who was a senior exec) - well never mind but I spent more time cleaning up after him than being productibe | 21:08 | |
| Infinoid | That sounds nasty regardless of which vcs was in use :( | ||
| mikehh | You better believe it | 21:09 | |
|
21:09
Whiteknight joined
|
|||
| mikehh | ome of my grandsons spilt a coke over my keyboard and I am having difficulty getting used to this new one | 21:11 | |
| Infinoid | :) different key springiness always trips me up | ||
| mikehh | you see s/ome/one/ | ||
| Coke | (split a coke) my daughter kicked over the cola I had near the imac keyboard. Grumble. | 21:12 | |
| mikehh | this is a slightly different size and I keep hitting the wrong key (next to it) | 21:13 | |
| Coke | SPILLED! | ||
| mikehh | the cola version not Will | ||
| Infinoid | I tend to work heavily off of a heirarchy of local checkouts, so in a case like that, I probably just wouldn't pull upstream changes until the end of a ticket/task | 21:14 | |
| effectively forking the project | |||
| tools like monotone let you control whose changes to trust / merge, maybe that situation is why | 21:17 | ||
| dalek | rrot: r37655 | barney++ | trunk (3 files): [dynpmc] readding the Pair PMC, removal needs a deprecation period |
||
|
21:30
mj41 joined
21:38
bsdz joined
|
|||
| dalek | rkdown: 1fbf6a8 | (Francois Perrad)++ | (5 files): implement code block |
21:42 | |
| rkdown: 13ac70e | (Francois Perrad)++ | (3 files): fix a TODO test |
|||
| shorten | dalek's url is at xrl.us/bemb7d | ||
| dalek's url is at xrl.us/bemb7f | |||
| dalek | rrot: r37656 | cotto++ | trunk/src/pmc/hash.pmc: [PMC] replace most instances of PMC_struct_val from the Hash PMC, plus simplify code thanks to a saner hash API |
21:53 | |
|
22:05
Whiteknight joined
|
|||
| GeJ | Good mornin g everyone | 22:29 | |
| cotto | hi | 22:30 | |
| purl | hola, cotto. | ||
| dalek | rrot: r37657 | cotto++ | trunk/src/pmc (4 files): [PMC] replace most remaining PMC_struct_val instances in Hash-like PMCs, remove some redundant code |
22:39 | |
| kudo: 7487710 | (Moritz Lenz)++ | t/spectest.data: Add S05-metasyntax/repeat.t to t/spectest.data |
22:43 | ||
| shorten | dalek's url is at xrl.us/bemcff | ||
|
23:14
tetragon joined
23:20
Whiteknight joined
23:32
eternaleye joined
|
|||