Update about:license with the relevant information for YARR

RESOLVED FIXED

Status

()

P2
normal
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: cdleary, Assigned: cdleary)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 attachment)

See bug 544755. Stuff like that.
Priority: -- → P2
Created attachment 484246 [details] [diff] [review]
YARR/PCRE license updates.
Attachment #484246 - Flags: review?(sayrer)

Updated

8 years ago
Attachment #484246 - Flags: review?(sayrer) → review?(gerv)
Comment on attachment 484246 [details] [diff] [review]
YARR/PCRE license updates.

You've got the "list of files this applies to" wrong, or incomplete. There needs to be a list associated with the University of Cambridge license; I don't know if you mistakenly added some paths to the Apple License instead, or if you did that correctly but didn't supply any paths for the University of Cambridge one. Either way, it needs fixing :-)

Gerv
Attachment #484246 - Flags: review?(gerv) → review-
(In reply to comment #2)
> You've got the "list of files this applies to" wrong, or incomplete. There
> needs to be a list associated with the University of Cambridge license

I don't understand what you're saying. There are files added to both the Apple license and Cambridge license...

+    <h1><a name="ucambridge"></a>University of Cambridge License</h1>
+
+    <p class="correctme">This license applies to certain files in the directory <span class="path">js/src/yarr/pcre/</span>.</p>
Comment on attachment 484246 [details] [diff] [review]
YARR/PCRE license updates.

You are quite right. Not sure where that blind spot came from. Sorry!

Gerv
Attachment #484246 - Flags: review- → review+

Comment 6

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