Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread Jean-Baptiste Onofre
Hi Justin,

I agree, with an updated DISCLAIMER, I guess we can move forward with 0.0.2 
release.

However, we have to double check that chardet is not a "direct" dependency (it 
seems to come from Airflow, but I didn’t actually check yet).

Regards
JB

> Le 21 avr. 2021 à 00:15, Justin Mclean  a écrit :
> 
> Hi,
> 
>> 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.
> 
> If you had a WIP-DISCLAIMER and listed this issue in it you could make a 
> release.
> 
> I wold contact Airflow and have a chat to them and see why it needed. 
> 
> BTW I’m not 100% sure it is an Airflow dependancy, but I’m not 100% familiar 
> with how python pip tools work.
> 
> pipdeptree for me gives:
> 
> pipreqs==0.4.10
>  - docopt [required: Any, installed: 0.6.2]
>  - yarg [required: Any, installed: 0.1.9]
>- requests [required: Any, installed: 2.25.1]
>  - certifi [required: >=2017.4.17, installed: 2020.4.5.1]
>  - chardet [required: >=3.0.2,<5, installed: 4.0.0]
>  - idna [required: >=2.5,<3, installed: 2.10]
>  - urllib3 [required: >=1.21.1,<1.27, installed: 1.26.2]
> 
> pipdeptree --reverse --packages chardet gives:
> 
> chardet==4.0.0 
>  - requests==2.25.1 [requires: chardet>=3.0.2,<5]
>- yarg==0.1.9 [requires: requests]
>  - pipreqs==0.4.10 [requires: yarg]
> 
> Which is different to what you have.
> 
> Pipregs is just for producing requirements.txt right? So it may not be a 
> runtime dependancy and be considered optional?
> 
> From  running  pip-licenses Yargs is MIT, requests is ALv2 and chardet is 
> LGPLv2.
> 
> 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



Re: [VOTE] Release Apache Doris 0.14.0-incubating-rc05

2021-04-20 Thread Justin Mclean
Hi,

> And on the website, there are also other cloud companies which use the name
> 'Doris' directly. For example:
> 
> https://cloud.tencent.com/product/cdwdoris

Which is absolutely no excuse for anther 3rd party to do so. The PPMC need to 
be managing its brand (which is why there is a question on it in every 
incubator report) and if you see something that isn’t following ASF trademarks 
you ned to report it to the PPMC and/or trademarks.

I notice in your last report you said:

### Is the PPMC managing the podling's brand / trademarks?

 Yes. 3rd parties are using the podlings name and brand correctly. And the
 project name is approved.


Clearly this is not the case. Issues like this will stop the project from 
graduating.

Thanks,
Justin
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: [VOTE] Release Apache Doris 0.14.0-incubating-rc05

2021-04-20 Thread ling miao
Hi Justin,

> - There are still trademark/ branding issues with [1]
We are ready to use apache official docker hub to publish the Doris Dev
Image instead of this website https://hub.docker.com/u/apachedoris.
The email  thread :
https://lists.apache.org/thread.html/rd66706e1713db23f16906eab825fadaa36d08fa2deda6961483912e0%40%3Cdev.doris.apache.org%3E
After we modify it successfully, this repo will be closed. Is this possible?

> - It looks like this still contains unreleased code [1]
This docker image is actually our development environment. It does not
contain any code.
Could you please specify where the unreleased code is included? Let me
correct it immediately.

Ling Miao



Justin Mclean  于2021年4月21日周三 上午10:28写道:

