diff options
| author | Carl Hetherington <cth@carlh.net> | 2025-03-03 20:51:48 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2025-03-08 00:12:07 +0100 |
| commit | 3f2675aab119e55f958563e2fe6949192a2b976d (patch) | |
| tree | 2ed8c7e2a0eb7147ab9b798e934d1f7a4ce453a9 /src/lib/util.cc | |
| parent | f180d199dcb0217ea9750ab0a788217689733c41 (diff) | |
Don't use an optional for the Grok config object.
I'm not sure why I did it like this in the first place - perhaps so that
if the API endpoint changes there aren't so many old values stuck in
config files all over the place? Anyway, it seems cleaner to do it like
this, as it's how we handle the other config.
Diffstat (limited to 'src/lib/util.cc')
| -rw-r--r-- | src/lib/util.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/lib/util.cc b/src/lib/util.cc index df15e1abb..1e2f7d61f 100644 --- a/src/lib/util.cc +++ b/src/lib/util.cc @@ -1146,7 +1146,7 @@ setup_grok_library_path() } } auto const grok = Config::instance()->grok(); - if (!grok || grok->binary_location.empty()) { + if (grok.binary_location.empty()) { setenv("LD_LIRARY_PATH", old_path.c_str(), 1); return; } @@ -1155,7 +1155,7 @@ setup_grok_library_path() if (!new_path.empty()) { new_path += ":"; } - new_path += grok->binary_location.string(); + new_path += grok.binary_location.string(); setenv("LD_LIBRARY_PATH", new_path.c_str(), 1); } |