| gary_poster | bac benji frankban http://tinyurl.com/yellowsquad in 3 | 12:07 |
|---|---|---|
| gary_poster | bac you around? | 12:10 |
| bac | benji: reminder to tell me about the property rental trick | 12:27 |
| gary_poster | benji, please make sure your Google Canonical calendar is up to date with your availability for this week ASAP. The recruiter will be using that to arrange interviews this week | 13:03 |
| benji | gary_poster: I am available all week, so we're good to go. | 13:03 |
| gary_poster | fantastic thanks benji | 13:04 |
| benji | np | 13:04 |
| benji | bac: is this the best list of reviewable things we were discussing in the meeting? https://code.launchpad.net/~launchpad-reviewers/+activereviews | 14:10 |
| bac | benji: i *think* so but haven't had a chance to look yet. the alternative would be +ar on the launchpad suite. but i think gary said maas wasn't a part of the suite | 14:11 |
| benji | bac: right, I don't see any MAAS reviews in the suite+ar, but they exist in the -reviewers+ar | 14:12 |
| bac | cool | 14:12 |
| benji | however, lpsetup reviwes aren't in -reviewers+ar but are listed in suite+ar | 14:12 |
| benji | frankban: I am finally done with https://code.launchpad.net/~frankban/lpsetup/bug-1023895-init-repo-no-checkout/+merge/115108 | 15:34 |
| frankban | benji: thanks, I agree with your second comment, that is related to the first. For the first suggestion: replacing ~ with namespace.home_dir is naive. expanduser is not used there to support command run as root, with a --user option. In that case --user can also refer to a non-existent user. | 15:43 |
| benji | frankban: mmm, I was worried somethign like that was happening but after reading the code I didn't realize it; two thoughts: | 15:44 |
| benji | 1) a comment explainnig that bit so we don't trip over this again | 15:44 |
| frankban | benji: but we could replace ~ only when it is at the beginning of the string and followed by '/' | 15:45 |
| benji | 2) some paranoia in the tilda-substitution... right | 15:45 |
| frankban | benji: ok | 15:46 |
| frankban | benji: I can also try to avoid the real ~/.bazaar is used (using sheltoolbox.eviron). in that case, should I remove the BackupFile context manager? | 15:51 |
| benji | frankban: I think it would be an improvment to use a fake .bazaar, and yeah, it seems BackupFile wouldn't be needed then (and we can always resurrect it if we need it later) | 15:53 |
| benji | I'm over here now. | 15:53 |
| frankban | benji: right, thank you | 15:53 |
| frankban | benji: branch updated, could you take another look at it? no rush, it's my EOD, have a nice evening | 17:26 |
| benji | frankban: sure; have a good evening | 17:26 |
| bac | hi gary_poster, benji -- y'all still around? | 21:35 |
| gary_poster | bac I am kinda, though k just came by with Julia | 21:36 |
| gary_poster | I was trying to finish something at the time :-) | 21:36 |
| bac | gary_poster: ok, perhaps we can talk in the morning. i'm stuck getting install_lpsetup_in_lxc to checkout the branch and i cannot figure out why. bzr permission problems that usually mean you are root, though i explicitly am not. | 21:37 |
| gary_poster | bac, key is sometimes shared | 21:38 |
| gary_poster | which is kinda grody | 21:38 |
| gary_poster | but maybe we are not anymore and that is the problem? | 21:38 |
| gary_poster | IIRC... | 21:38 |
| bac | don't understand. i should mention, this is actually happening in the host not the lxc | 21:38 |
| bac | so ssh isn't involved | 21:39 |
| bac | except for bzr using it | 21:39 |
| gary_poster | oh! um. why is this happening in host? | 21:39 |
| bac | huh, i may try --use-http and see if that helps. | 21:39 |
| gary_poster | oh | 21:39 |
| gary_poster | ok | 21:39 |
| gary_poster | that would be something to fix if so | 21:39 |
| gary_poster | but if it helps you make progress... | 21:40 |
| gary_poster | I'm all in favor | 21:40 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!