Closed
Bug 1019924
Opened 11 years ago
Closed 11 years ago
Update license.html for Loop-related third-party licenses
Categories
(mozilla.org :: Licensing, task, P1)
mozilla.org
Licensing
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: abr, Assigned: abr)
References
Details
(Whiteboard: [p=0.25])
Attachments
(1 file, 1 obsolete file)
15.86 KB,
patch
|
gerv
:
review+
|
Details | Diff | Splinter Review |
This is the update to license.html that needs to land once we have approval for the various third-party libraries we're using in Loop.
Assignee | ||
Comment 1•11 years ago
|
||
Assignee | ||
Comment 2•11 years ago
|
||
Note that we have sign-off on the lodash license in Bug 975548.
For the OpenTok license sign-off, see Bug 956821.
Assignee | ||
Updated•11 years ago
|
Assignee | ||
Comment 3•11 years ago
|
||
[Note that I read Gerv's comment in Bug 975548#c22 as applying to both Lodash and Backbone]
Assignee | ||
Updated•11 years ago
|
Attachment #8433606 -
Flags: review?(gerv)
Assignee | ||
Comment 4•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Attachment #8433606 -
Attachment is obsolete: true
Attachment #8433606 -
Flags: review?(gerv)
Assignee | ||
Comment 5•11 years ago
|
||
Comment on attachment 8435052 [details] [diff] [review]
Update license.html for Loop-related third-party licenses
Fixing path on OpenTok license
Attachment #8435052 -
Flags: review?(gerv)
Comment 6•11 years ago
|
||
Hi Gerv -- Do you think you'll have time to review the licensing patch from Adam soon? We'd like to land Loop-Desktop in Nightly shortly after Monday's uplift (either late Monday or early Tuesday). Thanks.
Flags: needinfo?(gerv)
Comment 7•11 years ago
|
||
Comment on attachment 8435052 [details] [diff] [review]
Update license.html for Loop-related third-party licenses
r=gerv if you need to get this landed. However, you should remove the ifdefs. There's a disclaimer at the top which says that depending on build options, not all these licenses will apply. Because if that, we don't make license inclusion conditional.
Gerv
Attachment #8435052 -
Flags: review?(gerv) → review+
Flags: needinfo?(gerv)
Comment 8•11 years ago
|
||
One further thing: minified JS code should not be in the tree without its corresponding source. Please make sure that gets checked in as well when you land Loop. (It is not required, by me at least, that the code be reminified by everyone building the source, although you should explain how it's done, either with a Makefile or a README.)
Gerv
Assignee | ||
Comment 9•11 years ago
|
||
Assignee | ||
Comment 10•11 years ago
|
||
(In reply to Gervase Markham [:gerv] from comment #8)
> One further thing: minified JS code should not be in the tree without its
> corresponding source. Please make sure that gets checked in as well when you
> land Loop. (It is not required, by me at least, that the code be reminified
> by everyone building the source, although you should explain how it's done,
> either with a Makefile or a README.)
I'll fix this in another bug. Thanks for the pointer. (Getting unminified code out of the sjcl build system wasn't immediately obvious, but I have it working now).
Assignee | ||
Comment 11•11 years ago
|
||
Resolved, with #ifdef guards removed.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 13•11 years ago
|
||
(In reply to Adam Roach [:abr] from comment #10)
> I'll fix this in another bug. Thanks for the pointer. (Getting unminified
> code out of the sjcl build system wasn't immediately obvious, but I have it
> working now).
See Bug 1022772.
Comment 14•11 years ago
|
||
Comment 15•11 years ago
|
||
Verified fixed in the latest Nightly.
Status: RESOLVED → VERIFIED
QA Contact: anthony.s.hughes
You need to log in
before you can comment on or make changes to this bug.
Description
•