liboggz/update.sh doesn't fit reality

RESOLVED WONTFIX

Status

()

Core
Audio/Video
RESOLVED WONTFIX
8 years ago
8 years ago

People

(Reporter: glandium, Unassigned)

Tracking

1.9.1 Branch
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

8 years ago
Created attachment 414241 [details] [diff] [review]
Patch

The bounded_seek patch was applied upstream some time ago, and while the patch was removed from the tree, it wasn't removed from update.sh. The attached patch fixes that.
(Reporter)

Updated

8 years ago
Attachment #414241 - Attachment is patch: true
Attachment #414241 - Attachment mime type: application/octet-stream → text/plain
Attachment #414241 - Flags: review?(chris.double)
(Reporter)

Comment 1

8 years ago
Created attachment 414242 [details] [diff] [review]
Patch v2

It was also in README_MOZILLA
Attachment #414241 - Attachment is obsolete: true
Attachment #414242 - Flags: review?(chris.double)
Attachment #414241 - Flags: review?(chris.double)

Comment 2

8 years ago
Comment on attachment 414242 [details] [diff] [review]
Patch v2

Chris Pearce would know better than me the status of the bounded_seek patch and whether is no longer needed. Changing reviewer to him.
Attachment #414242 - Flags: review?(chris.double) → review?(chris)
This only applies to the 1.9.1 branch.
Comment on attachment 414242 [details] [diff] [review]
Patch v2

This is already fixed on trunk and 1.9.2. We're only taking security and stability fixes on 1.9.1 now, so there's not a much chance of this being accepted. We'll no doubt run into this if we ever update liboggz in future on 1.9.1, so let's hold off fixing this until such a time.
Attachment #414242 - Flags: review?(chris) → review-
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.