> 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.
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. - Anthony ----------------------------------------------------------- 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 > >
