Last modified: 2014-04-15 06:27:25 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 T63733, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 61733 - Remove "php-" from wiki version numbers
Remove "php-" from wiki version numbers
Status: UNCONFIRMED
Product: Wikimedia
Classification: Unclassified
Deployment systems (Other open bugs)
wmf-deployment
All All
: Lowest enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
https://noc.wikimedia.org/conf/wikive...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-02-21 04:10 UTC by MZMcBride
Modified: 2014-04-15 06:27 UTC (History)
4 users (show)

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


Attachments

Description MZMcBride 2014-02-21 04:10:39 UTC
https://noc.wikimedia.org/conf/wikiversions.json currently reads (in part):

---
{
    "aawiki": "php-1.23wmf14",
    "aawikibooks": "php-1.23wmf14",
    "aawiktionary": "php-1.23wmf14",
    "abwiki": "php-1.23wmf14",
---

The "php-" part seems unnecessary and can probably be (carefully) removed. Or perhaps replaced with "mw-".
Comment 1 Sam Reed (reedy) 2014-02-21 04:27:02 UTC
(In reply to MZMcBride from comment #0)
> The "php-" part seems unnecessary and can probably be (carefully) removed.
> Or perhaps replaced with "mw-".

Decisions, decisions
Comment 2 Antoine "hashar" Musso (WMF) 2014-02-21 07:18:22 UTC
The php- prefix is hardcoded everywhere. There is no reason for us to invest time in fixing every possible occurrences or handle back compatibility code around.
Comment 3 MZMcBride 2014-02-21 13:52:03 UTC
(In reply to Antoine "hashar" Musso from comment #2)
> The php- prefix is hardcoded everywhere.

This is hyperbole. Please don't be hasty.
Comment 4 Andre Klapper 2014-02-21 22:17:31 UTC
What is the advantage / gain of removing it?
Comment 5 Bryan Davis 2014-04-14 22:22:40 UTC
The values in the wikiversions.json and compiled wikiversions.cdb files are the directory names used for the branch checkouts relative to the /usr/local/apache/common directory on each apache server. 

It is hardcoded in at least these locations in operations/mediawiki-config.git:
* .gitignore
* logmsg-git-hook
* multiversion/activeMWVersions.php
* multiversion/checkoutMediaWiki.php
* multiversion/deleteMediaWiki
* multiversion/MWMultiVersion.php
* multiversion/updateBitsBranchPointers
* multiversion/updateWikiversions.php
* wmf-config/StartProfiler.php

There is also hard coded knowledge of this prefix in both the legacy scap scripts and the new python code.
Comment 6 Antoine "hashar" Musso (WMF) 2014-04-15 06:20:18 UTC
Bryan see my comment #2 above. I dont think we should spend time dropping that prefix.
Comment 7 MZMcBride 2014-04-15 06:27:25 UTC
(In reply to Andre Klapper from comment #4)
> What is the advantage / gain of removing it?

It looks silly. wikiversions.json includes "php-1.23wmf14". There's probably a good historical reason for this, but I don't think it needs to linger indefinitely. I think "mw-1.23wmf14" or "1.23wmf14" would be clearer, but perhaps as hashar suggests, this isn't worth the effort to change.

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


Navigation
Links