Closed Bug 594615 Opened 12 years ago Closed 12 years ago

Update about:license with the relevant information for YARR

Categories

(Core :: JavaScript Engine, defect, P2)

x86_64
Linux
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: cdleary, Assigned: cdleary)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 file)

See bug 544755. Stuff like that.
Priority: -- → P2
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+
http://hg.mozilla.org/mozilla-central/rev/bc6eab617558
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.