> Hi,
>
> -1 (binding) as not all issues from last RC have not been corrected in
> particular the NOTICE file
>
> I checked:
> - incubating in name
> - signature and hash is fine
> - LICENSE has been updated and files mentioned in last RC before removed
> - NOTICE is still not correct. The notice file is not for listing
> included/bundled 3rd party software. The notice file should include any
> relocated copyright files, (usually from software grants) and parts of
> notice files from Apache licensed bundled software.
> - some files are missing ASF headers [4][5][6]
> - No unexpected binary files
> - I didn't compile from source
>
>
> I can also see:
> - There are still trademark/ branding issues with [1]
> - It looks like this still contains unreleased code [1]
> - This has still not been corrected [2] I can read the original but the
> translated text is quite concerning for a number of reasons "As the main
> maintenance team of the Apache Doris community, the Baidu Doris team also
> maintains a 3-bit iterative version (tags) based on the official Release
> version of Apache Doris. Including quick bug fixes and new feature updates.”
> - I also see mention of a product named "Baidu Data Warehouse Doris” this
> would not be allowed under ASF trademark policy.
> - I also see this domain name "http://doris.baidu.com/“ being used, This
> is not allowed under ASF trademark policy.
> - This page [3] is in serious violation of ASF trademark and branding
> policy. Again I cannot read the original but claiming it's the official
> website Baidu Doris for starters is unacceptable, as is the use of Doris
> logo on the page, not respecting ASF trademark etc etc IMO This site need
> to be corrected or removed ASAP.
>
> Thanks,
> Justin
>
>
>
> 1. https://hub.docker.com/u/apachedoris
> 2.
> https://m4jh7kuq4qkiaojsq5ytf2qzvi-ac4c6men2g7xr2a-blog-csdn-net.translate.goog/ucanuup_/article/details/115548997
> 3. http://doris.baidu.com/home
> 4. ./fe_plugins/pom.xml
> 5. ./fe_plugins/auditdemo/pom.xml
> 6. ./fe_plugins/auditloader/pom.xml
>
>
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>


Re: [VOTE] Release Apache Doris 0.14.0-incubating-rc05

2021-04-20 Thread Duo Zhang
On the docker hub, I saw the PPMC has already started to solve it.

https://lists.apache.org/thread.html/rd66706e1713db23f16906eab825fadaa36d08fa2deda6961483912e0%40%3Cdev.doris.apache.org%3E

And on the website, there are also other cloud companies which use the name
'Doris' directly. For example:

https://cloud.tencent.com/product/cdwdoris

Justin Mclean  于2021年4月21日周三 上午10:28写道:

> Hi,
>
> -1 (binding) as not all issues from last RC have not been corrected in
> particular the NOTICE file
>
> I checked:
> - incubating in name
> - signature and hash is fine
> - LICENSE has been updated and files mentioned in last RC before removed
> - NOTICE is still not correct. The notice file is not for listing
> included/bundled 3rd party software. The notice file should include any
> relocated copyright files, (usually from software grants) and parts of
> notice files from Apache licensed bundled software.
> - some files are missing ASF headers [4][5][6]
> - No unexpected binary files
> - I didn't compile from source
>
>
> I can also see:
> - There are still trademark/ branding issues with [1]
> - It looks like this still contains unreleased code [1]
> - This has still not been corrected [2] I can read the original but the
> translated text is quite concerning for a number of reasons "As the main
> maintenance team of the Apache Doris community, the Baidu Doris team also
> maintains a 3-bit iterative version (tags) based on the official Release
> version of Apache Doris. Including quick bug fixes and new feature updates.”
> - I also see mention of a product named "Baidu Data Warehouse Doris” this
> would not be allowed under ASF trademark policy.
> - I also see this domain name "http://doris.baidu.com/“ being used, This
> is not allowed under ASF trademark policy.
> - This page [3] is in serious violation of ASF trademark and branding
> policy. Again I cannot read the original but claiming it's the official
> website Baidu Doris for starters is unacceptable, as is the use of Doris
> logo on the page, not respecting ASF trademark etc etc IMO This site need
> to be corrected or removed ASAP.
>
> Thanks,
> Justin
>
>
>
> 1. https://hub.docker.com/u/apachedoris
> 2.
> https://m4jh7kuq4qkiaojsq5ytf2qzvi-ac4c6men2g7xr2a-blog-csdn-net.translate.goog/ucanuup_/article/details/115548997
> 3. http://doris.baidu.com/home
> 4. ./fe_plugins/pom.xml
> 5. ./fe_plugins/auditdemo/pom.xml
> 6. ./fe_plugins/auditloader/pom.xml
>
>
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>


Re: [VOTE] Release Apache Doris 0.14.0-incubating-rc05

2021-04-20 Thread Justin Mclean
Hi,

-1 (binding) as not all issues from last RC have not been corrected in 
particular the NOTICE file

