• Daniel Dromboski's avatar
    [tools] More Python 3 compatibility fixes · 8807f0ad
    Daniel Dromboski authored
    These should all be forward/backward compatible with Python 2/Python 3.
    
    [tools] Tweak statusfile.py for Python 3
    
    .iteritems() does not exist in Python 3, only .items().
    
    (While .iteritems() was meant to be an optimization over .items()
    in Python 2, .items() should work fine, and it is forward/backward
    compatible.)
    
    
    [tools] Fix another Python 3 issue in mb.py
    
    sys.platform used to return e.g. 'linux2', which is 'linux' plus
    whatever the first digit of `uname -r` was when Python was built.
    As of Python 3.3, it always returns just 'linux' for Linux OSes.
    Use `sys.platform.startswith('linux')` for forward/backward
    compatibility.
    
    
    [tools] Make base_runner.py Python 3 compatible
    
    dict.keys() returns a dict_keys in Python 3, whereas it
    used to return a simple array. list() is forward/backward
    compatible with identical results on Python 2/3 (returns array).
    
    (Tested on Linux x64, trying to recreate NodeJS's CI workflow.)
    
    
    [tools] Make tools/dev/v8gen.py work with Python 3
    
    dict.keys() returns a dict_keys in Python 3, whereas it
    used to return a simple array. list() is forward/backward
    compatible with identical results on Python 2/3 (returns array).
    
    Comparing a None-type value numerically used to result in the
    None-type value always being considered "less than" the thing
    it is compared to. As of Python 3, numerically comparing against
    None or None-typed values results in an error. Check if a value
    is truthy before numerically comparing it, for forward/backward
    compatibility.
    
    print() used to transparently decode byte strings in Python 2.
    In Python 3, they must be explicitly decoded first.
    
    (Tested on Linux 64-bit, trying to recreate NodeJS's CI workflow.)
    
    Bug: v8:9871
    Change-Id: I059bf98577a67649bbe7ec49848989d468da96b0
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2867270Reviewed-by: 's avatarMichael Achenbach <machenbach@chromium.org>
    Commit-Queue: Michael Achenbach <machenbach@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#74369}
    8807f0ad
v8gen.py 10.2 KB