[gstreamer-bugs] [Bug 517993] pipeline-tester should not be hardcoded to ALSA

[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=517993

  GStreamer | gst-python | Ver: HEAD CVS




------- Comment #6 from Brian Cameron  2008-05-08 20:46 UTC -------

Edward, I explained when I attached the patch that it was just an example
showing how things work on Solaris.  I realize that it doesn't make sense to
replace alsasink with sunaudiosink.  However, I was hoping these changes could
go upstream:

- changing "xvimagesink" to "ffmpegcolorspace ! ximagesink"
  This is needed so the script works on machines that don't support XVideo
- Fixing endianness from "1234" to "(int)BYTE_ORDER"
  This should be obvious.  Why hardcode the endianness when it is not
necessary?

These two changes seem appropriate.  They just make the script work better if
using different architectures.  If you would like, I could create a patch with 
only these changes.  Could these two changes, at least, go upstream?

I guess I'm surprised that ALSA has to be hardcoded, I would have thought that
the python interfaces would make it easy enough to get the default source and
sink.  I am not sure how to do this, or I would have updated the patch to work
this way.


-- 
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=517993.