I checked:
- incubating in name
- signature and hash is fine
- LICENSE has been updated and files mentioned in last RC before removed
- NOTICE is still not correct. The notice file is not for listing 
included/bundled 3rd party software. The notice file should include any 
relocated copyright files, (usually from software grants) and parts of notice 
files from Apache licensed bundled software.
- some files are missing ASF headers [4][5][6]
- No unexpected binary files
- I didn't compile from source


I can also see:
- There are still trademark/ branding issues with [1]
- It looks like this still contains unreleased code [1]
- This has still not been corrected [2] I can read the original but the 
translated text is quite concerning for a number of reasons "As the main 
maintenance team of the Apache Doris community, the Baidu Doris team also 
maintains a 3-bit iterative version (tags) based on the official Release 
version of Apache Doris. Including quick bug fixes and new feature updates.”
- I also see mention of a product named "Baidu Data Warehouse Doris” this would 
not be allowed under ASF trademark policy.
- I also see this domain name "http://doris.baidu.com/“ being used, This is not 
allowed under ASF trademark policy.
- This page [3] is in serious violation of ASF trademark and branding policy. 
Again I cannot read the original but claiming it's the official website Baidu 
Doris for starters is unacceptable, as is the use of Doris logo on the page, 
not respecting ASF trademark etc etc IMO This site need to be corrected or 
removed ASAP.

Thanks,
Justin



1. https://hub.docker.com/u/apachedoris
2. 
https://m4jh7kuq4qkiaojsq5ytf2qzvi-ac4c6men2g7xr2a-blog-csdn-net.translate.goog/ucanuup_/article/details/115548997
3. http://doris.baidu.com/home
4. ./fe_plugins/pom.xml
5. ./fe_plugins/auditdemo/pom.xml
6. ./fe_plugins/auditloader/pom.xml


-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: [VOTE] Release Apache Doris 0.14.0-incubating-rc05

2021-04-20 Thread Justin Mclean
Hi,

> The release candidate has been tagged in GitHub as 0.14.0-rc05, available
> here:
> https://github.com/apache/incubator-doris/releases/tag/0.14.0-rc05

Doing this is a little problematic, as you are advertising a uni-voted on 
release to the general public. Please see [1][2]

Thanks,
Justin

1. https://incubator.apache.org/guides/distribution.html
2. https://infra.apache.org/release-distribution.html#unreleased



-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread Justin Mclean
Hi,

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

If you had a WIP-DISCLAIMER and listed this issue in it you could make a 
release.

I wold contact Airflow and have a chat to them and see why it needed. 

BTW I’m not 100% sure it is an Airflow dependancy, but I’m not 100% familiar 
with how python pip tools work.

pipdeptree for me gives:

pipreqs==0.4.10
  - docopt [required: Any, installed: 0.6.2]
  - yarg [required: Any, installed: 0.1.9]
- requests [required: Any, installed: 2.25.1]
  - certifi [required: >=2017.4.17, installed: 2020.4.5.1]
  - chardet [required: >=3.0.2,<5, installed: 4.0.0]
  - idna [required: >=2.5,<3, installed: 2.10]
  - urllib3 [required: >=1.21.1,<1.27, installed: 1.26.2]

pipdeptree --reverse --packages chardet gives:

chardet==4.0.0 
  - requests==2.25.1 [requires: chardet>=3.0.2,<5]
- yarg==0.1.9 [requires: requests]
  - pipreqs==0.4.10 [requires: yarg]

Which is different to what you have.

 Pipregs is just for producing requirements.txt right? So it may not be a 
runtime dependancy and be considered optional?

From  running  pip-licenses Yargs is MIT, requests is ALv2 and chardet is 
LGPLv2.

Thanks,
Justin


-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: [VOTE] Release Apache Doris 0.14.0-incubating-rc05

2021-04-20 Thread Duo Zhang
+1 (binding)

Checked sigs and sums: Matched
DISCLAIMER: In place
LICENSE and NOTICE: In place
Rat check: Passed with the .rat-excludes provided

Tried to build doris with docker but failed to build the docker image, when
installing dependencies for building gcc. Will try later.

Thanks.

陈明雨  于2021年4月20日周二 下午10:12写道:

