Closed
Bug 1471164
Opened 6 years ago
Closed 6 years ago
Update cubeb from upstream to 2968cba
Categories
(Core :: Audio/Video: cubeb, defect, P2)
Core
Audio/Video: cubeb
Tracking
()
RESOLVED
FIXED
mozilla63
People
(Reporter: achronop, Assigned: achronop)
References
Details
Attachments
(2 files)
59 bytes,
text/x-review-board-request
|
padenot
:
review+
|
Details |
8.79 KB,
patch
|
padenot
:
review+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
Pick commits: 2968cba wasapi: Reduce timeout threshold (#447) 27be938 audiounit: new method on failed stream init to avoid deadlock (BMO 1470113). (#445) bf2c281 cubeb_winmm.c: Don't define __MSVCRT_VERSION__.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → achronop
Rank: 19
Priority: -- → P2
Assignee | ||
Updated•6 years ago
|
Comment hidden (mozreview-request) |
Comment 2•6 years ago
|
||
mozreview-review |
Comment on attachment 8987786 [details] Bug 1471164 - Update cubeb from upstream to 2968cba. https://reviewboard.mozilla.org/r/253064/#review259996
Attachment #8987786 -
Flags: review?(padenot) → review+
Pushed by achronop@gmail.com: https://hg.mozilla.org/integration/autoland/rev/6b9607c2eb18 Update cubeb from upstream to 2968cba. r=padenot
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/6b9607c2eb18
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Comment 5•6 years ago
|
||
The __MSVCRT_VERSION__ fix is something I'd like to backport to esr60... what would be the best way to do that? cherry-pick that individually?
Flags: needinfo?(padenot)
Comment 6•6 years ago
|
||
We usually do the following: - cherry-pick the patch from trunk, put it in `media/libcubeb`, add it for check in. - add a line to `media/libcubeb/update.sh`, adding the right patch command call to apply the patch file (like so [0]) - make a regular commit stating the changes. This should include the patch file, the file that has been patch, and the modification to `update.sh`. This is maybe a slightly unusual process, but the rationale here is to be able to know the base revision of libcubeb that is in the tree, and what exactly have been back-ported (or sometimes added on top, for example for a gecko-specific patch), and all that easily. Let me know if I can be of any help. [0]: https://hg.mozilla.org/releases/mozilla-esr60/file/tip/media/libcubeb/update.sh#l78
Flags: needinfo?(padenot)
Comment 7•6 years ago
|
||
I attached such esr60 version to bug 1471016.
Comment 8•6 years ago
|
||
Alex, can you request beta uplift for this patch (re: https://bugzilla.mozilla.org/show_bug.cgi?id=1470113#c5)
Flags: needinfo?(achronop)
Updated•6 years ago
|
status-firefox62:
--- → affected
tracking-firefox62:
--- → +
Assignee | ||
Comment 9•6 years ago
|
||
Approval Request Comment [Feature/Bug causing the regression]: Bug 1466066 [User impact if declined]: crash when request to play audio and the request fail [Is this code covered by automated tests?]: no [Has the fix been verified in Nightly?]: yes [Needs manual test from QE? If yes, steps to reproduce]: N/A [List of other uplifts needed for the feature/fix]: It is not needed but I have created this uplift patch on the top of beta uplift for Bug 1469152. I have done that because it would conflict otherwise. If you are not planning to uplift the other bug let me know to update it. [Is the change risky?]: No [Why is the change risky/not risky?]: It has been working well on Nightly and it's rather small [String changes made/needed]:N/A
Flags: needinfo?(achronop)
Attachment #8991357 -
Flags: review?(padenot)
Attachment #8991357 -
Flags: approval-mozilla-beta?
Updated•6 years ago
|
Attachment #8991357 -
Flags: review?(padenot) → review+
Comment 10•6 years ago
|
||
Comment on attachment 8991357 [details] [diff] [review] Bug 1471164 - Beta uplift for cubeb crash in stream destroy. Fix for audio crash, it's been on nightly for a while, let's take this for beta 8.
Attachment #8991357 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 11•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/77d51fa00f58
You need to log in
before you can comment on or make changes to this bug.
Description
•