Review at https://gerrit.osmocom.org/6614
Logger: Drop support to log into file This feature is currently not being used, so let's simplify current code to move to libosmocore logging system in the future. Change-Id: If2c77c776823f595130edac963be953026049423 --- M CommonLibs/Logger.cpp 1 file changed, 1 insertion(+), 19 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/14/6614/1 diff --git a/CommonLibs/Logger.cpp b/CommonLibs/Logger.cpp index d10ad02..463b6e0 100644 --- a/CommonLibs/Logger.cpp +++ b/CommonLibs/Logger.cpp @@ -39,7 +39,6 @@ // Switches to enable/disable logging targets bool gLogToConsole = true; -FILE *gLogToFile = NULL; Mutex gLogToLock; // Global log level threshold: @@ -103,7 +102,7 @@ cerr << mStream.str() << endl; } // Log to file and console - if (gLogToConsole||gLogToFile) { + if (gLogToConsole) { int mlen = mStream.str().size(); int neednl = (mlen==0 || mStream.str()[mlen-1] != '\n'); ScopedLock lock(gLogToLock); @@ -112,11 +111,6 @@ // so just use std::cout. std::cout << mStream.str(); if (neednl) std::cout<<"\n"; - } - if (gLogToFile) { - fputs(mStream.str().c_str(),gLogToFile); - if (neednl) {fputc('\n',gLogToFile);} - fflush(gLogToFile); } } } @@ -135,18 +129,6 @@ // Set the level if one has been specified. if (level) config_log_level = levelStringToInt(level); - - // Both the transceiver and OpenBTS use this same facility, but only OpenBTS/OpenNodeB may use this log file: - if (!gLogToFile && fn) { - gLogToFile = fopen(fn,"w"); // New log file each time we start. - if (gLogToFile) { - time_t now; - time(&now); - fprintf(gLogToFile,"Starting at %s",ctime(&now)); - fflush(gLogToFile); - std::cout << "Logging to file: " << fn << "\n"; - } - } } // vim: ts=4 sw=4 -- To view, visit https://gerrit.osmocom.org/6614 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: If2c77c776823f595130edac963be953026049423 Gerrit-PatchSet: 1 Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Owner: Pau Espin Pedrol <pes...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder