Do not print start banner if stderr is not a TTY.

The user is not going to see the banner, and this tends to clutter
error logs (X server, cron jobs...) - not only on MacOS.

diff --git a/bin/vlc.c b/bin/vlc.c
index 75a97a1379bcfed64342e621cbb7a1cca9df4576..ee30bd419659302280c6b3891015d245f2690a05 100644 (file)
--- a/bin/vlc.c
+++ b/bin/vlc.c
@@ -121,11 +121,11 @@ int main( int i_argc, const char *ppsz_argv[] )
 
     setlocale (LC_ALL, "");
 
-#ifndef __APPLE__
-    /* This clutters OSX GUI error logs */
-    fprintf( stderr, "VLC media player %s (revision %s)\n",
-             libvlc_get_version(), libvlc_get_changeset() );
-#endif
+    if (isatty (STDERR_FILENO))
+        /* This message clutters error logs. It is print it only on a TTY.
+         * Forunately, LibVLC prints version infos with -vv anyhow. */
+        fprintf (stderr, "VLC media player %s (revision %s)\n",
+                 libvlc_get_version(), libvlc_get_changeset());
 
     sigset_t set;