Closed Bug 1296600 Opened 3 years ago Closed 3 years ago

Review licenses for new external libraries: node-properties & sprintf.js (devtools)

Categories

(mozilla.org :: Licensing, task)

task
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jdescottes, Assigned: gerv)

References

Details

Attachments

(1 file)

(see Bug 1294220)

I want to use two external libraries in devtools (https://hg.mozilla.org/mozilla-central/file/tip/devtools). 

The libraries are :
- https://github.com/alexei/sprintf.js (license: https://github.com/alexei/sprintf.js/blob/master/LICENSE)
- https://github.com/gagle/node-properties (MIT license: https://github.com/gagle/node-properties/blob/master/LICENSE)

The files themselves would go in devtools/client/shared/vendor, which already contains other external libraries. 

Both libraries require modifications before we can use them.

What is the process to add those libraries here?
Blocks: 1294220
Both of these are fine. As devtools ships with Firefox (doesn't it? if it only ships with some versions, we may need to investigate ifdefs), you need to add the licenses to about:license, in alphabetical order - ask me to review the patch, please.

Gerv
(In reply to Gervase Markham [:gerv] from comment #1)
> Both of these are fine. As devtools ships with Firefox (doesn't it? if it
> only ships with some versions, we may need to investigate ifdefs), 
Yes it ships with Firefox, no need for ifdefs.

> you need
> to add the licenses to about:license, in alphabetical order - ask me to
> review the patch, please.

Will do, thanks for answering quickly!
Comment on attachment 8783200 [details]
Bug 1296600 - add sprintf.js and node-properties licenses to about:license;

https://reviewboard.mozilla.org/r/73132/#review71078

r=gerv.
Attachment #8783200 - Flags: review?(gerv) → review+
Thanks for the review!
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a22d1d1dbb3e
add sprintf.js and node-properties licenses to about:license;r=gerv
https://hg.mozilla.org/mozilla-central/rev/a22d1d1dbb3e
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.