From 37957aac0aea776e6f843af2358916f81056a405 Mon Sep 17 00:00:00 2001 From: Mamoru TASAKA Date: Thu, 29 Jun 2023 22:09:15 +0900 Subject: [PATCH] buildfix: support fmtlib 10 With fmtlib10, fmt::format does not accept pugi::xml_node as it is. Use name() member function, as this usage appear on other places of gerbera source code. Closes #2839 . Upstream: https://github.com/gerbera/gerbera/commit/37957aac0aea776e6f843af2358916f81056a405 Signed-off-by: Bernd Kuhls --- src/config/setup/config_setup_array.cc | 6 +++--- src/config/setup/config_setup_autoscan.cc | 2 +- src/config/setup/config_setup_client.cc | 2 +- src/config/setup/config_setup_dictionary.cc | 6 +++--- src/config/setup/config_setup_dynamic.cc | 2 +- src/config/setup/config_setup_transcoding.cc | 2 +- src/config/setup/config_setup_tweak.cc | 2 +- src/config/setup/config_setup_vector.cc | 4 ++-- 8 files changed, 13 insertions(+), 13 deletions(-) diff --git a/src/config/setup/config_setup_array.cc b/src/config/setup/config_setup_array.cc index 62dd5c9a7..a5f8958ce 100644 --- a/src/config/setup/config_setup_array.cc +++ b/src/config/setup/config_setup_array.cc @@ -137,11 +137,11 @@ std::vector ConfigArraySetup::getXmlContent(const pugi::xml_node& o std::vector result; if (initArray) { if (!initArray(optValue, result, ConfigDefinition::mapConfigOption(nodeOption))) { - throw_std_runtime_error("Invalid {} array value '{}'", xpath, optValue); + throw_std_runtime_error("Invalid {} array value '{}'", xpath, optValue.name()); } } else { if (!createOptionFromNode(optValue, result)) { - throw_std_runtime_error("Invalid {} array value '{}'", xpath, optValue); + throw_std_runtime_error("Invalid {} array value '{}'", xpath, optValue.name()); } } if (result.empty()) { @@ -150,7 +150,7 @@ std::vector ConfigArraySetup::getXmlContent(const pugi::xml_node& o result = defaultEntries; } if (notEmpty && result.empty()) { - throw_std_runtime_error("Invalid array {} empty '{}'", xpath, optValue); + throw_std_runtime_error("Invalid array {} empty '{}'", xpath, optValue.name()); } return result; } diff --git a/src/config/setup/config_setup_autoscan.cc b/src/config/setup/config_setup_autoscan.cc index 9ae4e61f8..5e1394585 100644 --- a/src/config/setup/config_setup_autoscan.cc +++ b/src/config/setup/config_setup_autoscan.cc @@ -245,7 +245,7 @@ std::shared_ptr ConfigAutoscanSetup::newOption(const pugi::xml_nod { auto result = std::vector(); if (!createOptionFromNode(optValue, result)) { - throw_std_runtime_error("Init {} autoscan failed '{}'", xpath, optValue); + throw_std_runtime_error("Init {} autoscan failed '{}'", xpath, optValue.name()); } optionValue = std::make_shared(result); return optionValue; diff --git a/src/config/setup/config_setup_client.cc b/src/config/setup/config_setup_client.cc index ffd12fc01..857db3c3e 100644 --- a/src/config/setup/config_setup_client.cc +++ b/src/config/setup/config_setup_client.cc @@ -183,7 +183,7 @@ std::shared_ptr ConfigClientSetup::newOption(const pugi::xml_node& auto result = std::make_shared(); if (!createOptionFromNode(isEnabled ? optValue : pugi::xml_node(nullptr), result)) { - throw_std_runtime_error("Init {} client config failed '{}'", xpath, optValue); + throw_std_runtime_error("Init {} client config failed '{}'", xpath, optValue.name()); } optionValue = std::make_shared(result); return optionValue; diff --git a/src/config/setup/config_setup_dictionary.cc b/src/config/setup/config_setup_dictionary.cc index 003b4cc59..69d1a7255 100644 --- a/src/config/setup/config_setup_dictionary.cc +++ b/src/config/setup/config_setup_dictionary.cc @@ -163,11 +163,11 @@ std::map ConfigDictionarySetup::getXmlContent(const pu std::map result; if (initDict) { if (!initDict(optValue, result)) { - throw_std_runtime_error("Init {} dictionary failed '{}'", xpath, optValue); + throw_std_runtime_error("Init {} dictionary failed '{}'", xpath, optValue.name()); } } else { if (!createOptionFromNode(optValue, result) && required) { - throw_std_runtime_error("Init {} dictionary failed '{}'", xpath, optValue); + throw_std_runtime_error("Init {} dictionary failed '{}'", xpath, optValue.name()); } } if (result.empty()) { @@ -176,7 +176,7 @@ std::map ConfigDictionarySetup::getXmlContent(const pu result = defaultEntries; } if (notEmpty && result.empty()) { - throw_std_runtime_error("Invalid dictionary {} empty '{}'", xpath, optValue); + throw_std_runtime_error("Invalid dictionary {} empty '{}'", xpath, optValue.name()); } return result; } diff --git a/src/config/setup/config_setup_dynamic.cc b/src/config/setup/config_setup_dynamic.cc index 0074cb6d3..d6962afe6 100644 --- a/src/config/setup/config_setup_dynamic.cc +++ b/src/config/setup/config_setup_dynamic.cc @@ -192,7 +192,7 @@ std::shared_ptr ConfigDynamicContentSetup::newOption(const pugi::x auto result = std::make_shared(); if (!createOptionFromNode(optValue, result)) { - throw_std_runtime_error("Init {} DynamicContentList failed '{}'", xpath, optValue); + throw_std_runtime_error("Init {} DynamicContentList failed '{}'", xpath, optValue.name()); } optionValue = std::make_shared(result); return optionValue; diff --git a/src/config/setup/config_setup_transcoding.cc b/src/config/setup/config_setup_transcoding.cc index 17d1780dc..1f6fb0604 100644 --- a/src/config/setup/config_setup_transcoding.cc +++ b/src/config/setup/config_setup_transcoding.cc @@ -501,7 +501,7 @@ std::shared_ptr ConfigTranscodingSetup::newOption(const pugi::xml_ auto result = std::make_shared(); if (!createOptionFromNode(isEnabled ? optValue : pugi::xml_node(nullptr), result)) { - throw_std_runtime_error("Init {} transcoding failed '{}'", xpath, optValue); + throw_std_runtime_error("Init {} transcoding failed '{}'", xpath, optValue.name()); } optionValue = std::make_shared(result); return optionValue; diff --git a/src/config/setup/config_setup_tweak.cc b/src/config/setup/config_setup_tweak.cc index 65dae2759..4da9fca1b 100644 --- a/src/config/setup/config_setup_tweak.cc +++ b/src/config/setup/config_setup_tweak.cc @@ -257,7 +257,7 @@ std::shared_ptr ConfigDirectorySetup::newOption(const pugi::xml_no auto result = std::make_shared(); if (!createOptionFromNode(optValue, result)) { - throw_std_runtime_error("Init {} DirectoryConfigList failed '{}'", xpath, optValue); + throw_std_runtime_error("Init {} DirectoryConfigList failed '{}'", xpath, optValue.name()); } optionValue = std::make_shared(result); return optionValue; diff --git a/src/config/setup/config_setup_vector.cc b/src/config/setup/config_setup_vector.cc index e5efe2767..9a9670569 100644 --- a/src/config/setup/config_setup_vector.cc +++ b/src/config/setup/config_setup_vector.cc @@ -181,7 +181,7 @@ std::vector>> ConfigVectorSetup: { std::vector>> result; if (!createOptionFromNode(optValue, result) && required) { - throw_std_runtime_error("Init {} vector failed '{}'", xpath, optValue); + throw_std_runtime_error("Init {} vector failed '{}'", xpath, optValue.name()); } if (result.empty()) { log_debug("{} assigning {} default values", xpath, defaultEntries.size()); @@ -189,7 +189,7 @@ std::vector>> ConfigVectorSetup: result = defaultEntries; } if (notEmpty && result.empty()) { - throw_std_runtime_error("Invalid vector {} empty '{}'", xpath, optValue); + throw_std_runtime_error("Invalid vector {} empty '{}'", xpath, optValue.name()); } return result; }