> On April 20, 2016, 11:58 p.m., Joey McAllister wrote: > > The Droid Fonts seem to be the only listing under the Apache License in > > LICENSE. > > Is that correct? (I thought we listed all Apache License stuff in NOTICE.) > > And, > > if so, should we format it similarly to the other License listings? > > Otherwise, > > it blends in with the introduction rather than looking like a separate > > listing. > > In other words, something like (I'm not sure whom the copyright goes to: > > 2010 DroidFonts [from the DroidFonts website], 2006 Google Corporation [from > > the font's SVG file], or something else): > > > > --------------------------------------------------------------------------- > > The Apache License (http://www.apache.org/licenses/LICENSE-2.0) > > --------------------------------------------------------------------------- > > > > Apache Geode bundles the following files under the Apache License: > > > > - Droid fonts (http://www.droidfonts.com), Copyright (c) 2010 DroidFonts > > Joey McAllister wrote: > Reviews killed some of my formatting, but I meant for it to look like the > other License listings in the LICENSE file. > > Anthony Baker wrote: > Joey, you're not wrong :-) > > One of the IPMC members was reviewing our M2 release and suggested that > because the droid fonts are binary files it would be good to call them out in > the license file. Because it is ASL, we don't need to include the copyright > IMO.
Ah, ok. In that case, I think it's good to go as it is. - Joey ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46458/#review129841 ----------------------------------------------------------- On April 20, 2016, 11:13 p.m., Anthony Baker wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46458/ > ----------------------------------------------------------- > > (Updated April 20, 2016, 11:13 p.m.) > > > Review request for geode and Joey McAllister. > > > Repository: geode > > > Description > ------- > > GEODE-1267: Update NOTICE references to Spring dependencies > > Improved NOTICE text referencing Spring libraries for LDAP, Shell, > and HATEOS. > > GEODE-1266: Remove Pivotal header > > > GEODE-1263: Add jit.js to LICENSE > > > GEODE-1264: Add MX4J to LICENSE > > > GEODE-1265: Add Droid fonts to LICENSE > > > Diffs > ----- > > LICENSE e1fc3cbbd8227c0a1bd03a29404f3b089829c071 > geode-assembly/src/main/dist/LICENSE > 5ea39ca144c90aed2b2d3942a5c47620b2178f60 > geode-assembly/src/main/dist/NOTICE > f054cf5a2a101094d8e26e3b443e0fbcc2b96dc0 > > geode-cq/src/test/java/com/gemstone/gemfire/internal/cache/RemoteCQTransactionDUnitTest.java > d617474d8220d56f80a1087583a0b1634a45153b > geode-pulse/src/main/webapp/META-INF/NOTICE > 25a229d041ef005ec8b49bd9d63e5cc4437d727b > geode-web-api/src/main/webapp/META-INF/NOTICE > b62c5dcbee216c862c0bddbd3ff49f9dc8fc44d9 > > Diff: https://reviews.apache.org/r/46458/diff/ > > > Testing > ------- > > > Thanks, > > Anthony Baker > >
