The merge will happen after release, but we could apply this patch first if community has concern about this. Thanks.
Regards! Luke Han On Tue, Nov 3, 2015 at 7:57 PM -0800, "John D. Ament" <johndam...@apache.org> wrote: That's a good way to fix it. Do you merge your release branches back to master/next develop? On Nov 3, 2015 22:04, "Luke Han" wrote: > Anyway, removed such reference in Kylin's code, there's no more Google > Fonts or Adobe Fonts now. > > Please help to check: > > https://github.com/apache/incubator-kylin/commit/a2fa3e8e93765bf3db39f5da935aca3a588789f1 > > Thanks. > > > Best Regards! > --------------------- > > Luke Han > > On Wed, Nov 4, 2015 at 10:56 AM, Justin Mclean > wrote: > > > Hi, > > > > > The referenced font is SIL OFL 1.1 > > > http://scripts.sil.org/cms/scripts/page.php?site_id=nrsi&id=OFL > > > > > > You're not technically bundling the font, but referencing it via URL. > > It's > > > a good question for legal. > > > > The fonts are actually being bundled as well. [1] It just not obvious > from > > their names. > > > > Thanks, > > Justin > > > > 1. > > > https://github.com/apache/incubator-kylin/tree/1.x-staging/webapp/app/fonts > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > > For additional commands, e-mail: general-h...@incubator.apache.org > > > > >
