From dc8d7e9e134fd5f489c494e1b4c8879b7c8b25f0 Mon Sep 17 00:00:00 2001 From: Daniel Patterson Date: Fri, 23 Mar 2018 14:29:48 -0700 Subject: [PATCH 1/4] Add new `datasource_names` annotation that returns the string version of the `datasources` annotation --- CHANGELOG.md | 2 ++ docs/http.md | 11 ++++--- features/support/shared_steps.js | 2 +- features/testbot/annotations.feature | 33 +++++++++++++++++++ include/engine/api/route_api.hpp | 7 ++++ include/engine/api/route_parameters.hpp | 3 +- .../server/api/route_parameters_grammar.hpp | 8 ++--- 7 files changed, 56 insertions(+), 10 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 233e48a319c..e0a39d680c2 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -23,6 +23,8 @@ - ADDED: Add documentation about OSM node ids in nearest service response [#4436](https://github.com/Project-OSRM/osrm-backend/pull/4436) - Performance - FIXED: Speed up response time when lots of legs exist and geojson is used with `steps=true` [#4936](https://github.com/Project-OSRM/osrm-backend/pull/4936) + - Misc: + - ADDED: support for new `datasource_names` annotation [#4973](https://github.com/Project-OSRM/osrm-backend/pull/4973) # 5.16.0 - Changes from 5.15.2: diff --git a/docs/http.md b/docs/http.md index 1f36fbcca36..e6866b074b5 100644 --- a/docs/http.md +++ b/docs/http.md @@ -192,7 +192,7 @@ In addition to the [general options](#general-options) the following options are |------------|---------------------------------------------|-------------------------------------------------------------------------------| |alternatives|`true`, `false` (default), or Number |Search for alternative routes. Passing a number `alternatives=n` searches for up to `n` alternative routes.\* | |steps |`true`, `false` (default) |Returned route steps for each route leg | -|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed` |Returns additional metadata for each coordinate along the route geometry. | +|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed`, `datasource_names` |Returns additional metadata for each coordinate along the route geometry. | |geometries |`polyline` (default), `polyline6`, `geojson` |Returned route geometry format (influences overview and per step) | |overview |`simplified` (default), `full`, `false` |Add overview geometry either full, simplified according to highest zoom level it could be display on, or not at all.| |continue\_straight |`default` (default), `true`, `false` |Forces the route to keep going straight at waypoints constraining uturns there even if it would be faster. Default value depends on the profile. | @@ -295,7 +295,7 @@ In addition to the [general options](#general-options) the following options are |------------|------------------------------------------------|------------------------------------------------------------------------------------------| |steps |`true`, `false` (default) |Returned route steps for each route | |geometries |`polyline` (default), `polyline6`, `geojson` |Returned route geometry format (influences overview and per step) | -|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed` |Returns additional metadata for each coordinate along the route geometry. | +|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed`, `datasource_names` |Returns additional metadata for each coordinate along the route geometry. | |overview |`simplified` (default), `full`, `false` |Add overview geometry either full, simplified according to highest zoom level it could be display on, or not at all.| |timestamps |`{timestamp};{timestamp}[;{timestamp} ...]` |Timestamps for the input locations in seconds since UNIX epoch. Timestamps need to be monotonically increasing. | |radiuses |`{radius};{radius}[;{radius} ...]` |Standard deviation of GPS precision used for map matching. If applicable use GPS accuracy.| @@ -351,7 +351,7 @@ In addition to the [general options](#general-options) the following options are |source |`any` (default), `first` |Returned route starts at `any` or `first` coordinate | |destination |`any` (default), `last` |Returned route ends at `any` or `last` coordinate | |steps |`true`, `false` (default) |Returned route instructions for each trip | -|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed` |Returns additional metadata for each coordinate along the route geometry. | +|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed`, `datasource_names` |Returns additional metadata for each coordinate along the route geometry. | |geometries |`polyline` (default), `polyline6`, `geojson` |Returned route geometry format (influences overview and per step) | |overview |`simplified` (default), `full`, `false` |Add overview geometry either full, simplified according to highest zoom level it could be display on, or not at all.| @@ -547,6 +547,7 @@ With `steps=false` and `annotations=true`: "distance": [5,5,10,5,5], "duration": [15,15,40,15,15], "datasources": [1,0,0,0,1], + "datasource_names": ["traffic","lua profile","lua profile","lua profile","traffic"], "nodes": [49772551,49772552,49786799,49786800,49786801,49786802], "speed": [0.3, 0.3, 0.3, 0.3, 0.3] } @@ -561,7 +562,8 @@ Annotation of the whole route leg with fine-grained information about each segme - `distance`: The distance, in metres, between each pair of coordinates - `duration`: The duration between each pair of coordinates, in seconds. Does not include the duration of any turns. -- `datasources`: The index of the datasource for the speed between each pair of coordinates. `0` is the default profile, other values are supplied via `--segment-speed-file` to `osrm-contract` +- `datasources`: The index of the datasource for the speed between each pair of coordinates. `0` is the default profile, other values are supplied via `--segment-speed-file` to `osrm-contract` or `osrm-customize` +- `datasource_names`: The names of the datasources used for the speed between each pair of coordinates. `lua profile` is the default profile, other values arethe filenames supplied via `--segment-speed-file` to `osrm-contract` or `osrm-customize` - `nodes`: The OSM node ID for each coordinate along the route, excluding the first/last user-supplied coordinates - `weight`: The weights between each pair of coordinates. Does not include any turn costs. - `speed`: Convenience field, calculation of `distance / duration` rounded to one decimal place @@ -573,6 +575,7 @@ Annotation of the whole route leg with fine-grained information about each segme "distance": [5,5,10,5,5], "duration": [15,15,40,15,15], "datasources": [1,0,0,0,1], + "datasource_names": ["traffic","lua profile","lua profile","lua profile","traffic"], "nodes": [49772551,49772552,49786799,49786800,49786801,49786802], "weight": [15,15,40,15,15] } diff --git a/features/support/shared_steps.js b/features/support/shared_steps.js index fa98fe12033..fb137e80878 100644 --- a/features/support/shared_steps.js +++ b/features/support/shared_steps.js @@ -158,7 +158,7 @@ module.exports = function () { // if header matches 'a:*', parse out the values for * // and return in that header headers.forEach((k) => { - let whitelist = ['duration', 'distance', 'datasources', 'nodes', 'weight', 'speed']; + let whitelist = ['duration', 'distance', 'datasources', 'nodes', 'weight', 'speed', 'datasource_names']; if (k.match(/^a:/)) { let a_type = k.slice(2); if (whitelist.indexOf(a_type) == -1) diff --git a/features/testbot/annotations.feature b/features/testbot/annotations.feature index 27d6a7b7581..ef3fda17a45 100644 --- a/features/testbot/annotations.feature +++ b/features/testbot/annotations.feature @@ -60,6 +60,39 @@ Feature: Annotations | a | i | abcdefghi,abcdefghi | 1:0:1:0:1:0:0:0 | 50:10:50:10:50:10:10:10 | | i | a | abcdefghi,abcdefghi | 0:1:0:0:0:0:0:1 | 10:50:10:10:10:10:10:50 | + Scenario: datasource name annotations + Given the profile "testbot" + + And the node map + """ + a b c + """ + + And the ways + | nodes | + | abc | + + And the contract extra arguments "--segment-speed-file {speeds_file}" + And the customize extra arguments "--segment-speed-file {speeds_file}" + + And the speed file + """ + 1,2,180,1 + 2,1,180,1 + """ + + And the query options + | annotations | datasource_names | + + # Note - the source names here are specific to how the tests are constructed, + # so if this test is moved around (changes line number) or support code + # changes how the filenames are generated, this test will need to be updated + When I route I should get + | from | to | route | a:datasource_names | + | a | c | abc,abc | 63_datasource_name_annotations_speeds:lua profile | + | c | a | abc,abc | lua profile:63_datasource_name_annotations_speeds | + + Scenario: Speed annotations should handle zero segments Given the profile "testbot" diff --git a/include/engine/api/route_api.hpp b/include/engine/api/route_api.hpp index 1e6c9764743..a3a33b56f0b 100644 --- a/include/engine/api/route_api.hpp +++ b/include/engine/api/route_api.hpp @@ -324,6 +324,13 @@ class RouteAPI : public BaseAPI } annotation.values["nodes"] = std::move(nodes); } + if (requested_annotations & RouteParameters::AnnotationsType::DatasourceNames) + { + annotation.values["datasource_names"] = GetAnnotations( + leg_geometry, [&](const guidance::LegGeometry::Annotation &anno) { + return std::string(facade.GetDatasourceName(anno.datasource)); + }); + } annotations.push_back(std::move(annotation)); } diff --git a/include/engine/api/route_parameters.hpp b/include/engine/api/route_parameters.hpp index c78ec70ea6f..f4fe53c2e90 100644 --- a/include/engine/api/route_parameters.hpp +++ b/include/engine/api/route_parameters.hpp @@ -76,7 +76,8 @@ struct RouteParameters : public BaseParameters Weight = 0x08, Datasources = 0x10, Speed = 0x20, - All = Duration | Nodes | Distance | Weight | Datasources | Speed + DatasourceNames = 0x40, + All = Duration | Nodes | Distance | Weight | Datasources | Speed | DatasourceNames }; RouteParameters() = default; diff --git a/include/server/api/route_parameters_grammar.hpp b/include/server/api/route_parameters_grammar.hpp index 2572532c987..4f9667c5516 100644 --- a/include/server/api/route_parameters_grammar.hpp +++ b/include/server/api/route_parameters_grammar.hpp @@ -65,10 +65,10 @@ struct RouteParametersGrammar : public BaseParametersGrammar Date: Mon, 2 Apr 2018 17:53:22 -0700 Subject: [PATCH 2/4] Refactor annotations into separate metadata structure --- CHANGELOG.md | 2 +- CMakeLists.txt | 3 +++ docs/http.md | 15 +++++++------- features/support/route.js | 16 +++++++++++++-- features/support/shared_steps.js | 10 +++++++++- features/testbot/annotations.feature | 6 +++--- include/engine/api/route_api.hpp | 20 ++++++++++++++----- include/engine/api/route_parameters.hpp | 3 +-- .../server/api/route_parameters_grammar.hpp | 8 ++++---- 9 files changed, 58 insertions(+), 25 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index e0a39d680c2..93ac75a09b7 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -24,7 +24,7 @@ - Performance - FIXED: Speed up response time when lots of legs exist and geojson is used with `steps=true` [#4936](https://github.com/Project-OSRM/osrm-backend/pull/4936) - Misc: - - ADDED: support for new `datasource_names` annotation [#4973](https://github.com/Project-OSRM/osrm-backend/pull/4973) + - ADDED: expose name for datasource annotations as metadata [#4973](https://github.com/Project-OSRM/osrm-backend/pull/4973) # 5.16.0 - Changes from 5.15.2: diff --git a/CMakeLists.txt b/CMakeLists.txt index 3e6e40828d3..820f1d09b16 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -42,6 +42,7 @@ if(ENABLE_MASON) set(MASON_LUA_VERSION "5.2.4") set(MASON_BZIP2_VERSION "1.0.6") set(MASON_TBB_VERSION "2017_U7") + set(MASON_CLANG_FORMAT_VERSION "3.8.1") message(STATUS "Enabling mason") @@ -447,6 +448,8 @@ if(ENABLE_MASON) mason_use(boost_libsystem VERSION ${MASON_BOOST_VERSION}) set(Boost_SYSTEM_LIBRARY ${MASON_PACKAGE_boost_libsystem_STATIC_LIBS}) + mason_use(clang-format VERSION ${MASON_CLANG_FORMAT_VERSION}) + if (ENABLE_STXXL) mason_use(stxxl VERSION ${MASON_STXXL_VERSION}) add_dependency_includes(${MASON_PACKAGE_stxxl_INCLUDE_DIRS}) diff --git a/docs/http.md b/docs/http.md index e6866b074b5..b14b0a341db 100644 --- a/docs/http.md +++ b/docs/http.md @@ -192,7 +192,7 @@ In addition to the [general options](#general-options) the following options are |------------|---------------------------------------------|-------------------------------------------------------------------------------| |alternatives|`true`, `false` (default), or Number |Search for alternative routes. Passing a number `alternatives=n` searches for up to `n` alternative routes.\* | |steps |`true`, `false` (default) |Returned route steps for each route leg | -|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed`, `datasource_names` |Returns additional metadata for each coordinate along the route geometry. | +|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed` |Returns additional metadata for each coordinate along the route geometry. | |geometries |`polyline` (default), `polyline6`, `geojson` |Returned route geometry format (influences overview and per step) | |overview |`simplified` (default), `full`, `false` |Add overview geometry either full, simplified according to highest zoom level it could be display on, or not at all.| |continue\_straight |`default` (default), `true`, `false` |Forces the route to keep going straight at waypoints constraining uturns there even if it would be faster. Default value depends on the profile. | @@ -295,7 +295,7 @@ In addition to the [general options](#general-options) the following options are |------------|------------------------------------------------|------------------------------------------------------------------------------------------| |steps |`true`, `false` (default) |Returned route steps for each route | |geometries |`polyline` (default), `polyline6`, `geojson` |Returned route geometry format (influences overview and per step) | -|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed`, `datasource_names` |Returns additional metadata for each coordinate along the route geometry. | +|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed` |Returns additional metadata for each coordinate along the route geometry. | |overview |`simplified` (default), `full`, `false` |Add overview geometry either full, simplified according to highest zoom level it could be display on, or not at all.| |timestamps |`{timestamp};{timestamp}[;{timestamp} ...]` |Timestamps for the input locations in seconds since UNIX epoch. Timestamps need to be monotonically increasing. | |radiuses |`{radius};{radius}[;{radius} ...]` |Standard deviation of GPS precision used for map matching. If applicable use GPS accuracy.| @@ -351,7 +351,7 @@ In addition to the [general options](#general-options) the following options are |source |`any` (default), `first` |Returned route starts at `any` or `first` coordinate | |destination |`any` (default), `last` |Returned route ends at `any` or `last` coordinate | |steps |`true`, `false` (default) |Returned route instructions for each trip | -|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed`, `datasource_names` |Returns additional metadata for each coordinate along the route geometry. | +|annotations |`true`, `false` (default), `nodes`, `distance`, `duration`, `datasources`, `weight`, `speed` |Returns additional metadata for each coordinate along the route geometry. | |geometries |`polyline` (default), `polyline6`, `geojson` |Returned route geometry format (influences overview and per step) | |overview |`simplified` (default), `full`, `false` |Add overview geometry either full, simplified according to highest zoom level it could be display on, or not at all.| @@ -547,7 +547,7 @@ With `steps=false` and `annotations=true`: "distance": [5,5,10,5,5], "duration": [15,15,40,15,15], "datasources": [1,0,0,0,1], - "datasource_names": ["traffic","lua profile","lua profile","lua profile","traffic"], + "metadata": { "datasource_names": ["traffic","lua profile","lua profile","lua profile","traffic"] }, "nodes": [49772551,49772552,49786799,49786800,49786801,49786802], "speed": [0.3, 0.3, 0.3, 0.3, 0.3] } @@ -562,11 +562,12 @@ Annotation of the whole route leg with fine-grained information about each segme - `distance`: The distance, in metres, between each pair of coordinates - `duration`: The duration between each pair of coordinates, in seconds. Does not include the duration of any turns. -- `datasources`: The index of the datasource for the speed between each pair of coordinates. `0` is the default profile, other values are supplied via `--segment-speed-file` to `osrm-contract` or `osrm-customize` -- `datasource_names`: The names of the datasources used for the speed between each pair of coordinates. `lua profile` is the default profile, other values arethe filenames supplied via `--segment-speed-file` to `osrm-contract` or `osrm-customize` +- `datasources`: The index of the datasource for the speed between each pair of coordinates. `0` is the default profile, other values are supplied via `--segment-speed-file` to `osrm-contract` or `osrm-customize`. String-like names are in the `metadata.datasource_names` array. - `nodes`: The OSM node ID for each coordinate along the route, excluding the first/last user-supplied coordinates - `weight`: The weights between each pair of coordinates. Does not include any turn costs. - `speed`: Convenience field, calculation of `distance / duration` rounded to one decimal place +- `metadata`: Metadata related to other annotations + - `datasource_names`: The names of the datasources used for the speed between each pair of coordinates. `lua profile` is the default profile, other values arethe filenames supplied via `--segment-speed-file` to `osrm-contract` or `osrm-customize` #### Example @@ -575,7 +576,7 @@ Annotation of the whole route leg with fine-grained information about each segme "distance": [5,5,10,5,5], "duration": [15,15,40,15,15], "datasources": [1,0,0,0,1], - "datasource_names": ["traffic","lua profile","lua profile","lua profile","traffic"], + "metadata": { "datasource_names": ["traffic","lua profile","lua profile","lua profile","traffic"] }, "nodes": [49772551,49772552,49786799,49786800,49786801,49786802], "weight": [15,15,40,15,15] } diff --git a/features/support/route.js b/features/support/route.js index 739bc46af68..cd713372df9 100644 --- a/features/support/route.js +++ b/features/support/route.js @@ -199,14 +199,26 @@ module.exports = function () { var merged = {}; instructions.legs.map(l => { - Object.keys(l.annotation).forEach(a => { + Object.keys(l.annotation).filter(a => !a.match(/metadata/)).forEach(a => { if (!merged[a]) merged[a] = []; merged[a].push(l.annotation[a].join(':')); }); + if (l.annotation.metadata) { + merged.metadata = {}; + Object.keys(l.annotation.metadata).forEach(a => { + if (!merged.metadata[a]) merged.metadata[a] = []; + merged.metadata[a].push(l.annotation.metadata[a].join(':')); + }); + } }); - Object.keys(merged).map(a => { + Object.keys(merged).filter(k => !k.match(/metadata/)).map(a => { merged[a] = merged[a].join(','); }); + if (merged.metadata) { + Object.keys(merged.metadata).map(a => { + merged.metadata[a] = merged.metadata[a].join(','); + }); + } return merged; }; diff --git a/features/support/shared_steps.js b/features/support/shared_steps.js index fb137e80878..b8743d2c18b 100644 --- a/features/support/shared_steps.js +++ b/features/support/shared_steps.js @@ -158,7 +158,8 @@ module.exports = function () { // if header matches 'a:*', parse out the values for * // and return in that header headers.forEach((k) => { - let whitelist = ['duration', 'distance', 'datasources', 'nodes', 'weight', 'speed', 'datasource_names']; + let whitelist = ['duration', 'distance', 'datasources', 'nodes', 'weight', 'speed' ]; + let metadata_whitelist = [ 'datasource_names' ]; if (k.match(/^a:/)) { let a_type = k.slice(2); if (whitelist.indexOf(a_type) == -1) @@ -166,6 +167,13 @@ module.exports = function () { if (annotation && !annotation[a_type]) return cb(new Error('Annotation not found in response', a_type)); got[k] = annotation && annotation[a_type] || ''; + } else if (k.match(/^am:/)) { + let a_type = k.slice(3); + if (metadata_whitelist.indexOf(a_type) == -1) + return cb(new Error('Unrecognized annotation field', a_type)); + if (annotation && (!annotation.metadata || !annotation.metadata[a_type])) + return cb(new Error('Annotation not found in response', a_type)); + got[k] = (annotation && annotation.metadata && annotation.metadata[a_type]) || ''; } }); diff --git a/features/testbot/annotations.feature b/features/testbot/annotations.feature index ef3fda17a45..706eb95041d 100644 --- a/features/testbot/annotations.feature +++ b/features/testbot/annotations.feature @@ -82,14 +82,14 @@ Feature: Annotations """ And the query options - | annotations | datasource_names | + | annotations | datasources | # Note - the source names here are specific to how the tests are constructed, # so if this test is moved around (changes line number) or support code # changes how the filenames are generated, this test will need to be updated When I route I should get - | from | to | route | a:datasource_names | - | a | c | abc,abc | 63_datasource_name_annotations_speeds:lua profile | + | from | to | route | am:datasource_names | + | a | c | abc,abc | lua profile:63_datasource_name_annotations_speeds | | c | a | abc,abc | lua profile:63_datasource_name_annotations_speeds | diff --git a/include/engine/api/route_api.hpp b/include/engine/api/route_api.hpp index a3a33b56f0b..a2427790132 100644 --- a/include/engine/api/route_api.hpp +++ b/include/engine/api/route_api.hpp @@ -324,12 +324,22 @@ class RouteAPI : public BaseAPI } annotation.values["nodes"] = std::move(nodes); } - if (requested_annotations & RouteParameters::AnnotationsType::DatasourceNames) + // Add any supporting metadata, if needed + if (requested_annotations & RouteParameters::AnnotationsType::Datasources) { - annotation.values["datasource_names"] = GetAnnotations( - leg_geometry, [&](const guidance::LegGeometry::Annotation &anno) { - return std::string(facade.GetDatasourceName(anno.datasource)); - }); + const auto MAX_DATASOURCE_ID = 255u; + util::json::Object metadata; + util::json::Array datasource_names; + for (auto i = 0u; i < MAX_DATASOURCE_ID; i++) + { + const auto name = facade.GetDatasourceName(i); + // Length of 0 indicates the first empty name, so we can stop here + if (name.size() == 0) + break; + datasource_names.values.push_back(std::string(facade.GetDatasourceName(i))); + } + metadata.values["datasource_names"] = datasource_names; + annotation.values["metadata"] = metadata; } annotations.push_back(std::move(annotation)); diff --git a/include/engine/api/route_parameters.hpp b/include/engine/api/route_parameters.hpp index f4fe53c2e90..c78ec70ea6f 100644 --- a/include/engine/api/route_parameters.hpp +++ b/include/engine/api/route_parameters.hpp @@ -76,8 +76,7 @@ struct RouteParameters : public BaseParameters Weight = 0x08, Datasources = 0x10, Speed = 0x20, - DatasourceNames = 0x40, - All = Duration | Nodes | Distance | Weight | Datasources | Speed | DatasourceNames + All = Duration | Nodes | Distance | Weight | Datasources | Speed }; RouteParameters() = default; diff --git a/include/server/api/route_parameters_grammar.hpp b/include/server/api/route_parameters_grammar.hpp index 4f9667c5516..2572532c987 100644 --- a/include/server/api/route_parameters_grammar.hpp +++ b/include/server/api/route_parameters_grammar.hpp @@ -65,10 +65,10 @@ struct RouteParametersGrammar : public BaseParametersGrammar Date: Tue, 3 Apr 2018 08:56:38 -0700 Subject: [PATCH 3/4] Fix unit tests - new metadata object will exist when all annotations are retrieved. --- unit_tests/library/route.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/unit_tests/library/route.cpp b/unit_tests/library/route.cpp index 4e60e75fc7c..2ce75b0e015 100644 --- a/unit_tests/library/route.cpp +++ b/unit_tests/library/route.cpp @@ -453,7 +453,7 @@ BOOST_AUTO_TEST_CASE(test_manual_setting_of_annotations_property) .values["annotation"] .get() .values; - BOOST_CHECK_EQUAL(annotations.size(), 5); + BOOST_CHECK_EQUAL(annotations.size(), 6); } BOOST_AUTO_TEST_SUITE_END() From 2328288d4ebf02970a7ea6187b94b40c07e23b12 Mon Sep 17 00:00:00 2001 From: Daniel Patterson Date: Tue, 3 Apr 2018 14:28:59 -0700 Subject: [PATCH 4/4] Remove mason lines - this was debugging cruft. --- CMakeLists.txt | 3 --- 1 file changed, 3 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 820f1d09b16..3e6e40828d3 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -42,7 +42,6 @@ if(ENABLE_MASON) set(MASON_LUA_VERSION "5.2.4") set(MASON_BZIP2_VERSION "1.0.6") set(MASON_TBB_VERSION "2017_U7") - set(MASON_CLANG_FORMAT_VERSION "3.8.1") message(STATUS "Enabling mason") @@ -448,8 +447,6 @@ if(ENABLE_MASON) mason_use(boost_libsystem VERSION ${MASON_BOOST_VERSION}) set(Boost_SYSTEM_LIBRARY ${MASON_PACKAGE_boost_libsystem_STATIC_LIBS}) - mason_use(clang-format VERSION ${MASON_CLANG_FORMAT_VERSION}) - if (ENABLE_STXXL) mason_use(stxxl VERSION ${MASON_STXXL_VERSION}) add_dependency_includes(${MASON_PACKAGE_stxxl_INCLUDE_DIRS})