----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66010/#review199355 -----------------------------------------------------------
Fix it, then Ship it! cmake/CompilationConfigure.cmake Lines 326-330 (original), 326-330 (patched) <https://reviews.apache.org/r/66010/#comment279664> Looks like this variable can go away too. - Joseph Wu On March 13, 2018, 1:45 p.m., Andrew Schwartzmeyer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66010/ > ----------------------------------------------------------- > > (Updated March 13, 2018, 1:45 p.m.) > > > Review request for mesos, Akash Gupta, Benjamin Bannier, Jeff Coffler, John > Kordich, Joseph Wu, and Michael Park. > > > Bugs: MESOS-8659 > https://issues.apache.org/jira/browse/MESOS-8659 > > > Repository: mesos > > > Description > ------- > > We previously attempted to manually override the CRT to be static > everywhere. Not only did this emit warnings, it was also error-prone > and unnecessary. We can, and should, just use the defaults, which is > `/MDd` in debug mode (multi-threaded, dynamic, debug linkage). Linking > to the CRT dynamically results in smaller libraries and executables, > reduces linking time, and avoids bugs when sharing allocated memory > across modules. > > > Diffs > ----- > > cmake/CompilationConfigure.cmake efee36c1ffda096a97af23d481fc0d0903124e54 > > > Diff: https://reviews.apache.org/r/66010/diff/2/ > > > Testing > ------- > > NOTE: I checked with Alex Ionescu and since we are deploying on the same OS > versions as we're targetting, we don't need to worry about redistributing the > CRT. > > If/when we start deploying on older OS versions, we can simply add > instructions on installing the correct CRT redistributable (or make it part > of the eventual installer). > > > Thanks, > > Andrew Schwartzmeyer > >
