Mark Seger wrote:
> I wouldn't call it a bug.  On the other hand, if you see that $DISPLAY
> has not been set and tell the user, it will eliminate a number of
> support calls/questions.  Call an enhancement.  8-)

I agree, now a warning is printed if $DISPLAY is not defined in the
environment.

Thanks,
-Andrea

-------- Original Message --------
Subject: [systemimager-commits] r3710 - trunk/sbin
Date: Fri, 04 Aug 2006 02:56:19 -0500
From: Andrea Righi <[EMAIL PROTECTED]>
Reply-To: [email protected]
To: [EMAIL PROTECTED]

Author: arighi
Date: 2006-08-04 02:56:19 -0500 (Fri, 04 Aug 2006)
New Revision: 3710

Modified:
   trunk/sbin/si_monitortk
Log:
print a warning if :0 is not defined in the environment (suggested by
Mark Seger)

Modified: trunk/sbin/si_monitortk
===================================================================
--- trunk/sbin/si_monitortk     2006-08-03 16:42:31 UTC (rev 3709)
+++ trunk/sbin/si_monitortk     2006-08-04 07:56:19 UTC (rev 3710)
@@ -121,6 +121,11 @@
     $refresh_rate *= 1000;
 }

+# Check if X is available.
+unless ($ENV{DISPLAY}) {
+    print STDERR "WARNING: this program requires an X server to run!\n";
+}
+
 # Define lock files.
 my $lock_file = $conf{'lock_dir'} . "/db.si_monitor.lock";

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Sisuite-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sisuite-devel

Reply via email to