=== teknico__ is now known as teknico === bcsaller__ is now known as bcsaller [13:45] frankban, land as is. Really nice. [13:56] gary_poster: cool thanks! [13:58] :-) [13:58] hazmat, zero rush, but you might be interested in double-checking and commenting on bug 1119412 at some point. [13:58] <_mup_> Bug #1119412: charm has difficulties when run behind a firewall < https://launchpad.net/bugs/1119412 > [14:35] frankban: did I miss something? I was reviewing bug-1117554-gui-tests-in-browser and it has now been merged [14:37] benji: oh, sorry, didn't notice that and merged after 2 reviews. please complete yours if you want to, I'll create another card and fix things if required [14:38] frankban: I don't think it's your fault. I just want to know why we duplicated the effort. When I tagged the card in review there was only one other name on it, now there are three. [14:38] race condition perhaps? [14:39] benji: heh, maybe... [14:44] benji, my fault, I forgot to tag myself when I started reviewing that branch [14:44] sorry [14:45] well, at least it isn't a bug in software we don't control; expect a wetware patch in your next update [14:49] wetware is like software in that way, patching never ends :-) [14:49] too true :) [14:53] grr, I broke the sprite generation somehow [15:28] bac bcsaller benji frankban goodspud hazmat Makyo teknico call in 2 [15:31] frankban, hazmat starting without you. arosales, 10 or 15 minute warning for tinyurl.com/jujugui [15:31] gary_poster: roger [16:09] in the HACKING file it requires a package called `lbox` is it possible the package has been renamed since this doc was written? [16:09] I am not having any luck finding it [16:12] hatch: that may be in a PPA; if so you'll have to add the PPA first [16:14] probably ppa:gophers/go [16:14] ppa:gphers/go [16:14] oops you beat me [16:14] heh [16:18] hatch: please improve the docs with your new-found knowledge [16:18] oh improvements are going to be made [16:18] once I figure out how the heck bazaar works ;) [16:27] :) [16:32] up and running....wow that was pretty painless [16:38] cool [17:57] this goes over all the source files, it would be nice to get it landed soon, or it'll be conflict hell: :-) https://codereview.appspot.com/7312067 [17:57] guihelp ;-) ^^ [17:57] on it [17:57] thanks [17:57] me too [17:58] aside from its size it looks like a trivial, but a welcome one [18:01] if by "trivial" you mean "it doesn't touch executing code", then yes [18:04] thats all I meant, its a great branch [18:06] teknico, land as is. great. was the branch useful as code overview as well? [18:09] gary_poster, yes, I finally got to look at all the code, at least :-) [18:10] :-) cool [18:12] teknico: done [18:13] arosales: fwiw, i was able to create a new live acct and get the subscription linked. thanks for the pointers. [18:13] Me too [18:13] I have a win 8 iso waiting for me [18:27] thanks teknico. i understand you needed to set limits to get the thing done. [18:28] bac, thank you for the detailed review. I applied your changes, and am landing the thing [18:41] bac: sorry, I was tied up in a meeting. [18:41] arosales: np. just making that comment. [18:41] bac: ah good to hear re the license. [18:41] bac: thanks for the feedback [18:42] bac: it actually looks like a pretty good testing license too [18:57] the browser warning branch is up for review: https://codereview.appspot.com/7299071 [18:58] and I am up for some lunch [19:42] benji, branch has conflicts [19:47] gary_poster: fixed: https://codereview.appspot.com/7299071 [19:47] thanks benji [21:46] benji fwiw, seeing weird behavior--not drawing after connection, etc. trying to debug. [21:46] hmm [21:47] benji may be noise, or other weirdness. just worked in debug, but worked first in prod too... [21:48] * benji tries to figure out the emoticon for furrowed eyebrows. [21:48] >:| [21:48] that one? :) [21:49] yeah that looks pretty good :-) [21:49] could maybe throw a nose in there if you want to be a little more awesome >:-| [21:49] I'm a big fan of noses [21:52] noses are over-rated, just ask Voldemort [21:52] or snakes generally I guess [21:52] he who should not be nostriled [21:53] lol [21:53] well if you smell with your tongue that might be kind of cool [21:53] haha [21:55] heh [21:58] I really should find the $250 and 116 hours required to buy and re-listen to Jim Dale's reading of Harry Potter. So good. [21:59] *sigh* 3H trying to get my wifi back up on my laptop and no luck [21:59] looks like me and broadcom are never going to get along [22:07] benji, make debug is working for me but make prod is not on your branch. My experience is that it connects, ws sends "Switching Protocols" and then nothing happens [22:07] works in prod [22:07] I mean debug [22:07] like I said [22:07] Chrome, FF, IE [22:07] all the same [22:09] gary_poster: good catch, I'll try to figure out what is wrong with it [22:09] benji, you see it too? [22:09] gary_poster: I haven't attepmted to verify it yet. [22:10] benji, oh ok. If you duped that would make me happy because I am currently going in the "it must only be happening to me" spiral because it seemed to work at first [22:10] gary_poster: ok, let me look [22:12] gary_poster: there does seem to be some problem, in Chrome I just get the background hash and in FF I get the browser message and when I click continue I get the background hash [22:12] benji, oh, ok great [22:12] I mean not great for the branch :-P [22:12] but great for my sanity :-) [22:13] ok benji, sounds like a Monday thnkg. thanks for looking [22:13] bye all [22:14] gary_poster: later [22:14] have a good weekend