D24084: Migrate kwin animation speed to kdeglobals

[Fabian Vogt]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190927/091a472c/attachment.html>