Re: Review Request 38817: Added a SIGPIPE handler for the libprocess tests.

2015-09-28 Thread Cong Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38817/#review100881 --- 3rdparty/libprocess/src/tests/main.cpp (line 41)

Re: Review Request 38817: Added a SIGPIPE handler for the libprocess tests.

2015-09-28 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38817/#review100873 --- Patch looks great! Reviews applied: [38817] All tests passed. -

Re: Review Request 38817: Added a SIGPIPE handler for the libprocess tests.

2015-09-28 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38817/#review100883 --- Ship it! Ship It! - Joris Van Remoortere On Sept. 28, 2015,

Review Request 38817: Added a SIGPIPE handler for the libprocess tests.

2015-09-28 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38817/ --- Review request for mesos and Joris Van Remoortere. Repository: mesos

Re: Review Request 38817: Added a SIGPIPE handler for the libprocess tests.

2015-09-28 Thread Ben Mahler
> On Sept. 28, 2015, 10:52 p.m., Cong Wang wrote: > > 3rdparty/libprocess/src/tests/main.cpp, line 41 > > > > > > Here you probably want a signal string (strsignal ()) instead of a raw > > value. >From what I can