Last modified: 2013-06-18 14:41:24 UTC
Hi there, please find the details in these posts. http://lists.wikimedia.org/pipermail/mediawiki-l/2009-November/032423.html http://lists.wikimedia.org/pipermail/mediawiki-l/2009-November/032452.html http://lists.wikimedia.org/pipermail/mediawiki-l/2009-November/032461.html http://lists.wikimedia.org/pipermail/mediawiki-l/2009-November/032473.html http://lists.wikimedia.org/pipermail/mediawiki-l/2009-November/032489.html http://lists.wikimedia.org/pipermail/mediawiki-l/2009-November/032495.html Thanks.
Created attachment 6812 [details] Proposed patch, in accordance with discussion on mediawiki-l
Please attach a patch in unified diff format.
Created attachment 6813 [details] Proposed patch, this time in unified format
Tweaked summary to be more specific. Also: not really liking that patch. All you're adding is "in Setup.php" to all of the comments. While it does clarify the situation to some degree, it's still not massively helpful--most people don't know our setup order and saying "in Setup.php" is meaningless to them.
I respectfully revert the summary, since some parts are actually wrong. Please following the links in the description for all the details. > All you're adding is "in Setup.php" to all of the comments. This is incorrect, please look more carefully.
Hello - any reason why this has not been integrated yet? Thanks.
*Bulk BZ Change: +Patch to open bugs with patches attached that are missing the keyword*
Ichiro, I'm sorry to say that your patch no longer applies against Subversion trunk (I am thus marking it reviewed). Would you be interested in coming into #mediawiki on FreeNode IRC and discussing the problem and coming up with an approach for a new patch? Thanks!
Created attachment 9398 [details] Re-based patch against trunk I went ahead and re-based the patch against trunk, it took about a minute :)
Went ahead and committed (albeit by accident) in r102532.