If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Add stlport license to about:license

RESOLVED FIXED

Status

mozilla.org
Licensing
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

4 years ago
Bug 879792 added a copy of the STLport source to our tree (with modifications from Android). Note, as mentioned in that bug or bug 850576, that before that, we were statically linking against a Android NDK-provided STLport library, meaning we might need to add the same license text on older versions, too.
glandium: as the guy who imported the code, can you prepare an about:license patch, please?

Thanks :-)

Gerv
Assignee: gerv → mh+mozilla
(Assignee)

Comment 2

4 years ago
Created attachment 792609 [details] [diff] [review]
Add STLport license to about:license
Attachment #792609 - Flags: review?(gerv)
Comment on attachment 792609 [details] [diff] [review]
Add STLport license to about:license

r=gerv.

Gerv
Attachment #792609 - Flags: review?(gerv) → review+
(Assignee)

Comment 4

4 years ago
Created attachment 795213 [details] [diff] [review]
Add STLport license to about:license

Same text, but with some ifdefs to avoid the license to spill in builds that don't have stlport.
Attachment #795213 - Flags: review?(gerv)
(Assignee)

Updated

4 years ago
Attachment #792609 - Attachment is obsolete: true
Comment on attachment 795213 [details] [diff] [review]
Add STLport license to about:license

This isn't strictly necessary because of the disclaimer at the top of the page, but assuming you've tested it works both ways, r=gerv.

Gerv
Attachment #795213 - Flags: review?(gerv) → review+
(Assignee)

Comment 6

4 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/f9abbaf73e6b

Comment 7

4 years ago
https://hg.mozilla.org/mozilla-central/rev/f9abbaf73e6b
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.