Closed Bug 716564 Opened 8 years ago Closed 8 years ago

MPL 2 upgrade: NSPR

Categories

(NSPR :: NSPR, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: gerv, Assigned: gerv)

References

Details

Attachments

(2 files)

This bug tracks the MPL 2 upgrade for the project named in the subject line.
The repo is here: http://mxr.mozilla.org/mozilla/source/nsprpub/

Gerv
Attached patch Patch v.1Splinter Review
Here is an initial patch against CVS trunk. Please let me know if it's right :-)

Gerv
Attachment #601969 - Flags: review?
Attachment #601969 - Flags: review? → review?(wtc)
Comment on attachment 601969 [details] [diff] [review]
Patch v.1

Gerv: could you add a copy of MPL 2.0 to mozilla/nsprpub?
People should not have to download it from http://mozilla.org/MPL/2.0
when they have an NSPR source distribution.  Thanks.
wtc: Certainly. Is the rest of the patch OK?

Gerv
Comment on attachment 601969 [details] [diff] [review]
Patch v.1

r=wtc.  Yes, the rest of the patch is OK.
Attachment #601969 - Flags: review?(wtc) → review+
Checked in. Plus a copy of the MPL 2.

Gerv
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Gerv: may I change this bug to be under the product NSPR?
Component: Licensing → NSPR
Product: mozilla.org → NSPR
Target Milestone: --- → 4.9.1
wtc: of course.

Gerv
Gerv: please review this patch.  I found that
mozilla/nsprpub/pkg/solaris/common_files/copyright
and all the files in mozilla/nsprpub/lib still had
the old triple license.

I've checked in this patch on the NSPR trunk for
the NSPR 4.9.1 release.
Attachment #628915 - Flags: review?(gerv)
Comment on attachment 628915 [details] [diff] [review]
Patch for mozilla/nsprpub/lib and mozilla/nsprpub/pkg/solaris/common_files/copyright

r=gerv. The copyright file got left out because it doesn't have a file extension. The "lib" directory got left out due to a script misconfiguration; I've since updated the way the script works so this cannot happen again.

Thanks for fixing it :-)

Gerv
Attachment #628915 - Flags: review?(gerv) → review+
You need to log in before you can comment on or make changes to this bug.