Closed
Bug 930242
Opened 11 years ago
Closed 11 years ago
Consolidate ogg moz.build files
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla28
People
(Reporter: rillian, Assigned: rillian)
Details
(Whiteboard: [qa-])
Attachments
(1 file, 1 obsolete file)
3.31 KB,
patch
|
Details | Diff | Splinter Review |
Improve readability and parse time by replacing recursive build files with single ones in the top level source directory.
Assignee | ||
Comment 1•11 years ago
|
||
Replace recursive build files with a single pair in the top level source for better readability and parse time.
Attachment #821318 -
Flags: review?(ted)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → giles
Comment 2•11 years ago
|
||
Comment on attachment 821318 [details] [diff] [review] Consolidate ogg build files Review of attachment 821318 [details] [diff] [review]: ----------------------------------------------------------------- Good stuff!
Attachment #821318 -
Flags: review?(ted) → review+
Comment 3•11 years ago
|
||
Comment on attachment 821318 [details] [diff] [review] Consolidate ogg build files Review of attachment 821318 [details] [diff] [review]: ----------------------------------------------------------------- ::: media/libogg/README_MOZILLA @@ +1,3 @@ > The source from this directory was copied from the libogg subversion > repository using the update.sh script. The only changes made were > +those applied by update.sh and the addition/upate of moz.build and update
Assignee | ||
Comment 4•11 years ago
|
||
Rebase on top of changes from bugs 929905 and 930674, and fix the typo Ms2ger noticed. Carrying forward r=ted.
Attachment #821318 -
Attachment is obsolete: true
Comment 5•11 years ago
|
||
I actually was motivated to write the patch for the latter of those because I noticed that was the only thing left in your Makefile.in. :)
Assignee | ||
Comment 6•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/773544d9c681
Assignee | ||
Comment 7•11 years ago
|
||
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #5) > I actually was motivated to write the patch for the latter of those because > I noticed that was the only thing left in your Makefile.in. :) Thanks! It was nice to be able to delete the Makefile entirely. A cc would have been nice though. -r
https://hg.mozilla.org/mozilla-central/rev/773544d9c681
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Updated•10 years ago
|
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•