Closed Bug 779089 Opened 10 years ago Closed 10 years ago

Add proper Apache 2 header to files based on readability.js

Categories

(Firefox for Android Graveyard :: Reader View, defect)

defect
Not set
normal

Tracking

(firefox16+ fixed)

RESOLVED FIXED
Firefox 17
Tracking Status
firefox16 + fixed

People

(Reporter: gerv, Assigned: bnicholson)

Details

Attachments

(1 file)

https://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/Readability.js#1
(and any other files based on Readability which are in a similar licensing position, if there are any) should have a proper Apache 2 header:

See the Appendix here for a copy of the text:
http://www.apache.org/licenses/LICENSE-2.0.html

Obviously, replace the copyright placeholder with Arc90's details from the current file.

Thanks :-)

Gerv
Again cc'ing blassey, mfinkle, and elancaster so that the right people are seeing this and assigning someone to work on the issue.
Attached patch patchSplinter Review
Assignee: nobody → bnicholson
Status: NEW → ASSIGNED
Attachment #650209 - Flags: review?(gerv)
Comment on attachment 650209 [details] [diff] [review]
patch

r=gerv.

Gerv
Attachment #650209 - Flags: review?(gerv) → review+
Comment on attachment 650209 [details] [diff] [review]
patch

[Approval Request Comment]
Comment-only change. We should have this in Aurora for licensing reasons.
Attachment #650209 - Flags: approval-mozilla-aurora?
Comment on attachment 650209 [details] [diff] [review]
patch

Thanks for getting this wrapped up, guys.
Attachment #650209 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/mozilla-central/rev/9372939f013e
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 17
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.