After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 574623 - [dot_file_with_ts] ':' not a valid character for a filename
[dot_file_with_ts] ':' not a valid character for a filename
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
0.10.22
Other Windows
: Normal normal
: 0.10.23
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-03-09 11:21 UTC by Julien Isorce
Modified: 2009-03-09 14:04 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18



Description Julien Isorce 2009-03-09 11:21:47 UTC
Hi,


-- steps to reproduce:

set GST_DEBUG_DUMP_DOT_DIR=.
set GST_DEBUG=2
gst-launch-0.10 fakesrc num_buffers=10 ! fakesink

-- result:

gst\gstdebugutils.c:571:_gst_debug_bin_to_dot_file:←[00m Failed to open file 'c:\0:00:00.170245000-gst-launch.NULL_READY.dot' for writing: Invalid argument

-- infos:

it fails on:  fopen (full_file_name, "wb")

On windows, a file name cannot contains any of the following charaters:
\ / : * ? " < > |
And in the timestamp there is: ':'

g_strdup_printf ("%" GST_TIME_FORMAT "-%s", GST_TIME_ARGS (elapsed),
      file_name);

and #define GST_TIME_FORMAT "u:%02u:%02u.%09u"

-- other:

I can make a patch that just replace the 2 characters ':' by a '_' but I would like to know what do you think about it first.
Comment 1 Stefan Sauer (gstreamer, gtkdoc dev) 2009-03-09 11:40:37 UTC
Right thats the timestamp. Too bad that glib has nothing there. Let me just fix it. Thanks for analysing it.
Comment 2 Stefan Sauer (gstreamer, gtkdoc dev) 2009-03-09 14:04:12 UTC
commit 50fdaa4f25c32c3971b5d0a559913c5138ac7e0e
Author: Stefan Kost <ensonic@users.sf.net>
Date:   Mon Mar 9 16:01:20 2009 +0200

    dump2dot: don't use GST_TIME_FORMAT when building filenames. fixes #574623