59 bytes, text/x-review-board-request
As https://bugzilla.mozilla.org/show_bug.cgi?id=1340368#c5 we have to add JSZip license to about:license to cover the latest versions of JSZip that we're using. grev, could you please tell me what should I do to update license.html? Thanks! https://stuk.github.io/jszip/ https://github.com/Stuk/jszip/blob/master/LICENSE.markdown
@Gerv: is JSZip license ok, can we add it into m-c? Here is the license file: https://github.com/Stuk/jszip/blob/master/LICENSE.markdown JSZip is dual-licensed. You may use it under the MIT license or the GPLv3 license. Honza
Yes; please add it to about:license. It should be obvious how to do this (alphabetical order, title "JSZip License") but let me know if not, and give me a patch to review. Gerv
Comment on attachment 8839817 [details] Bug 1340469 - Get licensing signoff on using JSZip https://reviewboard.mozilla.org/r/114388/#review116500 ::: toolkit/content/license.html:3633 (Diff revision 3) > > <hr> > > + <h1><a id="jszip"></a>JSZip License</h1> > + > + <p>This license applies to various files in the Mozilla codebase.</p> Are the files not grouped together in a directory somewhere, then? ::: toolkit/content/license.html:3638 (Diff revision 3) > + <p>This license applies to various files in the Mozilla codebase.</p> > + > +<pre> > +The MIT License > +=============== > + Remove these three lines ::: toolkit/content/license.html:3661 (Diff revision 3) > +OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN > +THE SOFTWARE. > + > + > +GPL version 3 > +============= Er, no. Please remove the entire GPLv3 section. We are using the code under the MIT license, so that's the only one we need to reproduce. Having a copy of the GPLv3 in here will deeply confuse people.
Attachment #8839817 - Flags: review?(gerv) → review-
Grev, three issues are addressed so please check it again. thanks!
Comment on attachment 8839817 [details] Bug 1340469 - Get licensing signoff on using JSZip https://reviewboard.mozilla.org/r/114388/#review117126
Attachment #8839817 - Flags: review?(gerv) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/1aecc3e03028 Get licensing signoff on using JSZip r=gerv
Assignee: gerv → rchien
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.