Actually, it is not exaclty what I mean.

With this patch viewman will not start if OpenAxion running without X
session (right?),
but the problem is not to start viewman if its executable is not present.

As I see, checking for existence of viewman (and others) is not so trivial
due to all these $AXIOM environment variable and execve calls.

>From my point, it is better to have full path to external program
(e. g. "/usr/lib/openaxiom/lib/viewman"), to make a 'stat' call
to check if this file is present, and if so - start this program via
fork+exec.

2011/6/25 Gabriel Dos Reis <g...@cs.tamu.edu>

> Gabriel Dos Reis <g...@cs.tamu.edu> writes:
>
> | [...]
> |
> | | I believe it can be changed to something like this:
> | | if (start_graphics && graphics_present)         start_the_graphics();
> | |
> | | Does it make sense?
> |
> | Yes, absolutely.  Thanks for the suggestion.
>
> I applied the folliwng patch.  Let me know how it works for you.
>
> 2011-06-25  Gabriel Dos Reis  <g...@cs.tamu.edu>
>
>        * sman/sman.c (main): Don't start the graphics and no X11 server
>        is running.
>
> *** src/sman/sman.c     (revision 20418)
> --- src/sman/sman.c     (local)
> *************** main(int argc, char *argv[])
> *** 777,783 ****
>    if (start_spadclient)       start_the_spadclient();
>    if (start_local_spadclient) start_the_local_spadclient();
>    if (start_ht)               start_the_hypertex();
> !   if (start_graphics)         start_the_graphics();
>    openaxiom_sleep(1);
>
>    if (fork_you(Die) != NULL) {
> --- 777,784 ----
>    if (start_spadclient)       start_the_spadclient();
>    if (start_local_spadclient) start_the_local_spadclient();
>    if (start_ht)               start_the_hypertex();
> !   if (start_graphics and in_X())
> !      start_the_graphics();
>    openaxiom_sleep(1);
>
>    if (fork_you(Die) != NULL) {
>
------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a 
definitive record of customers, application performance, security 
threats, fraudulent activity and more. Splunk takes this data and makes 
sense of it. Business sense. IT sense. Common sense.. 
http://p.sf.net/sfu/splunk-d2d-c1
_______________________________________________
open-axiom-devel mailing list
open-axiom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open-axiom-devel

Reply via email to