Review Request 68951: Updated verify-reviews.py to use current interpreter in subprocesses.

2018-10-08 Thread Armand Grillet
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68951/ --- Review request for mesos, Till Toenshoff and Vinod Kone. Bugs: MESOS-9253 h

Re: Review Request 68951: Updated verify-reviews.py to use current interpreter in subprocesses.

2018-10-08 Thread Till Toenshoff via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68951/#review209317 --- support/verify-reviews.py Line 97 (original), 97 (patched)

Re: Review Request 68951: Updated verify-reviews.py to use current interpreter in subprocesses.

2018-10-08 Thread Armand Grillet
> On Oct. 8, 2018, 8:12 p.m., Till Toenshoff wrote: > > support/verify-reviews.py > > Line 97 (original), 97 (patched) > > > > > > IIUC, then `sys.executable` may be `None`. Shall we guard against that? As described

Re: Review Request 68951: Updated verify-reviews.py to use current interpreter in subprocesses.

2018-10-08 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68951/#review209323 --- Bad patch! Reviews applied: [68951] Failed command: python suppo

Re: Review Request 68951: Updated verify-reviews.py to use current interpreter in subprocesses.

2018-10-08 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68951/#review209324 --- Ship it! Ship It! - Vinod Kone On Oct. 8, 2018, 6:06 p.m., A

Re: Review Request 68951: Updated verify-reviews.py to use current interpreter in subprocesses.

2018-10-08 Thread Vinod Kone
> On Oct. 8, 2018, 6:12 p.m., Till Toenshoff wrote: > > support/verify-reviews.py > > Line 97 (original), 97 (patched) > > > > > > IIUC, then `sys.executable` may be `None`. Shall we guard against that? > > Armand Gr

Re: Review Request 68951: Updated verify-reviews.py to use current interpreter in subprocesses.

2018-10-08 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68951/#review209334 --- PASS: Mesos patch 68951 was successfully built and tested. Review