[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Making mediawiki FHS compliant



Steve Langasek <vorlon@debian.org> wrote:

> On Thu, Feb 22, 2007 at 11:17:41AM +0100, Frank Küster wrote:
>> > It's at least an order of magnitude more complicated (=error-prone) than the
>> > solution Romain has implemented, 
>
>> That's correct, not solving the FHS violation is simpler than doing it.
>
>> > and it still doesn't address the FHS
>> > requirement for the actual configuration to be stored under /etc, so I don't
>> > see the point.
>
>> Then we misunderstood each other.  I'm always talking about moving the
>> actual configuration (LocalSettings.php and AdminSettings.php) to
>> /etc/mediawiki1.7.  
>
> Ok, it was my understanding from previous comments in the bug history that
> this couldn't be done non-intrusively, because mediawiki would then look up
> the real directory and use that value for things that it shouldn't?

In fact it's easy, it's a one-line patch

$ diff -u src/packages_for_mydebwork/mediawiki1.7-1.7.1/config/index.php{.orig,}
--- src/packages_for_mydebwork/mediawiki1.7-1.7.1/config/index.php.orig	2007-02-16 21:39:02.000000000 +0100
+++ src/packages_for_mydebwork/mediawiki1.7-1.7.1/config/index.php	2007-02-24 12:44:00.000000000 +0100
@@ -1303,6 +1303,7 @@
 
 # If you customize your file layout, set \$IP to the directory that contains
 # the other MediaWiki files. It will be used as a base to locate files.
+define(MW_INSTALL_PATH,"/var/lib/mediawiki1.7");
 if( defined( 'MW_INSTALL_PATH' ) ) {
 	\$IP = MW_INSTALL_PATH;
 } else {

I haven't tested it that way (instead I hardcoded the path in the else
statement), but this is what the maintainer suggested in #388616:

,----
| Obviously, adding a 
| define(MW_INSTALL_PATH,"/var/lib/mediawiki1.7"° on top of the file is the 
| good solution. It is also what is done in my next 1.9 package.
`----

Regards, Frank

-- 
Dr. Frank Küster
Single Molecule Spectroscopy, Protein Folding @ Inst. f. Biochemie, Univ. Zürich
Debian Developer (teTeX/TeXLive)



Reply to: