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 323471 - [PLUGIN-MOVE] osxaudio
[PLUGIN-MOVE] osxaudio
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal enhancement
: 0.10.6
Assigned To: Edward Hervey
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-12-07 13:13 UTC by Benjamin Reed
Modified: 2007-03-15 11:41 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
patch for osxaudio (14.74 KB, patch)
2007-03-14 15:10 UTC, Edward Hervey
committed Details | Review

Description Benjamin Reed 2005-12-07 13:13:48 UTC
Please describe the problem:
It looks like osxaudiosink and osxvideoslink are in gst-plugins-good in CVS
HEAD, just not referenced in the Makefile.am, and they're not in the final
0.10.0 tarball.

Is this an oversight, or is there something actually wrong with these plugins?

Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Tim-Philipp Müller 2005-12-07 13:33:00 UTC
They haven't been ported yet to 0.10.
Comment 2 Zaheer Abbas Merali 2006-05-04 14:15:20 UTC
osxaudiosink and osxaudiosrc have been ported and now work better than the 0.8 versions.  They are however not yet enabled in the build as there is still things I want to improve on them.

osxvideosink i am in middle of porting.
Comment 3 Benjamin Reed 2006-05-11 17:58:12 UTC
Awesome, thanks!  If you need any help testing, I've got access to 10.3/ppc, 10.4/ppc, and 10.4/intel boxes.  Just let me know what to try building.  :)
Comment 4 Zaheer Abbas Merali 2006-06-07 13:42:07 UTC
Thanks, I confirmed over the weekend that osxaudiosink works on 10.4/intel.  Before I had only tested on 10.4/ppc.  Also should not be too long before osxvideosink is working.

Comment 5 Edward Hervey 2007-01-18 11:53:31 UTC
Zaheer, could you attach your pending patches here ?
Comment 6 Edward Hervey 2007-02-03 13:14:46 UTC
osxvideosink has been ported, it's now available in gst-plugins-bad.

I'm working on cleaning up minor issues with osxaudiosink so that it can go in -good.
Comment 7 Edward Hervey 2007-03-14 15:10:05 UTC
Created attachment 84577 [details] [review]
patch for osxaudio

Patch for osxaudio in -good.
Adds:
 * proper build setup (configure.ac / Makefile.am)
 * documentation
 * debug statement fixes
 * last fixes for ringbuffer
Comment 8 Edward Hervey 2007-03-14 15:11:35 UTC
the patch above fixes all known issues, and fulfills the requirements for having the osxaudio plugin in good.

Could somebody please review it so we can commit it ?
Comment 9 Zaheer Abbas Merali 2007-03-14 15:12:50 UTC
I will review it.
Comment 10 Zaheer Abbas Merali 2007-03-15 11:30:08 UTC
Looks good.
Comment 11 Edward Hervey 2007-03-15 11:41:06 UTC
2007-03-15  Edward Hervey  <edward@fluendo.com>

	* configure.ac:
	* sys/Makefile.am:
	* sys/osxaudio/Makefile.am:
	* sys/osxaudio/gstosxaudio.c:
	* sys/osxaudio/gstosxaudiosink.c:
	(gst_osx_audio_sink_osxelement_do_init), (gst_osx_audio_sink_init),
	(gst_osx_audio_sink_getcaps),
	(gst_osx_audio_sink_create_ringbuffer), (plugin_init):
	* sys/osxaudio/gstosxaudiosrc.c:
	(gst_osx_audio_src_osxelement_do_init), (gst_osx_audio_src_init),
	(gst_osx_audio_src_create_ringbuffer):
	* sys/osxaudio/gstosxringbuffer.c: (gst_osx_ring_buffer_get_type),
	(gst_osx_ring_buffer_class_init), (gst_osx_ring_buffer_init),
	(gst_osx_ring_buffer_acquire), (gst_osx_ring_buffer_start),
	(gst_osx_ring_buffer_pause), (gst_osx_ring_buffer_stop):
	* sys/osxaudio/gstosxringbuffer.h:
	Activate osxaudio in gst-plugins-good with proper build setup.
	Add inlined documentation.
	Fix debug statements
	Fix ringbuffer when pausing.
	Fixes #323471