Last modified: 2014-11-19 18:18:30 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T75567, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 73567 - GeSHi version constant defined twice
GeSHi version constant defined twice
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
SyntaxHighlight (GeSHi) (Other open bugs)
master
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: code-update-regression
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-11-18 18:17 UTC by Joel Sahleen
Modified: 2014-11-19 18:18 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Joel Sahleen 2014-11-18 18:17:54 UTC
The following commit has created a situation where the GeSHi version constant is defined twice, resulting in a notice.

https://gerrit.wikimedia.org/r/#/c/172581/

This commit needs to be reverted or fixed.
Comment 1 Joel Sahleen 2014-11-19 13:54:17 UTC
I believe this bug is causing a failure on all Wikipedias in beta-labs. Anything that uses EventLogging is bitten by this bug because EventLogging depends on the SyntaxHighlighting_GeSHi extension. In my dev environment, having EventLogging and GeSHi enabled causes pages to load with a notice that the version constant is defined twice. On beta-labs you just get a white page. I'm not 100% sure the beta outage is caused by this bug but it seems highly likely.
Comment 2 Andre Klapper 2014-11-19 14:52:23 UTC
(In reply to Joel Sahleen from comment #1)
> I believe this bug is causing a failure on all Wikipedias in beta-labs.

Might be one potential reason, might not.  From #ops:
<_joe_> marktraceur: it needs a restart, and maybe some tuning
<marktraceur> /var is at 100% on deployment-mediawiki01 at least
<marktraceur> So logging is killing it
Comment 3 Greg Grossmeier 2014-11-19 16:23:32 UTC
Beta is back up right now, but I don't believe this bug is fixed in master, correct? (Beta's problem might have been space running out on one of the machines, but since I woke up to both that and this, I want to make sure we find the real issue.)
Comment 4 Sam Reed (reedy) 2014-11-19 16:25:20 UTC
Dont think so. Going to revert in master
Comment 5 Sam Reed (reedy) 2014-11-19 16:31:49 UTC
Reverted in https://gerrit.wikimedia.org/r/#/c/174432/
Comment 6 Gerrit Notification Bot 2014-11-19 16:38:03 UTC
Change 174433 had a related patch set uploaded by Paladox:
Fix GESHI version

https://gerrit.wikimedia.org/r/174433
Comment 7 paladox2015 2014-11-19 16:57:19 UTC
Hi I have fixed it in https://gerrit.wikimedia.org/r/#/c/174433/
Comment 8 Krinkle 2014-11-19 18:00:18 UTC
This bug is about the version declared twice. That was fixed by reverting the commit.

> Revert "Fix tracking for version"
>
> This reverts commit d69ae1f3ac97404831da292145ad843574a14579.
> Change-Id: Ic724f6fe1b1cbc969ac195762bdf517e129cd9dd
Comment 9 paladox2015 2014-11-19 18:14:02 UTC
But the api doesent detect it.
Comment 10 paladox2015 2014-11-19 18:18:30 UTC
What I mean by api doesent detect it is it does not detect the version that is in geshi.php instead of it saying 1.0.8.12 in the api it says GESHI_VERSION.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links