----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67055/ -----------------------------------------------------------
(Updated mai 11, 2018, 9:49 après-midi) Review request for mesos, Andrew Schwartzmeyer, Benjamin Bannier, Eric Chung, and Kevin Klues. Changes ------- Changed Python version. Bugs: MESOS-8770 https://issues.apache.org/jira/browse/MESOS-8770 Repository: mesos Description ------- The Python 3 supports scripts currently live in support/python3. Once all support scripts will have been updated to Python 3 and tested by the developers, we will replace the Python 2 scripts by the new ones in an ongoing effort of modernizing our Python codebase. Diffs (updated) ----- support/python3/cpplint.patch PRE-CREATION support/python3/cpplint.py PRE-CREATION Diff: https://reviews.apache.org/r/67055/diff/2/ Changes: https://reviews.apache.org/r/67055/diff/1-2/ Testing ------- Used `mesos-style.py` with the old `cpplint.py` (no change on `mesos-style.py`): ``` apache-mesos (cpplint) $ ./support/mesos-style.py Checking 1289 C++ files Total errors found: 0 Checking 4 JavaScript files Total errors found: 0 Checking 47 Python files py27-lint: commands succeeded congratulations :) py27-lint: commands succeeded congratulations :) Total errors found: 0 ``` Updated `mesos-style.py` to use the new version of cpplint: ``` # We do not use a version of cpplint available through pip as # we use a custom version (see cpplint.path) to lint C++ files. process = subprocess.Popen( ['python3', 'support/python3/cpplint.py', rules_filter] + source_paths, stderr=subprocess.PIPE, close_fds=True) ``` Run `mesos-style.py` again: ``` apache-mesos (cpplint) $ ./support/mesos-style.py Checking 1289 C++ files Total errors found: 0 Checking 4 JavaScript files Total errors found: 0 Checking 47 Python files py27-lint: commands succeeded congratulations :) py27-lint: commands succeeded congratulations :) ************* Module mesos-style C:279, 0: Line too long (83/80) (line-too-long) Total errors found: 1 ``` As we can see, the number of errors found by cpplint is the same as before. I have also used cpplint separately bu running `python3 support/python3/cpplint` on a few C++ files to see that it was running as expected. Thanks, Armand Grillet