> On Feb. 14, 2014, 12:59 a.m., Ben Mahler wrote: > > Looks good, just wondering if we can use the new process::socket function > > to prevent SIGPIPE on OS X. > > Charlie Carson wrote: > can we open a seperate JIRA for that ? - I think it's good practice to > keep file / directory changes separate from logic / code changes.
Thanks for the follow up, will you be adding the missing includes? - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17341/#review34447 ----------------------------------------------------------- On Feb. 13, 2014, 10:58 p.m., Charlie Carson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17341/ > ----------------------------------------------------------- > > (Updated Feb. 13, 2014, 10:58 p.m.) > > > Review request for mesos, Benjamin Hindman and Jie Yu. > > > Bugs: https://issues.apache.org/jira/browse/MESOS-936 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/MESOS-936 > > > Repository: mesos-git > > > Description > ------- > > move process::http namespace into http.cpp > > This moves the code declared in the process::http namespace to > be in it's own file (http.cpp). It already had it's own header. > > SEE: https://issues.apache.org/jira/browse/MESOS-936 > > Review: https://reviews.apache.org/r/17341 > > > Diffs > ----- > > 3rdparty/libprocess/Makefile.am bcb78a45788fe93c520a787c2b614d49c354e633 > 3rdparty/libprocess/src/http.cpp PRE-CREATION > 3rdparty/libprocess/src/process.cpp > 2e7764a8e0badec704b8610f3f72f0bd16cc9612 > > Diff: https://reviews.apache.org/r/17341/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Charlie Carson > >
