Hate to continue the train... Can you please review the license of MD5 w/ legal? https://www.npmjs.com/package/MD5
In addition, intro.js seems to be a MIT license but is not called as such https://github.com/usablica/intro.js Things you may want to list in your notice file. John On Tue Dec 16 2014 at 11:48:38 AM Shawn Feldman <[email protected]> wrote: > it was a reference in the gruntfile. no code was in our repo > > On Tue Dec 16 2014 at 9:46:40 AM Rod Simpson <[email protected]> wrote: > >> Huh, I thought we had gotten rid of all the highcharts code. >> >> -- >> Rod Simpson >> T @rockerston >> W rodsimpson.com >> >> On December 16, 2014 at 9:43:11 AM, John D. Ament ([email protected]) >> wrote: >> >> My UI guy just started flailing his arms about complaining about build >> tools consistency when i asked him about this. :-) >> >> Thanks guys. >> >> On Tue Dec 16 2014 at 11:07:15 AM Shawn Feldman <[email protected]> >> wrote: >> >> > the pr will remove that. we must have cleaned it up a while ago and >> forgot >> > about that line. >> > >> > On Tue Dec 16 2014 at 9:03:33 AM John D. Ament <[email protected]> >> > wrote: >> > >> > > Yeah that's weird. I guess grunt doesn't complain about missing files >> if >> > > they're not used? >> > > >> > > On Tue Dec 16 2014 at 11:02:30 AM Shawn Feldman < >> [email protected] >> > > >> > > wrote: >> > > >> > > > https://github.com/apache/incubator-usergrid/pull/124 >> > > > >> > > > On Tue Dec 16 2014 at 8:57:33 AM Shawn Feldman < >> > [email protected]> >> > > > wrote: >> > > > >> > > > > here is the only reference i can find >> > > > > https://github.com/apache/incubator-usergrid/blob/ >> > > > master/portal/Gruntfile.js#L78 >> > > > > >> > > > > we can just remove that line >> > > > > >> > > > > On Tue Dec 16 2014 at 8:56:40 AM Shawn Feldman < >> > > [email protected]> >> > > > > wrote: >> > > > > >> > > > >> John do you see the file? all i can see is one line in the grunt >> > > file. >> > > > >> >> > > > >> On Tue Dec 16 2014 at 8:53:45 AM John D. Ament < >> > [email protected] >> > > > >> > > > >> wrote: >> > > > >> >> > > > >>> From my point of view, you cannot create an apache release w/ >> this >> > > lib >> > > > in >> > > > >>> here. This was missed in the 1.0.0 release but that should not >> > have >> > > > been >> > > > >>> releasable. >> > > > >>> >> > > > >>> On Tue Dec 16 2014 at 10:50:57 AM Shawn Feldman < >> > > > [email protected] >> > > > >>> > >> > > > >>> wrote: >> > > > >>> >> > > > >>> > i think we need to remove the lib, then remove the chart on >> the >> > > > >>> application >> > > > >>> > home page, then verify the tests pass >> > > > >>> > >> > > > >>> > On Tue Dec 16 2014 at 8:48:14 AM Dave Johnson < >> > [email protected] >> > > > >> > > > >>> wrote: >> > > > >>> > >> > > > >>> > > We should definitely remove it and as soon as we can. I’d >> like >> > to >> > > > >>> get an >> > > > >>> > > RC out for 1.0.1 today if possible. >> > > > >>> > > >> > > > >>> > > What needs to be done to remove it? >> > > > >>> > > >> > > > >>> > > - Dave >> > > > >>> > > >> > > > >>> > > >> > > > >>> > > >> > > > >>> > > >> > > > >>> > > > On Dec 16, 2014, at 10:30 AM, Shawn Feldman < >> > > > >>> [email protected]> >> > > > >>> > > wrote: >> > > > >>> > > > >> > > > >>> > > > We should probably remove it. It was used in our >> proprietary >> > > > >>> codebase >> > > > >>> > > and I >> > > > >>> > > > think only serves one chart on the application page >> > > > >>> > > > >> > > > >>> > > > On Tue, Dec 16, 2014, 7:11 AM John D. Ament < >> > > > [email protected] >> > > > >>> > >> > > > >>> > > wrote: >> > > > >>> > > > >> > > > >>> > > >> Hi, >> > > > >>> > > >> >> > > > >>> > > >> In the portal app, there's a dependency on Highcharts >> 2.3.5. >> > > As >> > > > >>> best >> > > > >>> > > as I >> > > > >>> > > >> can tell, this is a commercial license. >> > > > >>> http://www.highcharts.com/ >> > > > >>> > > >> >> > > > >>> > > >> Can you confirm this? >> > > > >>> > > >> >> > > > >>> > > >> John >> > > > >>> > > >> >> > > > >>> > > >> > > > >>> > > >> > > > >>> > >> > > > >>> >> > > > >> >> > > > >> > > >> > >> >
