mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benno Evers <bev...@mesosphere.com>
Subject Re: Review Request 71879: Renamed stout's path-related absolute functions to is_absolute.
Date Mon, 09 Dec 2019 11:50:15 GMT

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


Fix it, then Ship it!





3rdparty/stout/include/stout/path.hpp
Line 154 (original), 154 (patched)
<https://reviews.apache.org/r/71879/#comment306962>

    Ideally we'd still keep a deprecated `absolute()` function around, to not break backwards
compatibility.
    
    But I'm not sure what our current policy on stout backwards compatibility is, if we're
regularly doing similar refactorings in stout, feel free to drop this issue.


- Benno Evers


On Dec. 5, 2019, 12:48 p.m., Benjamin Bannier wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/71879/
> -----------------------------------------------------------
> 
> (Updated Dec. 5, 2019, 12:48 p.m.)
> 
> 
> Review request for mesos and Benno Evers.
> 
> 
> Bugs: MESOS-10062
>     https://issues.apache.org/jira/browse/MESOS-10062
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Renamed stout's path-related absolute functions to is_absolute.
> 
> 
> Diffs
> -----
> 
>   3rdparty/stout/include/stout/internal/windows/longpath.hpp 499eef30a3d16ac1f6c2e3334ef773e91e987a45

>   3rdparty/stout/include/stout/os/posix/copyfile.hpp 631babc17fff978b4e2048e5041a9f8db9b282b2

>   3rdparty/stout/include/stout/os/windows/copyfile.hpp 34723bce9a151582de481d63a865509d4d29c02c

>   3rdparty/stout/include/stout/path.hpp ba1f665ce94b9636d88a7ecce8643c56758f7b5c 
>   3rdparty/stout/tests/path_tests.cpp 19dd910a534040468aeb48f15ebdf56dff32bc15 
> 
> 
> Diff: https://reviews.apache.org/r/71879/diff/1/
> 
> 
> Testing
> -------
> 
> `make check`
> 
> 
> Thanks,
> 
> Benjamin Bannier
> 
>


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