----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70567/#review214963 -----------------------------------------------------------
Fix it, then Ship it! LGTM src/slave/containerizer/mesos/isolators/linux/seccomp.cpp Lines 35 (patched) <https://reviews.apache.org/r/70567/#comment301226> s/the/The/ - Vinod Kone On April 29, 2019, 6:39 p.m., Andrei Budnik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70567/ > ----------------------------------------------------------- > > (Updated April 29, 2019, 6:39 p.m.) > > > Review request for mesos, Benno Evers, Gilbert Song, and Qian Zhang. > > > Repository: mesos > > > Description > ------- > > This patch removes dependency on `linux/seccomp.h` header, which > may be missing on some Linux distributions. > > > Diffs > ----- > > configure.ac e5afde4fcae320facd4a7cf2961999b238812b4b > src/slave/containerizer/mesos/isolators/linux/seccomp.cpp > 5624c24fc08f0eeb5cccff9cfb2fe22eef4bfb50 > > > Diff: https://reviews.apache.org/r/70567/diff/1/ > > > Testing > ------- > > internal CI > > > Thanks, > > Andrei Budnik > >
