From: Chris Koeritz <fred@gruntose.com>
Date: Tue, 17 Oct 2017 22:07:08 +0000 (+0000)
Subject: fixes for unfortunate log change in screen parms
X-Git-Tag: 2.140.90~128
X-Git-Url: https://feistymeow.org/gitweb/?a=commitdiff_plain;h=dfdb68204f1df5ce47e352034440403aef6c45f6;p=feisty_meow.git

fixes for unfortunate log change in screen parms

the maintainers of screen added a required parameter to the -L flag on
screen, which breaks every existing usage of the otherwise excellent
screen tool.  so, now we're forced to just add this useless parameter,
when my screenrc already takes care of setting a log file.  this change
is not even reflected on the gnu site for screen, so there's no
documentation for it at the official site.  shame on you, whoever just
stuffed this change into the screen tool.
---

diff --git a/scripts/opensim/opensim_utils.sh b/scripts/opensim/opensim_utils.sh
index 695c09cf..48291e64 100644
--- a/scripts/opensim/opensim_utils.sh
+++ b/scripts/opensim/opensim_utils.sh
@@ -36,9 +36,9 @@ function launch_screen()
   screen_name="$1"; shift
   app_name="$1"; shift
   echo "$(date_stringer ' '): starting $screen_name now..."
-  screen -L -S "$screen_name" -d -m nice -n $NICENESS_LEVEL mono "$app_name" 
-##why? --debug 
-#no, makes it ugly: -console=basic 
+#hmmm: version check for if we're using old screen?  this -L change was a mistake though for the screen project owners on ubuntu.
+local boguslog=/tmp/screen_junk.log
+  screen -L $boguslog -S "$screen_name" -d -m nice -n $NICENESS_LEVEL mono "$app_name" 
 
   echo "$(date_stringer ' '): $screen_name started."
   # only sleep if we are not at the last process that gets started.