594615 - Update about:license with the relevant information for YARR

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect, P2)

Description

[Chris Leary [:cdleary] (not checking bugmail)]

See bug 544755. Stuff like that.

Comment 1

[Chris Leary [:cdleary] (not checking bugmail)]

Attachment: YARR/PCRE license updates.

Comment 2

[Gervase Markham [: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

Comment 3

[Chris Leary [:cdleary] (not checking bugmail)]

(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 4

[Gervase Markham [:gerv]]

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

Comment 5

[Chris Leary [:cdleary] (not checking bugmail)]

http://hg.mozilla.org/tracemonkey/rev/bc6eab617558

Comment 6

[Robert Sayre]

http://hg.mozilla.org/mozilla-central/rev/bc6eab617558