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)

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: abr, Assigned: abr)

References

Details

(Whiteboard: [p=0.25])

Attachments

(1 file, 1 obsolete file)

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.
Note that we have sign-off on the lodash license in Bug 975548. For the OpenTok license sign-off, see Bug 956821.
Depends on: 1017880, 1017861
See Also: → 975548
Blocks: loop_mlp
Priority: -- → P1
Whiteboard: [p=0.25]
[Note that I read Gerv's comment in Bug 975548#c22 as applying to both Lodash and Backbone]
Attachment #8433606 - Flags: review?(gerv)
Attachment #8433606 - Attachment is obsolete: true
Attachment #8433606 - Flags: review?(gerv)
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)
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 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)
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
(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).
Resolved, with #ifdef guards removed.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
(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.
Blocks: 1033965
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.

Attachment

General

Created:
Updated:
Size: