Remove log filename deprecation

Signed-off-by: Russell McGuire <russell.w.mcguire@intel.com>

Author: rwmcguir

README.md

@@ -53,16 +53,16 @@ To enable this debug tracing feature, set the environment variable `ZE_ENABLE_LO
 This will enforce the Loader to print all errors whether fatal or non-fatal to stderr with the PREFIX `ZE_LOADER_DEBUG_TRACE:`.
 
 
-# Logging to File - PREVIEW
+# Logging to File
 The Level Zero Loader provides built-in logging controlled via environment variables:
 
 | Environment Variable | Default | Description |
 |---|---|---|
 | `ZEL_ENABLE_LOADER_LOGGING` | `0` | Set to `1` to enable file logging |
-| `ZEL_LOADER_LOG_CONSOLE` | `0` | Set to `1` to enable console (stderr) logging |
+| `ZEL_LOADER_LOG_CONSOLE` | `0` | Set to `1` to enable console (stderr) logging, overrides file logging |
 | `ZEL_LOADER_LOGGING_LEVEL` | `warn` | Log level: `trace`, `debug`, `info`, `warn`, `error`, `critical`, `off` |
 | `ZEL_LOADER_LOG_DIR` | `~/.oneapi_logs` | Directory to write the log file into |
-| `ZEL_LOADER_LOG_FILE` | `ze_loader.log` | Log filename (**deprecated**, will be removed in a future release) |
+| `ZEL_LOADER_LOG_FILE` | `ze_loader.log` | Log filename |
 | `ZEL_LOADER_LOG_PATTERN` | see below | Custom log format pattern |
 
 ## Output destination
@@ -98,8 +98,6 @@ Supported pattern tokens:
 - `%$` — end color range
 - `%v` — log message
 
-This feature is in early development and is preview only.
-
 # Logging API calls
 The Level Zero Loader will log all API calls whenever logging level is set to `trace` and
 validation layer is enabled. Following variables need to be set to enable API logging:

source/utils/ze_logger.cpp

@@ -472,8 +472,6 @@ std::shared_ptr<ZeLogger> createLogger() {
     auto loader_file = getenv_string("ZEL_LOADER_LOG_FILE");
     if (loader_file.empty()) {
         loader_file = LOADER_LOG_FILE;
-    } else {
-        std::cout << "ZEL_LOADER_LOG_FILE will be deprecated in a future release" << std::endl;
     }
 
 #ifdef _WIN32
@@ -533,17 +531,13 @@ std::shared_ptr<ZeLogger> createLogger() {
     // confirm what was enabled and where output is going.
     std::string cfg;
     cfg  = "Loader logging enabled:";
-    cfg += "\n  Output    : " + output_dest;
-    cfg += "\n  Level     : " + log_level;
-    cfg += "\n  Pattern   : " + log_pattern;
-    if (!getenv_string("ZEL_LOADER_LOG_DIR").empty())
-        cfg += "\n  Log dir   : " + log_directory + " (ZEL_LOADER_LOG_DIR)";
-    if (!getenv_string("ZEL_LOADER_LOG_FILE").empty())
-        cfg += "\n  Log file  : " + loader_file + " (ZEL_LOADER_LOG_FILE, deprecated)";
-    if (!getenv_string("ZEL_LOADER_LOGGING_LEVEL").empty())
-        cfg += "\n  Level src : ZEL_LOADER_LOGGING_LEVEL";
-    if (!getenv_string("ZEL_LOADER_LOG_PATTERN").empty())
-        cfg += "\n  Pattern src: ZEL_LOADER_LOG_PATTERN";
+    cfg += "\n  ZEL_LOADER_LOG_CONSOLE          : " + std::string(log_console ? "stderr" : "disabled");
+    cfg += "\n  ZEL_ENABLE_LOADER_LOGGING        : " + std::string(logging_enabled ? "enabled" : "disabled");
+    cfg += "\n  ZEL_LOADER_LOGGING_LEVEL         : " + log_level;
+    cfg += "\n  ZEL_LOADER_LOG_DIR               : " + log_directory;
+    cfg += "\n  ZEL_LOADER_LOG_FILE              : " + loader_file;
+    cfg += "\n  ZEL_LOADER_LOG_PATTERN           : " + log_pattern;
+    cfg += "\n  Output                           : " + output_dest;
     logger->info(cfg);
 
     return logger;