On 25/08/17 17:32, Pietro wrote:
On Fri, Aug 25, 2017 at 5:14 PM, Moritz Lennert
<mlenn...@club.worldonline.be> wrote:
On 25/08/17 17:06, Pietro wrote:
Unfortunately builtins is used also in other files, and I’m afraid
that I should also fix these:
Actually most of them are working properly, I was thinking that the
module builtins was present in both python2 and python3 and only
contains different functions.
Instead builtins exists only in python3, so most of them works because
in python2 try to import some function from builtins and it raise an
exception not because the function does not exists but because the
module does not exists!
Yes, but the exception then acts as if it is in Python 3, so this might
cause problems (just as we saw for utils.py).
Do you think that should we fix them in any case?
In light of the above, yes.
Is this in grass72 as well, or only trunk ?
Changes in scripts.utils is only in trunk:
https://trac.osgeo.org/grass/browser/grass/tags/release_20170810_grass_7_2_2RC1/lib/python/script/utils.py
the other are already in the code.
https://trac.osgeo.org/grass/browser/grass/tags/release_20170810_grass_7_2_2RC1/lib/python/temporal/core.py#L38
So this probably needs to be taken care of before the imminent release.
Moritz
_______________________________________________
grass-dev mailing list
grass-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/grass-dev
