mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jason Lai <ja...@jasonlai.net>
Subject Re: Review Request 65811: Added Stout `path::normalize` function for POSIX paths.
Date Tue, 25 Sep 2018 00:08:30 GMT

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

(Updated Sept. 25, 2018, 12:08 a.m.)


Review request for mesos, Anish Gupta, Eric Chung, Gilbert Song, Jie Yu, James Peach, and
Zhitao Li.


Changes
-------

Updated comment for failure on attempting to escape root and minor change.


Bugs: MESOS-8257
    https://issues.apache.org/jira/browse/MESOS-8257


Repository: mesos


Description
-------

Added `path::normalize` to normalize a given pathname and remove
redundant separators and up-level references.

This function follows the rules described in `path_resolution(7)`
for Linux. However, it only performs pure lexical processing without
touching the actual filesystem.


Diffs (updated)
-----

  3rdparty/stout/include/stout/path.hpp ef5a2f3d14cc8cf7416ded5cad00d30a49fd5cf7 


Diff: https://reviews.apache.org/r/65811/diff/8/

Changes: https://reviews.apache.org/r/65811/diff/7-8/


Testing (updated)
-------

`make tests` with https://reviews.apache.org/r/68832/


Thanks,

Jason Lai


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