123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166 |
- From 37957aac0aea776e6f843af2358916f81056a405 Mon Sep 17 00:00:00 2001
- From: Mamoru TASAKA <mtasaka@fedoraproject.org>
- 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 <bernd@kuhls.net>
- ---
- 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<std::string> ConfigArraySetup::getXmlContent(const pugi::xml_node& o
- std::vector<std::string> 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<std::string> 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<ConfigOption> ConfigAutoscanSetup::newOption(const pugi::xml_nod
- {
- auto result = std::vector<AutoscanDirectory>();
- 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<AutoscanListOption>(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<ConfigOption> ConfigClientSetup::newOption(const pugi::xml_node&
- auto result = std::make_shared<ClientConfigList>();
-
- 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<ClientConfigListOption>(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<std::string, std::string> ConfigDictionarySetup::getXmlContent(const pu
- std::map<std::string, std::string> 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<std::string, std::string> 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<ConfigOption> ConfigDynamicContentSetup::newOption(const pugi::x
- auto result = std::make_shared<DynamicContentList>();
-
- 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<DynamicContentListOption>(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<ConfigOption> ConfigTranscodingSetup::newOption(const pugi::xml_
- auto result = std::make_shared<TranscodingProfileList>();
-
- 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<TranscodingProfileListOption>(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<ConfigOption> ConfigDirectorySetup::newOption(const pugi::xml_no
- auto result = std::make_shared<DirectoryConfigList>();
-
- 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<DirectoryTweakOption>(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<std::vector<std::pair<std::string, std::string>>> ConfigVectorSetup:
- {
- std::vector<std::vector<std::pair<std::string, std::string>>> 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<std::vector<std::pair<std::string, std::string>>> 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;
- }
|