diff --git a/Utils/Console/Executor.cc b/Utils/Console/Executor.cc
index e464906e330c2299be7e5776ef399d7122828be7..1a93b6e95647e249e0ac976c5d5bf26f389ebb4d 100644
--- a/Utils/Console/Executor.cc
+++ b/Utils/Console/Executor.cc
@@ -340,10 +340,10 @@ senf::console::Executor::traverseDirectory(ParseCommandInfo::TokensRange const &
}
}
catch (std::bad_cast &) {
- throw InvalidDirectoryException();
+ throw InvalidDirectoryException(errorPath);
}
catch (UnknownNodeNameException &) {
- throw InvalidDirectoryException();
+ throw InvalidDirectoryException(errorPath);
}
}
diff --git a/Utils/Exception.hh b/Utils/Exception.hh
index 42e1326fbca957754412da8b56985c5f48eb9953..cc23ae350d1157c5fde9d9e76ea17e0726207819 100644
--- a/Utils/Exception.hh
+++ b/Utils/Exception.hh
@@ -261,7 +261,7 @@ namespace senf {
the exception description:
\code
// Standard usage: Take \c errno from environment
- SENF_THROW_SYSTEM_EXCEPTION()
+ SENF_THROW_SYSTEM_EXCEPTION("::open()")
<< " while opening configuration file: " << filename;
// You may however explicitly specify the errno value
diff --git a/Utils/Logger/FileTarget.cc b/Utils/Logger/FileTarget.cc
index 568cc9d6b33fc768d18d60d5e1b0beddac2b7fe6..4cc69ad4f3ef88cd14b186abb79b77b6fd7326db 100644
--- a/Utils/Logger/FileTarget.cc
+++ b/Utils/Logger/FileTarget.cc
@@ -28,6 +28,8 @@
// Custom includes
#include "../Console/Console.hh"
+#include "../Exception.hh"
+#include <boost/filesystem/path.hpp>
//#include "FileTarget.mpp"
#define prefix_
@@ -35,21 +37,23 @@
namespace {
- std::string quoteFilename(std::string filename)
+ std::string getNodename(std::string const & filename, std::string const & nodename)
{
- for (std::string::iterator i (filename.begin()); i != filename.end(); ++i)
- if (! senf::console::CommandParser::isWordChar(*i))
- *i = '_';
- return filename;
+ if (! nodename.empty())
+ return nodename;
+ return boost::filesystem::path(filename).leaf();
}
}
-prefix_ senf::log::FileTarget::FileTarget(std::string const & file)
- : ofstream_t(file.c_str(), std::ofstream::app),
- IOStreamTarget(quoteFilename(file), ofstream_t::member),
- file_(file)
+prefix_ senf::log::FileTarget::FileTarget(std::string const & filename,
+ std::string const & nodename)
+ : ofstream_t (filename.c_str(), std::ofstream::app),
+ IOStreamTarget (getNodename(filename, nodename), ofstream_t::member),
+ file_ (filename)
{
+ if (! ofstream_t::member)
+ SENF_THROW_SYSTEM_EXCEPTION("logfile open") << ": " << filename;
consoleDir().add( "reopen", senf::membind(
static_cast<void (FileTarget::*)()>(&FileTarget::reopen),
this))
@@ -59,6 +63,8 @@ prefix_ senf::log::FileTarget::FileTarget(std::string const & file)
this))
.arg("filename","new filename")
.overloadDoc("Reopen logfile under new name");
+ consoleDir().add("file", boost::cref(file_))
+ .doc("Show filename log messages are sent to");
}
prefix_ void senf::log::FileTarget::reopen()
@@ -79,17 +85,21 @@ prefix_ void senf::log::FileTarget::reopen(std::string const & file)
prefix_ senf::log::FileTarget::RegisterConsole::RegisterConsole()
{
+ namespace kw = senf::console::kw;
+
detail::TargetRegistry::instance().consoleDir().add("file-target",&RegisterConsole::create)
.arg("filename", "name of logfile")
+ .arg("nodename", "name of node in console. Defaults to the files basename",
+ kw::default_value = "")
.doc("Create new file target. Examples:\n"
"\n"
"Create new file target '/var/log/example.log\n"
" $ file-target \"/var/log/example.log\"\n"
- " <Directory '/sys/log/_var_log_example.log'>\n"
+ " <Directory '/sys/log/example.log'>\n"
"\n"
"In a configuration file, create new file target '/var/log/example.log' and set\n"
"some parameters (If written on one line, this works at the console too:\n"
- " /sys/log/file-target \"/var/log/example.log\" {\n"
+ " /sys/log/file-target \"/var/log/example.log\" mainlog {\n"
" route (IMPORTANT); # route all important messages\n"
" timeFormat \"\"; # use non-formatted time format\n"
" showArea false; # don't show log area\n"
@@ -97,9 +107,10 @@ prefix_ senf::log::FileTarget::RegisterConsole::RegisterConsole()
}
prefix_ boost::shared_ptr<senf::console::DirectoryNode>
-senf::log::FileTarget::RegisterConsole::create(std::string const & filename)
+senf::log::FileTarget::RegisterConsole::create(std::string const & filename,
+ std::string const & nodename)
{
- std::auto_ptr<Target> tp (new FileTarget(filename));
+ std::auto_ptr<Target> tp (new FileTarget(filename, nodename));
Target & target (*tp.get());
detail::TargetRegistry::instance().dynamicTarget(tp);
return target.consoleDir().node().thisptr();
diff --git a/Utils/Logger/FileTarget.hh b/Utils/Logger/FileTarget.hh
index 11543d74177ab35a4141117a9e2cbb08e2bc1fed..5f3082e64075fb78ae0d85861889745d61bc5f60 100644
--- a/Utils/Logger/FileTarget.hh
+++ b/Utils/Logger/FileTarget.hh
@@ -69,7 +69,7 @@ namespace log {
///\name Structors and default members
///@{
- explicit FileTarget(std::string const & filename);
+ explicit FileTarget(std::string const & filename, std::string const & nodename = "");
///< Construct FileTarget writing to \a file
///@}
@@ -84,7 +84,7 @@ namespace log {
struct RegisterConsole {
RegisterConsole();
static boost::shared_ptr<senf::console::DirectoryNode> create(
- std::string const & filename);
+ std::string const & filename, std::string const & nodename);
static RegisterConsole instance;
};
};
diff --git a/Utils/Logger/Target.cc b/Utils/Logger/Target.cc
index 7896bf0787983a9ad2911a80776f00728bdc8009..9c883dc979c2758bc9acc5b00548dab7e94a82c6 100644
--- a/Utils/Logger/Target.cc
+++ b/Utils/Logger/Target.cc
@@ -313,7 +313,7 @@ namespace {
std::string formatLabel(std::string const & l)
{
if (l.empty())
- return "-";
+ return "*";
if (l.size() > 29)
return l.substr(l.size()-29);
return l;
@@ -323,7 +323,7 @@ namespace {
prefix_ void senf::log::Target::consoleList(std::ostream & os)
{
static char const * levels[] = {
- "none", "verbose", "notice", "message", "important", "critical", "fatal", "disabled" };
+ "verbose", "verbose", "notice", "message", "important", "critical", "fatal", "disabled" };
boost::format fmt ("%2d %-29s %-29s %-9s %-6s\n");
os << fmt % "#" % "STREAM" % "AREA" % "LEVEL" % "ACTION";