----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67059/#review203732 -----------------------------------------------------------
Patch looks great! Reviews applied: [67247, 67099, 67059] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On May 22, 2018, 3:55 p.m., Armand Grillet wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67059/ > ----------------------------------------------------------- > > (Updated May 22, 2018, 3:55 p.m.) > > > Review request for mesos, Andrew Schwartzmeyer, Benjamin Bannier, Eric Chung, > and Kevin Klues. > > > Repository: mesos > > > Description > ------- > > The scripts are in a temporary directory, support/python3. > > The scripts have been ported using 2to3, the official tool to do so. > Many of these scripts require testing from the community before being > used by default. > > The script building the virtual environment and the git hooks have > been updated to use the new scripts if the environment variable > `MESOSSUPPORTPYTHON` is set to `3` by the user. > > > Diffs > ----- > > support/README.md e812e9638ca555c56e4c52e52485240e75cb8229 > support/build-virtualenv 850af89326721f34de20eb45a7e78fa391d031be > support/hooks/post-rewrite 1ab14abf711d1923a7ae69beb33581317009a94a > support/hooks/pre-commit 6faba98ab6db68aef1a54091a08b8db1eaac8701 > support/python3/apply-reviews.py PRE-CREATION > support/python3/generate-endpoint-help.py PRE-CREATION > support/python3/jsonurl.py PRE-CREATION > support/python3/mesos-gtest-runner.py PRE-CREATION > support/python3/mesos-split.py PRE-CREATION > support/python3/mesos-style.py PRE-CREATION > support/python3/post-reviews.py PRE-CREATION > support/python3/push-commits.py PRE-CREATION > support/python3/test-upgrade.py PRE-CREATION > support/python3/verify-reviews.py PRE-CREATION > > > Diff: https://reviews.apache.org/r/67059/diff/3/ > > > Testing > ------- > > All the files are OK for our linter, I have tested `mesos-style.py` and > `post-reviews.py` (used for this review request). > > We will likely see error messages `TypeError: cannot use a string pattern on > a bytes-like object`, they are very easy to fix (we just need to add > `.decode("utf-8")` but detecting all of them requires to use the scripts. > Please do so and create issues. > > > Thanks, > > Armand Grillet > >