Hi Matt, And that’s my point: chardet is not a direct Liminal dependency, but actually an Airflow dependency.
So, I think that’s either Liminal can release "as it is" or Airflow should address this dependency. Regards JB > Le 20 avr. 2021 à 15:46, Matt Sicker <boa...@gmail.com> a écrit : > > LGPL is only allowed in optional dependencies. > > On Tue, Apr 20, 2021 at 08:31 Lior Schachter <lior...@gmail.com> wrote: > >> We can't just remove this dependency as it comes from the Airflow package >> deoendencies. >> >> On Tue, Apr 20, 2021, 15:51 JB Onofré <j...@nanthrax.net> wrote: >> >>> Did it change about LGPL ? I’m pretty sure LGPL was cat B and GPL was cat >>> X. >>> Maybe it changed (or I’m wrong). >>> >>> Anyway I would cancel this vote to remove this dependency. >>> >>> Regards >>> JB >>> >>>> Le 20 avr. 2021 à 14:20, Lior Schachter <lior...@gmail.com> a écrit : >>>> >>>> Hi Justin, >>>> chardet is indeed LGPL (https://pypi.org/project/chardet/). >>>> >>>> However, from a quick analysis it seems it is installed as Apache >> Airflow >>>> project dependency. >>>> See below the dependency analysis of this package. >>>> >>>> (apache-liminal-002rc1) lior.schachter@liors-mac >>> liminal-getting-started % >>>> pipdeptree --reverse --packages chardet >>>> >>>> chardet==4.0.0 >>>> >>>> - requests==2.25.1 [requires: chardet>=3.0.2,<5] >>>> >>>> - apache-airflow==1.10.12 [requires: requests>=2.20.0,<3] >>>> >>>> - apache-liminal==0.0.2rc1 [requires: apache-airflow==1.10.12] >>>> >>>> - docker==4.2.0 [requires: requests>=2.14.2,!=2.18.0] >>>> >>>> - apache-liminal==0.0.2rc1 [requires: docker==4.2.0] >>>> >>>> - kubernetes==12.0.1 [requires: requests] >>>> >>>> - apache-liminal==0.0.2rc1 [requires: kubernetes==12.0.1] >>>> >>>> - requests-oauthlib==1.3.0 [requires: requests>=2.0.0] >>>> >>>> - kubernetes==12.0.1 [requires: requests-oauthlib] >>>> >>>> - apache-liminal==0.0.2rc1 [requires: kubernetes==12.0.1] >>>> >>>> Please advise how to proceed, >>>> Lior >>>> >>>>> On Tue, Apr 20, 2021 at 11:53 AM Hans Van Akelyen < >>>>> hans.van.akel...@gmail.com> wrote: >>>>> >>>>> Aren't both (L)GPL Category X [1]? >>>>> >>>>> Cheers, >>>>> Hans >>>>> >>>>> [1] https://www.apache.org/legal/resolved.html >>>>> >>>>> On Tue, 20 Apr 2021 at 10:48, Jean-Baptiste Onofre <j...@nanthrax.net> >>>>> wrote: >>>>> >>>>>> Good catch. AFAIR chardet is LGPL based (not GPL), and LGPL is Cat B >>> (not >>>>>> X). >>>>>> >>>>>> @Liminal team, do you confirm LGPL license for chardet (on Pypi) ? >>>>>> >>>>>> Regards >>>>>> JB >>>>>> >>>>>>> Le 20 avr. 2021 à 08:43, Justin Mclean <jus...@classsoftware.com> a >>>>>> écrit : >>>>>>> >>>>>>> Hi, >>>>>>> >>>>>>> Sorry it’s -1 (binding) form me. If issue re the dependancy is >>>>> clarified >>>>>> I’ll change my vote. >>>>>>> >>>>>>> I checked: >>>>>>> - incubating in name >>>>>>> - signatures and hashes are fine >>>>>>> - DISCLAIMER exists >>>>>>> - LICENSE and NOTICE fine >>>>>>> - no expected binary files >>>>>>> - All source files have ASF headers >>>>>>> >>>>>>> I was looking the dependancies and noticed one with Category X GPL >>>>>> license - chardet >>>>>>> >>>>>>> Thanks, >>>>>>> Justin >>>>>>> >>>>>> >>>>>> >>>>>> --------------------------------------------------------------------- >>>>>> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org >>>>>> For additional commands, e-mail: general-h...@incubator.apache.org >>>>>> >>>>>> >>>>> >>> >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org >>> For additional commands, e-mail: general-h...@incubator.apache.org >>> >>> >> --------------------------------------------------------------------- To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org
