> On Jan. 8, 2016, 4:43 p.m., Mesos ReviewBot wrote: > > Bad patch! > > > > Reviews applied: [41096, 41185, 41108] > > > > Failed command: ./support/apply-review.sh -n -r 41108 > > > > Error: > > 2016-01-08 16:43:43 URL:https://reviews.apache.org/r/41108/diff/raw/ > > [4052/4052] -> "41108.patch" [1] > > error: patch failed: src/slave/cmake/FindCurl.cmake:31 > > error: src/slave/cmake/FindCurl.cmake: patch does not apply
It looks like you need to update your master and rebase this commit to come after the last commit on the master branch? Ping me if you need help with this. - Alex ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41108/#review113487 ----------------------------------------------------------- On Jan. 8, 2016, 3:29 p.m., Diana Arroyo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41108/ > ----------------------------------------------------------- > > (Updated Jan. 8, 2016, 3:29 p.m.) > > > Review request for mesos, Artem Harutyunyan, Alex Clemmer, Joris Van > Remoortere, and Joseph Wu. > > > Bugs: MESOS-3843 > https://issues.apache.org/jira/browse/MESOS-3843 > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/slave/cmake/FindCurl.cmake PRE-CREATION > src/slave/cmake/SlaveConfigure.cmake > fbdfdaa27fbd8c7429861eea5baf401a221f748b > > Diff: https://reviews.apache.org/r/41108/diff/ > > > Testing > ------- > > Tested on Ubuntu and OSX. > Tested if and else path of new logic added to FindCurl.cmake. > > > Thanks, > > Diana Arroyo > >
