mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Coffler <j...@taltos.com>
Subject Re: Review Request 64720: Upgraded to use cURL v7.57.0 on Windows Platform.
Date Tue, 19 Dec 2017 22:27:12 GMT


> On Dec. 19, 2017, 9:12 p.m., Mesos Reviewbot Windows wrote:
> > FAIL: Cannot overwrite the item D:DCOSmesosbuild-outputbinariescurl.exe with itself.
> > 
> > Reviews applied: `['64720']`
> > 
> > All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/64720

I'm not 100% sure why reviewbot had this problem. I tested successfully on both Windows and
Linux, and ran both STOUT and MESOS tests on both Windows on Linux prior to submitting the
review. Note that my tests require changes that Andy made, so to get successful test passes
on Windows (and to get a successful manual test), Andy's changes need to be in the chain.
My changes require Andy's changes, which I indicated in "depends on".

The logs (http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/64720/logs/mesos-tests-stdout.log)
show that it built and passed all tests successfully. We will follow up with our CI team to
fix this failure, but we can ignore it here.


- Jeff


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64720/#review194179
-----------------------------------------------------------


On Dec. 19, 2017, 7:33 p.m., Jeff Coffler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64720/
> -----------------------------------------------------------
> 
> (Updated Dec. 19, 2017, 7:33 p.m.)
> 
> 
> Review request for mesos, Andrew Schwartzmeyer and John Kordich.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Along with this, a number of changes were made:
> 
> 1. Debug versions of cURL build with libcurl-d and curl-d.exe.
>    Modified build system to handle these issues.
> 2. cURL is built to make use of SChannel (native Windows encryption)
>    by default. This means we no longer need to build Mesos with
>    cmake flag -DENABLE_SSL=ON, cURL isn't sensitive to that.
> 
> 
> Diffs
> -----
> 
>   3rdparty/CMakeLists.txt d9f52e20f2aa3bdc1544a9db70857aba02598439 
>   3rdparty/cmake/Versions.cmake 13bd33ddc0dacc6dc8c4ac7e460e39fdb2c1327d 
> 
> 
> Diff: https://reviews.apache.org/r/64720/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jeff Coffler
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message