----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59105/ -----------------------------------------------------------
(Updated May 10, 2017, 4:08 a.m.) Review request for mesos and Benjamin Mahler. Changes ------- Split comment changes to a separate review. Bugs: MESOS-7464 https://issues.apache.org/jira/browse/MESOS-7464 Repository: mesos Description ------- The original behavior was to allow leading zeros. When Version was enhanced with more complete support for SemVer [1], it was also changed to reject leading zeros in numeric components (version numbers and prerelease identifiers). Although this change was consistent with the SemVer spec (which mandates that such version strings "MUST" be considered invalid), it breaks compatibility with the versions used by some common software packages (e.g., Docker). This commit reverts the change in behavior, so that leading zeros are once again allowed. In the future, we might consider adding a "strict" mode to the Version parser, and/or supporting an "options" scheme that would allow the user to customize the version parser's behavior. [1] 287556284d76e03c11cff3fc076224fe966096e0 Diffs (updated) ----- 3rdparty/stout/include/stout/version.hpp 4be722208329f838a3a3ebaa3b44affb2a6905f4 3rdparty/stout/tests/version_tests.cpp bce185ec493054ec81d0764088d04f3e4147303d Diff: https://reviews.apache.org/r/59105/diff/2/ Changes: https://reviews.apache.org/r/59105/diff/1-2/ Testing ------- `make check` Thanks, Neil Conway