Here's my initial VoIP support, as sent to the mailing list. I'm moving this to Bugzilla so large patches aren't mailed around (but discussion can continue wherever).
We'll close this bug once we decide to integrate it into Subversion, or reject inclusion.
--ryan.
Comment 2Zachary J. Slater
2008-05-24 15:58:04 EDT
I've had this idea in my head that I want to do the next linuxgames podcast in a quake3 game with audio nicely rendered out.
The ultimate thing is dumping the video + audio to one big avi that I can then edit in iMovie and put up on LG.
Comment 3Zachary J. Slater
2008-05-24 15:58:47 EDT
Oh, so the intent of the last comment is: Make it happen :)
You also said that it won't record my own voice, I can work around that for the moment but we should be able to find some nice way.
Comment 4Zachary J. Slater
2008-05-24 16:23:19 EDT
Yeah ok one of the things that would be nice to have is automatic ducking.
Created attachment 1769[details]
Updated patch with makefile, Mumble support.
Here's an updated patch.
I realized the original patch didn't have the Makefile changes, so that's been added, and I've also plugged in Mumble support ( http://mumble.sourceforge.net/ ) for those that prefer it.
If you enable both the cl_useMumble and the voip cvar, you'll get in-game voip, but you'll send data over mumble...basically you should use one or the other, though, really, since you're cutting out the non-mumble players.
Mumble's "integration" basically consists of telling the application where your player is in the game world, so there's not much further you can go down this route. Also, it's a little hacky by nature. There's a lot more interesting things we could do with the in-game integration, I think, but there it is.
--ryan.
Created attachment 1774[details]
Updated patch, withOUT Mumble support.
Here's an update VoIP patch for svn revision #1347. This removes the Mumble support from the patch, instead buildin on Ludwig's work that is already in Subversion.
Also: VoIP is now enabled by default, for both client and server, and is built by default in the Makefile.
--ryan.
Comment 7Zachary J. Slater
2008-05-30 18:58:29 EDT
Excellent, I'll get this running on a test platform as soon as possible
Created attachment 1775[details]
Updated patch, with non-OpenAL voip playback.
Updated patch: this fixes some minor things, and adds support for multiple audio streams in snd_dma.c, so you can play VoIP packets without OpenAL. Still no audio recording support without OpenAL, though!
--ryan.
Comment 9Zachary J. Slater
2008-06-01 03:48:49 EDT
Commit this to subversion, or I'm going to have a small team of jellyfish in your home tomorrow.
Okay, the VoIP patch is in Subversion, revision #1348!
It's the latest patch attached to this bug report, plus some fixes from play-testing tonight.
--ryan.
Created attachment 1767 [details] original VoIP patch (Attaching patch to Bugzilla.)
Created attachment 1774 [details] Updated patch, withOUT Mumble support. Here's an update VoIP patch for svn revision #1347. This removes the Mumble support from the patch, instead buildin on Ludwig's work that is already in Subversion. Also: VoIP is now enabled by default, for both client and server, and is built by default in the Makefile. --ryan.
Created attachment 1775 [details] Updated patch, with non-OpenAL voip playback. Updated patch: this fixes some minor things, and adds support for multiple audio streams in snd_dma.c, so you can play VoIP packets without OpenAL. Still no audio recording support without OpenAL, though! --ryan.