Closed Bug 758300 Opened 8 years ago Closed 8 years ago

Remove MPL headers from non-MPL code

Categories

(Firefox for Android :: Android Sync, defect, P3)

ARM
Android
defect

Tracking

()

RESOLVED FIXED
mozilla15

People

(Reporter: rnewman, Assigned: rnewman)

References

Details

Attachments

(1 file)

The auto-switch touched these:

M mobile/android/base/json-simple/ItemList.java
M mobile/android/base/json-simple/JSONArray.java
M mobile/android/base/json-simple/JSONAware.java
M mobile/android/base/json-simple/JSONObject.java
M mobile/android/base/json-simple/JSONStreamAware.java
M mobile/android/base/json-simple/JSONValue.java
M mobile/android/base/json-simple/parser/ContainerFactory.java
M mobile/android/base/json-simple/parser/ContentHandler.java
M mobile/android/base/json-simple/parser/JSONParser.java
M mobile/android/base/json-simple/parser/ParseException.java
M mobile/android/base/json-simple/parser/Yylex.java
M mobile/android/base/json-simple/parser/Yytoken.java
Also going to use this bug to fix manifest includes and other such files -- don't want licenses popping up in the middle of preprocessed files.
Assignee: nobody → rnewman
Blocks: mpl2
Status: NEW → ASSIGNED
Attached patch Patch.Splinter Review
This reverts json-simple (third-party code) and our manifest includes (which are inlined into the middle of Fennec's AndroidManifest.xml, which should already be licensed).
Attachment #626893 - Flags: review+
Obviously, no objections here :-)

Gerv
Can the json-simple directory please acquire a LICENSE file, if it's 3rd party code? Currently, there's nothing to say what the licence is.

Thanks :-)

Gerv
Priority: -- → P3
(In reply to Gervase Markham [:gerv] from comment #5)
> Can the json-simple directory please acquire a LICENSE file, if it's 3rd
> party code? Currently, there's nothing to say what the licence is.

Absolutely! Thanks for the reminder.
https://hg.mozilla.org/mozilla-central/rev/ccee89d6d9e0
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Product: Mozilla Services → Android Background Services
Product: Android Background Services → Firefox for Android
You need to log in before you can comment on or make changes to this bug.