> Dear IPMC:
> Please vote on Apache Doris 0.14.0-incubating-rc05 release
>
>
>
> --
>
> 此致!Best Regards
> 陈明雨 Mingyu Chen
>
> Email:
> chenmin...@apache.org
>
>
>
>
>
> At 2021-04-19 15:41:21, "寒江雪"  wrote:
> >Hi all,
> >
> >Please review and vote on Apache Doris 0.14.0-incubating-rc05 release.
> >
> >Apache Doris is an MPP-based interactive SQL data warehousing for
> reporting
> >and analysis.
> >
> >The Apache Doris community has voted on and approved this release:
> >
> https://lists.apache.org/thread.html/rea65ea7ab0e4cc68553bd4eb5c3602a9f6d73bd8765e2177bb073840%40%3Cdev.doris.apache.org%3E
> >
> >The vote result email thread:
> >
> https://lists.apache.org/thread.html/rb9bfbf92c445a6de19c5cc5f186bcf15f3f6a73b3609f13a2a9eabd5%40%3Cdev.doris.apache.org%3E
> >
> >The release candidate has been tagged in GitHub as 0.14.0-rc05, available
> >here:
> >https://github.com/apache/incubator-doris/releases/tag/0.14.0-rc05
> >
> >Thanks to everyone who has contributed to this release, and there is a
> >simple release notes can be found here:
> >https://github.com/apache/incubator-doris/issues/5374
> >
> >The artifacts (source, signature and checksum) corresponding to this
> release
> >candidate can be found here:
> >https://dist.apache.org/repos/dist/dev/incubator/doris/0.14/0.14.0-rc05
> >
> >This has been signed with PGP key 59F2ACED, corresponding to
> >yang...@apache.org.
> >KEYS file is available here:
> >https://dist.apache.org/repos/dist/dev/incubator/doris/KEYS
> >It is also listed here:
> >https://people.apache.org/keys/committer/yangzhg.asc
> >
> >The vote will be open for at least 72 hours.
> >[ ] +1 Approve the release
> >[ ] +0 No opinion
> >[ ] -1 Do not release this package because ...
> >
> >To verify and build, you can refer to the following wiki:
> >
> https://github.com/apache/incubator-doris/wiki/How-to-verify-Apache-Release
> >https://wiki.apache.org/incubator/Incubator+Release+Checklist
> >
> >You can also refer to instruction:
> >
> >Firstly, you must install and start docker service, and then you could
> >build Doris as the following steps:
> >
> >Step1: Pull the docker image with Doris building environment
> >$ docker pull apachedoris/doris-dev:build-env-1.2
> >You can check it by listing images, its IMAGE ID is d854b488afaf
> >
> >Step2: Run the Docker image
> >You can run the image directly:
> >$ docker run -it apachedoris/doris-dev:build-env1.2
> >
> >Step3: Download Doris source
> >Now you should in the docker environment, and you can download the Doris
> >source package.
> >(If you have downloaded the source and it is not in image, you can map its
> >path to image in Step2.)
> >$ wget
> >
> https://dist.apache.org/repos/dist/dev/incubator/doris/0.14/0.14.0-rc05/apache-doris-0.14.0-incubating-src.tar.gz
> >
> >Step4: Build Doris
> >Now you can decompress and enter Doris source path and build Doris.
> >$ tar -zxvf apache-doris-0.14.0-incubating-src.tar.gz
> >$ cd apache-doris-0.14.0-incubating-src
> >$ sh build.sh
> >
> >Best Regards,
> >Yang Zhengguo
>


Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread Jean-Baptiste Onofre
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  a écrit :
> 
> LGPL is only allowed in optional dependencies.
> 
> On Tue, Apr 20, 2021 at 08:31 Lior Schachter  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é  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  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 
> 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  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



Re:[VOTE] Release Apache Doris 0.14.0-incubating-rc05

2021-04-20 Thread 陈明雨
Dear IPMC:
Please vote on Apache Doris 0.14.0-incubating-rc05 release



--

此致!Best Regards
陈明雨 Mingyu Chen

Email:
chenmin...@apache.org





At 2021-04-19 15:41:21, "寒江雪"  wrote:
>Hi all,
>
>Please review and vote on Apache Doris 0.14.0-incubating-rc05 release.
>
>Apache Doris is an MPP-based interactive SQL data warehousing for reporting
>and analysis.
>
>The Apache Doris community has voted on and approved this release:
>https://lists.apache.org/thread.html/rea65ea7ab0e4cc68553bd4eb5c3602a9f6d73bd8765e2177bb073840%40%3Cdev.doris.apache.org%3E
>
>The vote result email thread:
>https://lists.apache.org/thread.html/rb9bfbf92c445a6de19c5cc5f186bcf15f3f6a73b3609f13a2a9eabd5%40%3Cdev.doris.apache.org%3E
>
>The release candidate has been tagged in GitHub as 0.14.0-rc05, available
>here:
>https://github.com/apache/incubator-doris/releases/tag/0.14.0-rc05
>
>Thanks to everyone who has contributed to this release, and there is a
>simple release notes can be found here:
>https://github.com/apache/incubator-doris/issues/5374
>
>The artifacts (source, signature and checksum) corresponding to this release
>candidate can be found here:
>https://dist.apache.org/repos/dist/dev/incubator/doris/0.14/0.14.0-rc05
>
>This has been signed with PGP key 59F2ACED, corresponding to
>yang...@apache.org.
>KEYS file is available here:
>https://dist.apache.org/repos/dist/dev/incubator/doris/KEYS
>It is also listed here:
>https://people.apache.org/keys/committer/yangzhg.asc
>
>The vote will be open for at least 72 hours.
>[ ] +1 Approve the release
>[ ] +0 No opinion
>[ ] -1 Do not release this package because ...
>
>To verify and build, you can refer to the following wiki:
>https://github.com/apache/incubator-doris/wiki/How-to-verify-Apache-Release
>https://wiki.apache.org/incubator/Incubator+Release+Checklist
>
>You can also refer to instruction:
>
>Firstly, you must install and start docker service, and then you could
>build Doris as the following steps:
>
>Step1: Pull the docker image with Doris building environment
>$ docker pull apachedoris/doris-dev:build-env-1.2
>You can check it by listing images, its IMAGE ID is d854b488afaf
>
>Step2: Run the Docker image
>You can run the image directly:
>$ docker run -it apachedoris/doris-dev:build-env1.2
>
>Step3: Download Doris source
>Now you should in the docker environment, and you can download the Doris
>source package.
>(If you have downloaded the source and it is not in image, you can map its
>path to image in Step2.)
>$ wget
>https://dist.apache.org/repos/dist/dev/incubator/doris/0.14/0.14.0-rc05/apache-doris-0.14.0-incubating-src.tar.gz
>
>Step4: Build Doris
>Now you can decompress and enter Doris source path and build Doris.
>$ tar -zxvf apache-doris-0.14.0-incubating-src.tar.gz
>$ cd apache-doris-0.14.0-incubating-src
>$ sh build.sh
>
>Best Regards,
>Yang Zhengguo


Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread John D. Ament
Seems like a good first step if not already done is to reach out to
airflow.

On Tue, Apr 20, 2021 at 09:30 Lior Schachter  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é  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  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 
> > >> 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  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
> >
> >
>


Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread Matt Sicker
LGPL is only allowed in optional dependencies.

On Tue, Apr 20, 2021 at 08:31 Lior Schachter  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é  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  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 
> > >> 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  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
> >
> >
>


Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread Lior Schachter
We can't just remove this dependency as it comes from the Airflow package
deoendencies.

On Tue, Apr 20, 2021, 15:51 JB Onofré  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  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 
> >> 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  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
>
>


Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread JB Onofré
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  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 
>> 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  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



Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread Lior Schachter
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 
> 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  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
> >
> >
>


Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread Hans Van Akelyen
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  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  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
>
>


Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread Justin Mclean
Hi

> Good catch. AFAIR chardet is LGPL based (not GPL), and LGPL is Cat B (not X).

LGPL is Category X not B. [1]

"Places restrictions on larger works:
• GNU GPL 1, 2, 3
• Special exceptions to the GNU GPL (e.g. GNU Classpath) unless 
otherwise permitted elsewhere on this page. 
• GNU Affero GPL 3
• GNU LGPL 2, 2.1, 3"

If it was category B it would be OK as a dependancy.

Thanks,
Justin

1. https://www.apache.org/legal/resolved.html#category-x
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread Jean-Baptiste Onofre
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  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



Re: [VOTE] Release Apache Liminal (incubating) 0.0.2rc1-INCUBATING

2021-04-20 Thread Justin Mclean
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