X-Git-Url: https://feistymeow.org/gitweb/?a=blobdiff_plain;f=nucleus%2Flibrary%2Fconfiguration%2Fapplication_configuration.cpp;h=9f01391f8c61ebd1fdab010d5a0f7655c409941f;hb=81ff468e27ccaf567a2c25d4356d9c8e10810609;hp=9302c0b8b10f8a5fd3fdcc250410aeab33f705fd;hpb=36a9a07fc0d58746df4b6e9f2d1accca9b77d64a;p=feisty_meow.git

diff --git a/nucleus/library/configuration/application_configuration.cpp b/nucleus/library/configuration/application_configuration.cpp
index 9302c0b8..9f01391f 100644
--- a/nucleus/library/configuration/application_configuration.cpp
+++ b/nucleus/library/configuration/application_configuration.cpp
@@ -19,6 +19,7 @@
 #include <basis/guards.h>
 #include <basis/mutex.h>
 #include <basis/utf_conversion.h>
+#include <filesystem/directory.h>
 #include <filesystem/filename.h>
 #include <mathematics/chaos.h>
 #include <structures/static_memory_gremlin.h>
@@ -101,9 +102,9 @@ astring application_configuration::get_cmdline_from_proc()
   filename testing(__check_once_app_path);
   if (testing.had_directory()) return __check_once_app_path;  // all set.
 
-//hmmm: the below might be better off as a find app in path method, which relies on which.
-printf("no dir part found, app name after chewing: %s\n", __check_once_app_path.s());
+//printf("no dir part found, app name after chewing: %s\n", __check_once_app_path.s());
 
+//hmmm: the below might be better off as a find app in path method, which relies on which.
   // there was no directory component, so we'll try to guess one.
   astring temp_filename(environment::TMP()
       + a_sprintf("/zz_cmdfind.%d", chaos().inclusive(0, 999999999)));
@@ -136,7 +137,7 @@ printf("no dir part found, app name after chewing: %s\n", __check_once_app_path.
 // deprecated; better to use the /proc/pid/cmdline file.
 astring application_configuration::query_for_process_info()
 {
-//  FUNCDEF("query_for_process_info");
+  FUNCDEF("query_for_process_info");
   astring to_return = "unknown";
   // we ask the operating system about our process identifier and store
   // the results in a temporary file.
@@ -197,7 +198,7 @@ astring application_configuration::query_for_process_info()
 
 astring application_configuration::application_name()
 {
-//  FUNCDEF("application_name");
+  FUNCDEF("application_name");
   astring to_return;
 #ifdef __APPLE__
   char buffer[MAX_ABS_PATH] = { '\0' };
@@ -293,13 +294,14 @@ const astring &application_configuration::LOGGING_FOLDER_NAME() { STATIC_STRING(
 
 //////////////
 
-const int MAX_LOG_PATH = 200;
+////const int MAX_LOG_PATH = 512;
   // the maximum length of the entry stored for the log path.
 
 astring application_configuration::get_logging_directory()
 {
-  // start with the root of our installation.
-  astring def_log = application_directory();
+  // new scheme is to just use the temporary directory, which can vary per user
+  // and which hopefully is always set to something usable.
+  astring def_log = environment::TMP();
   // add logs directory underneath that.
   def_log += "/logs";
     // add the subdirectory for logs.
@@ -309,6 +311,7 @@ astring application_configuration::get_logging_directory()
     // get the entry for the logging path.
   if (!log_dir) {
     // if the entry was absent, we set it.
+//printf("did not find log dir in config file\n");
     ini_configurator ini(application_configuration_file(),
         ini_configurator::RETURN_ONLY,
         ini_configurator::APPLICATION_DIRECTORY);
@@ -317,25 +320,25 @@ astring application_configuration::get_logging_directory()
     // they gave us something.  let's replace the environment variables
     // in their string so we resolve paths and such.
     log_dir = parser_bits::substitute_env_vars(log_dir);
+//printf("%s", (char *)a_sprintf("got log dir with %s value\n", log_dir.s()).s());
   }
 
   // now we make sure the directory exists.
-  struct stat to_fill;
-  int stat_ret = stat(log_dir.observe(), &to_fill);
-  if (stat_ret || !(to_fill.st_mode & S_IFDIR) ) {
-    // if it's not anything yet or if it's not a directory, then we need
-    // to create it.
-//if it's something besides a directory... should it be deleted?
+  filename testing(log_dir);
+  if (!testing.exists()) {
+    bool okay = directory::recursive_create(log_dir);
+    if (!okay) {
+      LOG(astring("failed to create logging directory: ") + log_dir);
+      // return a directory almost guaranteed to exist; best we can do in this case.
 #ifdef __UNIX__
-    int mk_ret = mkdir(log_dir.s(), 0777);
+      return "/tmp";
 #endif
 #ifdef __WIN32__
-    int mk_ret = mkdir(log_dir.s());
+      return "c:/";
 #endif
-    if (mk_ret) return "";
-//can't have a log file if we can't make the directory successfully???
+    }
   }
-
+    
   return log_dir;
 }