[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-buildsystem
Subject: Re: [PATCH] bug 174806
From: Thiago Macieira <thiago () kde ! org>
Date: 2009-04-11 17:59:27
Message-ID: 200904111959.36828.thiago () kde ! org
[Download RAW message or body]
[Attachment #2 (multipart/signed)]
Raphael Kubo da Costa wrote:
>> Well, if we detect Python 2.6, then we should install to the KDE
>> prefix by default.
>
>According to PEP 370[1], we should install it to the local
>site-packages dir, usually ~/.local/lib/python2.6/site-packages.
>
>[1] http://www.python.org/dev/peps/pep-0370/
So if you install as root, it ends up in /root/.local/lib/... ?
That sounds wrong.
>
>> The problem then is, of course, the surprise factor: if someone
>> upgrades, suddenly KDE starts installing to the wrong path.
>>
>> I recommend, instead, that we default to the KDE prefix and, if Python
>> 2.5 or earlier is detected, we print a big warning at the end of the
>> cmake run, saying that you will not be able to run and that you should
>> set the PYTHON_SITE_PACKAGES_DIR variable instead. Even make it a
>> fatal error.
>
>Except for the KDE prefix part, this seems reasonable.
>
>However, I haven't looked at Michael's patch for a while, so I don't
>remember if this is actually wrong too.
Sorry, no. When installing KDE, by default it should not install anything
outside its prefix. If I want it to install somewhere else, I'll give it
specific paths to do so (like old configure --prefix=/usr --sysconfdir=/etc).
I stand by this: when installing, unless told otherwise, install ONLY to
your prefix.
--
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
PGP/GPG: 0x6EF45358; fingerprint:
E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358
["signature.asc" (application/pgp-signature)]
_______________________________________________
Kde-buildsystem mailing list
Kde-buildsystem@kde.org
https://mail.kde.org/mailman/listinfo/kde-buildsystem
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic