From 99bb7ac8bd5844d8ebb4795bb72a02024a28e688 Mon Sep 17 00:00:00 2001 From: Sadie Powell Date: Fri, 19 Mar 2021 00:51:09 +0000 Subject: Add an option to allow certain config files to not exist. This is useful when using modules that generate a config file such as the filter and permchannels modules. --- docs/conf/modules.conf.example | 2 +- src/configparser.cpp | 17 ++++++++++++++++- 2 files changed, 17 insertions(+), 2 deletions(-) diff --git a/docs/conf/modules.conf.example b/docs/conf/modules.conf.example index 4be14c960..6eb61cc38 100644 --- a/docs/conf/modules.conf.example +++ b/docs/conf/modules.conf.example @@ -1766,7 +1766,7 @@ # -# +# # # You may also create channels on startup by using the block. # diff --git a/src/configparser.cpp b/src/configparser.cpp index 556442627..9f5a49f1d 100644 --- a/src/configparser.cpp +++ b/src/configparser.cpp @@ -39,7 +39,10 @@ enum ParseFlags FLAG_NO_INC = 4, // &env.FOO; is disabled. - FLAG_NO_ENV = 8 + FLAG_NO_ENV = 8, + + // It's okay if an include doesn't exist. + FLAG_MISSING_OKAY = 16 }; // Represents the position within a config file. @@ -420,11 +423,18 @@ void ParseStack::DoInclude(ConfigTag* tag, int flags) { if (tag->getBool("noinclude", false)) flags |= FLAG_NO_INC; + if (tag->getBool("noexec", false)) flags |= FLAG_NO_EXEC; + if (tag->getBool("noenv", false)) flags |= FLAG_NO_ENV; + if (tag->getBool("missingokay", false)) + flags |= FLAG_MISSING_OKAY; + else + flags &= ~FLAG_MISSING_OKAY; + if (!ParseFile(ServerInstance->Config->Paths.PrependConfig(name), flags, mandatorytag)) throw CoreException("Included"); } @@ -504,7 +514,12 @@ bool ParseStack::ParseFile(const std::string& path, int flags, const std::string FileWrapper file((isexec ? popen(path.c_str(), "r") : fopen(path.c_str(), "r")), isexec); if (!file) + { + if (flags & FLAG_MISSING_OKAY) + return true; + throw CoreException("Could not read \"" + path + "\" for include"); + } reading.push_back(path); Parser p(*this, flags, file, path, mandatory_tag); -- cgit v1.2.3