Closed Bug 485713 Opened 11 years ago Closed 11 years ago

Files added by Red Hat recently have missing texts in license headers.

Categories

(NSS :: Libraries, defect, P2, trivial)

defect

Tracking

(Not tracked)

RESOLVED FIXED
3.12.3

People

(Reporter: wtc, Assigned: elio.maldonado.batiz)

Details

Attachments

(2 files)

Attachment #369831 - Flags: review?
Comment on attachment 369831 [details] [diff] [review]
Add missing text to copyright section.

Fixes item #1 in the bug report.
Doing items 2 though 5 as well.
Attachment #369832 - Flags: review?
Attachment #369831 - Flags: review? → review+
Comment on attachment 369831 [details] [diff] [review]
Add missing text to copyright section.

This patch is fine, except for one thing.

>- * The Original Code is the Netscape security libraries. *
>+ * The Original Code is Netscape Security Libraries.
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^
We now call them the "Network Security Services".
We don't label them as Netscape, and we use the word "Services".
Please fix that when you commit this change.  
No further review of the corrected patch is necessary.
Priority: -- → P2
Target Milestone: --- → 3.12.3
Version: 3.12.3 → trunk
Attachment #369832 - Flags: review? → review+
Comment on attachment 369832 [details] [diff] [review]
Add missing text to copyright section.


>- * The Initial Developer of the Original Code is Red Hat, Inc, 2008.
>+ * The Original Code is Netscape Security Libraries.

Please change all the files patched by this patch to say 
"Network Security Services" instead of "Netscape Security Libraries".
After making that change, you may commit the modified patch without 
further review.
Comment on attachment 369832 [details] [diff] [review]
Add missing text to copyright section.

The new license headers in stubs.h and stubs.c are still missing the
lines from
  ***** BEGIN LICENSE BLOCK *****
up to
  * http://www.mozilla.org/MPL/
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.