<gavin_> huh
<gavin_> mconnor: I don’t understand 347402 comment 29
<gavin_> the go button issue was my fault?
* rob_strong can’t imagine gavin ever doing that
<gavin_> hrm, actually, the problem is in http://screwedbydesign.com/mozilla/rc.diff but apparently not in any of the patches on the bug
* gavin_ blames Ryan ![]()
<Mossop> haha
<poningru> hehe
* poningru shakes fist at ryan
* rob_strong wants some of what ryan was smoking when he wrote that
<Mossop> As I recall there was an xul error in the original patch which is probably where the mistake crept in
<Ryan> Wait… what!? I’m to blame for the go button?
<Ryan> I did no such thing ![]()
<Mossop> Ryan: Yeah its all your fault, we decided
<Ryan> ![]()
<Mossop> Ryan: Feel free to try to prove your innocence ![]()
<Ryan> I didn’t even touch that part of the patch!
<Ryan> Though it’s worth noting that that patch was against the trunk ![]()
<Lucy_> mconnor blames gavin ![]()
<Lucy_> but that might have changed since last night
<Ryan> Well gavin did check in my patch ![]()
…
<Ryan> Woo. It was totally my fault ![]()
<Mossop> lol
<Ryan> I moved the wrong line in the original patch :/
<Ryan> But in my defense, it wouldn’t have happened if it was a unified diff with context!
<Mossop> Ah here come the excuses ![]()
<Ryan> Dude, have you seen the other patch? ![]()
<Ryan> https://bugzilla.mozilla.org/attachment.cgi?id=231885
<Mossop> Ok maybe I’ll let you off
<Ryan> Yay!
* Ryan hides from mconnor
<mconnor> Ryan: smooth ;P
<Ryan> eep
<mconnor> Ryan: typical sysadmin “looks right, must be right” ![]()
<Lucy_> lol
<Ryan> ![]()
<Ryan> Hey, it worked okay too!
<mconnor> pfft
Oops