[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