[00:01:13] (03PS1) 10EBernhardson: Standards compliant reset of min/max-width [extensions/Flow] - 10https://gerrit.wikimedia.org/r/123138 [00:02:26] (03CR) 10Anomie: "It would get the next random number from old seed (and would affect the random numbers returned by the next call to foo, too). Lua doesn't" [extensions/Scribunto] - 10https://gerrit.wikimedia.org/r/122862 (owner: 10Anomie) [00:03:16] interesting. talk pages don't have language links. [00:03:32] Hm, what would the point be? [00:03:56] I've actually been seriously confused by that in the past [00:04:05] ori: https://bugzilla.wikimedia.org/show_bug.cgi?id=63399 [00:04:10] yeah. you'd think you'd go to the discussion in the other language. [00:04:33] Krinkle: :) [00:05:26] (03CR) 10Springle: "Schema change done." [extensions/Flow] - 10https://gerrit.wikimedia.org/r/115576 (owner: 10EBernhardson) [00:06:17] (03CR) 10Anomie: "Haven't looked at the code yet (I'll do that tomorrow), but my first reaction is to wonder how this is different from list=allpages&appref" [core] - 10https://gerrit.wikimedia.org/r/123118 (owner: 10MaxSem) [00:06:22] (03CR) 10EBernhardson: [C: 031] "Schema change applied, this can be merged and should go on thursdays deploy train" [extensions/Flow] - 10https://gerrit.wikimedia.org/r/122865 (owner: 10EBernhardson) [00:10:06] (03CR) 10MaxSem: "@Anomie, the default implementation is indeed not different, however when a real search engine is plugged into it, the prefix search takes" [core] - 10https://gerrit.wikimedia.org/r/123118 (owner: 10MaxSem) [00:13:16] jorm: some Beta Features discussions use Special:MyLanguage/Talk:SomePage to send you to a /fr subpage, e.g. Visionneuse de Médias [00:23:02] (03PS1) 10Ori.livneh: Change default prefix of StatsD writer [extensions/EventLogging] - 10https://gerrit.wikimedia.org/r/123142 [00:23:23] (03CR) 10Ori.livneh: [C: 032] Change default prefix of StatsD writer [extensions/EventLogging] - 10https://gerrit.wikimedia.org/r/123142 (owner: 10Ori.livneh) [00:23:50] (03Merged) 10jenkins-bot: Change default prefix of StatsD writer [extensions/EventLogging] - 10https://gerrit.wikimedia.org/r/123142 (owner: 10Ori.livneh) [00:28:35] (03PS1) 10Bsitu: Always use cache to get last revision + add rev_type to key [extensions/Flow] - 10https://gerrit.wikimedia.org/r/123144 [00:28:40] (03PS1) 10Krinkle: Zuul status page: Add support for highlighting a change by id [integration/docroot] - 10https://gerrit.wikimedia.org/r/123145 [00:29:06] (03CR) 10Krinkle: "http://i.imgur.com/AWnE8I3.png" [integration/docroot] - 10https://gerrit.wikimedia.org/r/123145 (owner: 10Krinkle) [00:29:14] ori: something like that ^ ? [00:29:32] Krinkle: hey, that looks really nice [00:30:22] (03PS1) 10CSteipp: Prevent GC during phpunit shutdown [core] - 10https://gerrit.wikimedia.org/r/123146 [00:32:12] hi [00:32:23] (03PS1) 10Ori.livneh: eventlogging-reporter: send the stat, don't print it. [extensions/EventLogging] - 10https://gerrit.wikimedia.org/r/123147 [00:32:28] (03CR) 10Parent5446: "You've got to be kidding me. What the hell is going on with this bug? XD" [core] - 10https://gerrit.wikimedia.org/r/123146 (owner: 10CSteipp) [00:32:30] jenkins doesn't validate json [00:32:32] ... [00:33:00] There's a bug for that [00:33:04] (03CR) 10Ori.livneh: [C: 032] eventlogging-reporter: send the stat, don't print it. [extensions/EventLogging] - 10https://gerrit.wikimedia.org/r/123147 (owner: 10Ori.livneh) [00:33:25] * bawolff knows because I just recently submitted something with invalid json, and it was rather embarassing ;) [00:33:30] (03Merged) 10jenkins-bot: eventlogging-reporter: send the stat, don't print it. [extensions/EventLogging] - 10https://gerrit.wikimedia.org/r/123147 (owner: 10Ori.livneh) [00:34:06] I agree with parant's setiment on gerrit 123146 - wtf seriously? [00:35:55] bawolff: I just messed it up too [00:36:01] why isn't it validated? [00:36:07] seems like it would be easy to do [00:36:18] (03CR) 10TTO: [WIP] Add log for TB hits (031 comment) [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123128 (owner: 10Gerrit Patch Uploader) [00:36:37] I think its accidentally [00:36:40] *Accidental [00:38:41] (03CR) 10Hiong3-eng5: "Yes, that is a good idea. :)" [extensions/WikiLexicalData] - 10https://gerrit.wikimedia.org/r/122280 (owner: 10Hiong3-eng5) [00:41:37] (03Abandoned) 10CSteipp: Disable gc during tests [integration/jenkins] - 10https://gerrit.wikimedia.org/r/121697 (owner: 10CSteipp) [00:41:46] (03PS2) 10MZMcBride: [WIP] Add log for TitleBlacklist hits [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123128 (owner: 10Gerrit Patch Uploader) [00:43:25] (03CR) 10Hoo man: Prevent GC during phpunit shutdown (031 comment) [core] - 10https://gerrit.wikimedia.org/r/123146 (owner: 10CSteipp) [00:45:19] (03PS1) 10Gerrit Patch Uploader: [WIP] Add TitleBlacklist hit log [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123150 [00:45:25] (03CR) 10Gerrit Patch Uploader: "This commit was uploaded using the Gerrit Patch Uploader [1]." [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123150 (owner: 10Gerrit Patch Uploader) [00:45:28] dammit [00:45:38] bawolff: is it possible to put two changes together? [00:45:47] After I forgot to just make a new patchset [00:45:59] Abandon the newer changeset. [00:46:01] And re-commit. [00:46:10] What Gloria said [00:46:56] although if you're feeling adventorus, and you really want to "combine" both, you can do that with an interactive rebase [00:46:57] How to abandon? [00:47:13] one supersedes the other [00:47:18] There's an abandon button. [00:48:40] (03CR) 10PiRSquared17: [C: 04-1] "This was an accident. Merge with 123128?" [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123150 (owner: 10Gerrit Patch Uploader) [00:49:44] Gloria: I don't own it because I used the gerrit patch uploader (apparently git-review doesn't work without XCode, long story...)' [00:50:00] actually, are you allowed to abandon other people's patches if you're not +2? [00:50:17] (I spent at least 5 hours trying to install git/git-review on old Mac OS X) [00:50:24] (03Abandoned) 10Brian Wolff: [WIP] Add TitleBlacklist hit log [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123150 (owner: 10Gerrit Patch Uploader) [00:50:25] huh_: You can't install XCode? [00:50:36] on Leopard? [00:50:39] That seems like a poor use of time. [00:50:48] Which one is Leopard? [00:51:00] old [00:51:08] silly name, leopard is supposed to speedy, like a leopard [00:51:09] Do you need a new computer? [00:51:33] I'd rather make use of what I have [00:51:35] Linux: It just works(tm) [In one very specific obscure field] [00:52:09] [if you know C, C++, and bash very well, and aren't afraid to code everything that doesn't work] [00:52:33] huh_: Did you manage to get git installed, or is it both git and git review that are the problem? [00:52:44] both [00:53:13] Apparently I need a C compiler, which means I need XCode, which means I need to download it from somewhere, ... [00:53:22] Well that sucks... [00:54:16] There's nothing wrong with just using gerrit patch uploader, but if all else fails you could always ssh into tool labs, and work from there (or at least transfer the patch there, and then call git review from there) [00:54:37] really? [00:55:01] for now I've been working on an own instance on Labs and then getting the diff with git diff, then uploading [00:56:15] Well in that case you should be able to install git review on your labs account, and it should just work [00:56:29] maybe need to do some playing to make it use your ssh key [00:56:40] ssh-add and -A wouldn [00:56:43] 't be enough? [00:57:00] Actually, that would probably be fine [00:57:43] (03PS3) 10Gerrit Patch Uploader: [WIP] Add log for TitleBlacklist hits [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123128 [00:57:45] (03CR) 10Gerrit Patch Uploader: "This commit was uploaded using the Gerrit Patch Uploader [1]." [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123128 (owner: 10Gerrit Patch Uploader) [01:01:58] (03CR) 10PiRSquared17: "Problem:" (035 comments) [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123128 (owner: 10Gerrit Patch Uploader) [01:02:14] (03CR) 10PiRSquared17: "Will add qqq when ready" [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123128 (owner: 10Gerrit Patch Uploader) [01:03:01] (03CR) 10PiRSquared17: "Jenkins needs to check JSON validity." [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123128 (owner: 10Gerrit Patch Uploader) [01:04:04] (03CR) 10PiRSquared17: [WIP] Add log for TitleBlacklist hits (031 comment) [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123128 (owner: 10Gerrit Patch Uploader) [01:12:33] (03PS1) 10Ori.livneh: Update WikimediaEvents to master [core] (wmf/1.23wmf20) - 10https://gerrit.wikimedia.org/r/123154 [01:13:41] (03CR) 10jenkins-bot: [V: 04-1] Update WikimediaEvents to master [core] (wmf/1.23wmf20) - 10https://gerrit.wikimedia.org/r/123154 (owner: 10Ori.livneh) [01:13:45] (03PS1) 10Ori.livneh: Update WikimediaEvents for 1.23wmf19 cherry-picks [core] (wmf/1.23wmf19) - 10https://gerrit.wikimedia.org/r/123155 [01:14:49] (03CR) 10jenkins-bot: [V: 04-1] Update WikimediaEvents for 1.23wmf19 cherry-picks [core] (wmf/1.23wmf19) - 10https://gerrit.wikimedia.org/r/123155 (owner: 10Ori.livneh) [01:16:58] (03Abandoned) 10Ori.livneh: Update WikimediaEvents for 1.23wmf19 cherry-picks [core] (wmf/1.23wmf19) - 10https://gerrit.wikimedia.org/r/123155 (owner: 10Ori.livneh) [01:17:06] (03Abandoned) 10Ori.livneh: Update WikimediaEvents to master [core] (wmf/1.23wmf20) - 10https://gerrit.wikimedia.org/r/123154 (owner: 10Ori.livneh) [01:20:05] (03PS4) 10Mwalker: Better WorldPay form settings [core] (fundraising/REL1_22) - 10https://gerrit.wikimedia.org/r/121504 [01:20:28] Krinkle: tin:/home/krinkle/deprecated.py [01:35:42] (03PS1) 10Brian Wolff: Make sure batch preview page shows correct link colour [extensions/GWToolset] - 10https://gerrit.wikimedia.org/r/123157 [01:41:08] (03CR) 10Jackmcbarn: [C: 031] Prevent passing information between #invokes using math.random [extensions/Scribunto] - 10https://gerrit.wikimedia.org/r/122862 (owner: 10Anomie) [01:48:08] --^ lol, those crazy wikipedians [01:50:11] we try! [02:00:32] I'm always amazed at the crap they come up with to get around template restrictions [02:06:19] (03PS1) 10Brian Wolff: Mark GWToolset namespace as being json for code editor. [extensions/GWToolset] - 10https://gerrit.wikimedia.org/r/123159 [02:33:47] (03PS1) 10Jforrester: Update OOjs UI to v0.1.0-pre (8986c46d35) [core] - 10https://gerrit.wikimedia.org/r/123167 [02:34:17] (03CR) 10Swalling: [C: 031] "Patchset 4 looks good from a UX perspective, in Chrome and FF." [extensions/GettingStarted] - 10https://gerrit.wikimedia.org/r/119972 (owner: 10Phuedx) [02:47:56] (03CR) 10BryanDavis: [C: 032] Mark GWToolset namespace as being json for code editor. [extensions/GWToolset] - 10https://gerrit.wikimedia.org/r/123159 (owner: 10Brian Wolff) [02:48:01] (03Merged) 10jenkins-bot: Mark GWToolset namespace as being json for code editor. [extensions/GWToolset] - 10https://gerrit.wikimedia.org/r/123159 (owner: 10Brian Wolff) [02:48:13] bd808|BUFFER: Weee! that was fast. Thanks :) [03:00:06] (03PS10) 10Brian Wolff: Allow hiding patrol entries from Special:Log for all users [core] - 10https://gerrit.wikimedia.org/r/42279 [03:01:09] (03CR) 10jenkins-bot: [V: 04-1] Allow hiding patrol entries from Special:Log for all users [core] - 10https://gerrit.wikimedia.org/r/42279 (owner: 10Brian Wolff) [03:01:20] yeah yeah, I'm rebasing it [03:03:26] (03PS11) 10Brian Wolff: Allow hiding patrol entries from Special:Log for all users [core] - 10https://gerrit.wikimedia.org/r/42279 [03:03:46] (03CR) 10Brian Wolff: "PS11: rebase. again..." [core] - 10https://gerrit.wikimedia.org/r/42279 (owner: 10Brian Wolff) [03:15:21] (03PS6) 10Brian Wolff: Use rgba() for gallery caption background instead of opacity [core] - 10https://gerrit.wikimedia.org/r/116227 [03:16:11] (03CR) 10Brian Wolff: "Ok. I also changed background-color, to just background, as old IE wasn't falling back properly, and google told me that would fix that." [core] - 10https://gerrit.wikimedia.org/r/116227 (owner: 10Brian Wolff) [03:43:56] (03CR) 10Legoktm: [C: 04-1] "Haven't tested it yet" (033 comments) [extensions/TitleBlacklist] - 10https://gerrit.wikimedia.org/r/123128 (owner: 10Gerrit Patch Uploader) [03:51:01] legoktm: also it only works with redlink=1 ... [03:51:17] o.O [03:51:20] that seems weird. [03:51:40] I think the logging should be removed from the userCan hook, but I'm not sure where it should go. [03:52:03] I'm not even sure logging GET requests is a good idea, but it seems fine if there's a config flag for it [03:52:03] I don't think logging userCan is a safe thing to do [03:52:29] huh_: I think you should start with just logging account creation attempts [03:52:44] ...that part works [03:52:45] thats something we can safely log without the fear of logging any GET request [03:52:57] and that was what ajr wanted in his request [03:53:15] logging GET requests is probably a security issue and won't happen [03:53:39] even if only admins can see the log? [03:54:06] yeah [03:54:39] because I could do