[gstreamer-bugs] [Bug 574623] New: [dot_file_with_ts] ':' not a valid character for a filename

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=574623

  GStreamer | gstreamer (core) | Ver: 0.10.22
           Summary: [dot_file_with_ts] ':' not a valid character for a
                    filename
           Product: GStreamer
           Version: 0.10.22
          Platform: Other
        OS/Version: Windows
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gstreamer (core)
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: julien.isorce at gmail.com
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: 2.17/2.18
   GNOME milestone: Unspecified


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.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=574623.