D24084: Migrate kwin animation speed to kdeglobals
Fabian Vogt <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
fvogt added inline comments. INLINE COMMENTS > zzag wrote in kwin-5.18-move-animspeed.py:1 > > Does this really have to be python2? > > `python` doesn't necessarily means python2, e.g. Python 3 is default on Arch Linux. > > > IMO using /usr/bin/python3 would be the best option > > Does it really matter though? > python doesn't necessarily means python2, e.g. Python 3 is default on Arch Linux. Yes, but in other cases it does mean python2. The PEP about that will probably never be implemented everywhere (and is IMO also wrong, a script has to state whether it's 2 or 3 compatible). > Does it really matter though? Yes. Currently without patching kwin5 would not be installable as /bin/env doesn't exist leading to dependency issues. Ignoring that, it would be the only package in a default installation pulling in python 2 again, which is a regression. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D24084 To: davidedmundson, zzag Cc: lbeltrame, fvogt, zzag, kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart