Closed Bug 897300 Opened 6 years ago Closed 6 years ago
RTC Pulse driver does not build
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20130722 Firefox/24.0 Aurora/24.0a2 (Nightly/Aurora) Build ID: 20130722024604 Steps to reproduce: Tried to compile with --enable-pulseaudio. Actual results: Build failure: /build/aurora/src/mozilla-aurora/media/webrtc/trunk/webrtc/modules/audio_device/linux/audio_device_pulse_linux.cc: In constructor 'webrtc::AudioDeviceLinuxPulse::AudioDeviceLinuxPulse(WebRtc_Word32)': /build/aurora/src/mozilla-aurora/media/webrtc/trunk/webrtc/modules/audio_device/linux/audio_device_pulse_linux.cc:82:5: error: class 'webrtc::AudioDeviceLinuxPulse' does not have any field named 'sample_rate_hz_' Expected results: Looks like the patch from http://review.webrtc.org/1384004/ was not applied correctly, and the field in audio_device_pulse_linux.h was named sampling_rate_hz instead of the sample_rate_hz_ the rest of the code uses.
doing a check build now
Attachment #780158 - Attachment is obsolete: true
Attachment #780163 - Flags: review?(paul)
Comment on attachment 780163 [details] [diff] [review] Correct mis-merge for pulse audio on Aurora from bug 886886 [Approval Request Comment] Bug caused by (feature/regressing bug #): 886886 (merge to aurora) User impact if declined: --enable-pulse is broken; breaks all BSD ports Testing completed (on m-c, etc.): on m-c; corrects mis-merge Risk to taking this patch (and alternatives if risky): none String or IDL/UUID changes made by this patch: none
Attachment #780163 - Flags: approval-mozilla-aurora?
Attachment #780163 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Status: UNCONFIRMED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.