Refactor log file location to use application strings

This commit is contained in:
Vasily
2019-03-11 11:32:44 +01:00
parent 5b9e94869c
commit 2d6d482f2d
+5 -10
View File
@@ -272,17 +272,12 @@ void setUpLogging()
conf.set( Level::Trace, ConfigurationType::Enabled, confDisabled ); conf.set( Level::Trace, ConfigurationType::Enabled, confDisabled );
conf.set( Level::Debug, ConfigurationType::Enabled, confDisabled ); conf.set( Level::Debug, ConfigurationType::Enabled, confDisabled );
// This places the log file in const auto appDataLocation
// Roaming/AppData/matzman666/OpenVRAdvancedSettings/AdvancedSettings.log. = std::string( "/" ) + application_strings::applicationOrganizationName
// The log file placement has been broken since at least git tag "v2.7". + "/" + application_strings::applicationName;
// It was being placed in the working dir of the executable. const auto logFilePath = QDir( QStandardPaths::writableLocation(
// The change hasn't been documented anywhere, so it is likely that it was
// unintentional. This fixes the probable regression until a new path is
// decided on.
constexpr auto appDataFolders = "/matzman666/OpenVRAdvancedSettings";
const QString logFilePath = QDir( QStandardPaths::writableLocation(
QStandardPaths::AppDataLocation ) QStandardPaths::AppDataLocation )
+ appDataFolders ) + appDataLocation.c_str() )
.absoluteFilePath( "AdvancedSettings.log" ); .absoluteFilePath( "AdvancedSettings.log" );
conf.set( el::Level::Global, conf.set( el::Level::Global,
el::ConfigurationType::Filename, el::ConfigurationType::Filename,