Closed
Bug 741289
Opened 12 years ago
Closed 12 years ago
Sydneyaudio does not include MOZ_ALSA_CFLAGS
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla14
People
(Reporter: prabu, Assigned: prabu)
Details
Attachments
(1 file, 3 obsolete files)
1.87 KB,
patch
|
glandium
:
review+
romaxa
:
feedback+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux i686; rv:10.0.1) Gecko/20100101 Firefox/10.0.1 Build ID: 20120209214325 Steps to reproduce: Crosscompiled Mozilla embedded for ARM. Actual results: Error: alsa/asoundlib.h not found, coming from sydneyaudio/src compilation. Expected results: MOZ_ALSA_CFLAGS should be a dependency in libsydneyaudio make, as it is a dependent component. However, current autoconf.mk, as well as config.in, only "check" for ALSA, but do not pass it on. Hence if <CROSSCOMPILEROOT>/usr/include is not in the CFLAGS path, libsydneyaudio make fails. Most times we typically add <CROSSCOMPILEROOT>/usr/include globally to CFLAGS and hence this goes unnoticed. Attached patchset to fix this.
Added :glandium" or ":ted" per inputs from :romaxa
Attachment #611374 -
Attachment is obsolete: true
Attachment #611512 -
Flags: review?(ted.mielczarek)
Attachment #611512 -
Flags: review?(romaxa)
Attachment #611512 -
Flags: review?(mh+mozilla)
Comment 2•12 years ago
|
||
Comment on attachment 611512 [details] [diff] [review] Unified patch for changes to enable passing of MOZ_ALSA_CFLAGS One more thing, In order to simplify patch integration, it is better to generate patch with hg export, or hg diff, also use -p -U 8 And add to header Bug number and description. See for example https://bug738865.bugzilla.mozilla.org/attachment.cgi?id=609019
Attachment #611512 -
Flags: review?(romaxa)
In order to simplify patch integration, generated patch with hg diff, also used -p -U 8 And added to header Bug number and description.
Attachment #611512 -
Attachment is obsolete: true
Attachment #611512 -
Flags: review?(ted.mielczarek)
Attachment #611512 -
Flags: review?(mh+mozilla)
Attachment #611759 -
Flags: review?(romaxa)
Comment 4•12 years ago
|
||
Comment on attachment 611759 [details] [diff] [review] hg patch updated per romaxa's comments Review of attachment 611759 [details] [diff] [review]: ----------------------------------------------------------------- > MOZ_ENABLE_LIBNOTIFY = @MOZ_ENABLE_LIBNOTIFY@ > > MOZ_ALSA_LIBS = @MOZ_ALSA_LIBS@ >+MOZ_ALSA_CFLAGS = @MOZ_ALSA_CFLAGS@ Make this line indented with line above pls Also I'm not able to review configure part, see https://wiki.mozilla.org/Modules/Activities#Module_Ownership_System Ask review from glandium OR ted, both are not needed for this patch
Attachment #611759 -
Flags: review?(romaxa)
Attachment #611759 -
Attachment is obsolete: true
Attachment #611788 -
Flags: review?(romaxa)
Attachment #611788 -
Flags: review?(mh+mozilla)
Updated•12 years ago
|
Attachment #611788 -
Flags: review?(romaxa) → feedback+
Updated•12 years ago
|
Attachment #611788 -
Flags: review?(mh+mozilla) → review+
Updated•12 years ago
|
Updated•12 years ago
|
Assignee: nobody → prabu
Version: 2.0 Branch → Trunk
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/810e8295b080
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•