-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pAntler + GNU screen #2
base: devel
Are you sure you want to change the base?
Conversation
…top. no logic changes.
…n added LAUNCHTYPE param to override NEWCONSOLE. stubs for LaunchType=Screen
I have done preliminary testing with this patch (on Ubuntu 12.04 LTS amd64) and this appears to be a useful feature. -Toby |
I have found a bug with a previously working configuration file, that now breaks with this patch. I have stripped out all the configuration except that line that makes it break. pAntler will launch "MOOSDB" and "uMS" with no parameters: MOOSDB whereas the expected behavior is to launch them with the typical MOOS parameters last-small.moos: ProcessConfig = pSomeApp ProcessConfig = ANTLER |
MOOSValFromString(bNewConsole, sOption, "NEWCONSOLE",true); | ||
|
||
//do we want to inhibit the passing of MOOS parameters (mission file and MOOSName) | ||
bool bInhibitMOOSParams = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, here's the initialization that was removed.
Does the problem still persist? Are there any other problems with this pull request? |
Launch Processes in GNU Screen
This patch is a refactor of pAntler in order to add a new launch option: GNU screen (http://en.wikipedia.org/wiki/GNU_Screen).
The major benefit of launching screens is that unlike
Newconsole = false
(in which stdout/stderr text isn't visible) andNewconsole = true
(in which the window with the program output is always visible), screen sessions provide a way to connect to a running process when needed, without putting xterms in your face.Before
This is an example of a pAntler config block.
After
LaunchType
replacesNewConsole
(which is now the fallback option).LaunchType
may be (case-insensitively)Process
,Console
, orScreen
.Here's the same block with the new configuration style.
Better
Here's how to launch it in a screen. We're adding the
ScreenConfig
option-L
which enables logging for the console output, but this is entirely unnecessary; it's here for demonstration.Running pAntler will launch this screen session. You can observe it by typing
screen -ls
in a terminal. You'll get something like the following:And if you type
screen -R MOOSDB
, you can connect to it:Testing
A sample MOOS configuration file,
test-antler-nix.moos
, is provided to test each of the launch methods under *nix. Among variousxmessage
dialog boxes that will come up to indicate the launch parameters of each line, there will be a summary for the screen sessions that's displayed in an xterm. Output should be similar to the following:This has been lightly tested in Ubuntu, but is untested in Win32. Presumed to work, because it's just using the old code in a new place. Please report back.