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 Wed, 07 Nov 2018 21:22:26 GMT

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

(Updated Nov. 7, 2018, 9:22 p.m.)


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


Changes
-------

Rebased to master


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/9/

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


Testing
-------

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


Thanks,

Jason Lai


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