----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67059/#review202886 -----------------------------------------------------------
Patch looks great! Reviews applied: [67055, 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 10, 2018, 9:24 a.m., Armand Grillet wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67059/ > ----------------------------------------------------------- > > (Updated May 10, 2018, 9:24 a.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. > > > Diffs > ----- > > support/build-virtualenv 850af89326721f34de20eb45a7e78fa391d031be > support/hooks/commit-msg a0c218deee3fb4b7594fe39b76c1025045ba0725 > 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/1/ > > > 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 > >