Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-30 Thread SHUANG SU
Thanks all for the comments and discussions! The voting for the release of Apache ECharts 4.1.0 (release candidate 3) is closed. And a new voting for the release of Apache ECharts 4.1.0 (release candidate 4) has been initialized in another thread: https://lists.apache.org/thread.html/4a551cd183a7e

Re: Minified Javascript in source releases (was Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3)

2018-05-23 Thread Bertrand Delacretaz
On Wed, May 23, 2018 at 6:16 AM, Greg Stein wrote: > On Mon, May 21, 2018 at 2:52 AM Justin Mclean > wrote: >>... Do people think it OK to include minified JS in a source release if: >> 1. It's ASF developed code and the full unminified source code is included >> as well. >> > Absolutely... Same

Re: Minified Javascript in source releases (was Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3)

2018-05-22 Thread Christopher
On Wed, May 23, 2018 at 12:16 AM Greg Stein wrote: > On Mon, May 21, 2018 at 2:52 AM Justin Mclean > wrote: > > > Hi, > > > > > Javascript code that is minified or combined in any major way is much > > more > > > like binary code in that respect. It is true that somebody *could* > > inspect > >

Re: Minified Javascript in source releases (was Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3)

2018-05-22 Thread Greg Stein
On Mon, May 21, 2018 at 2:52 AM Justin Mclean wrote: > Hi, > > > Javascript code that is minified or combined in any major way is much > more > > like binary code in that respect. It is true that somebody *could* > inspect > > the correlation, but it is not true that this inspection is either > n

Re: Minified Javascript in source releases (was Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3)

2018-05-22 Thread Hen
On Mon, May 21, 2018 at 4:22 PM, Ted Dunning wrote: > On Mon, May 21, 2018, 21:12 Justin Mclean > wrote: > > > Hi, > > > > > Why does this need to be included at all? Why not just provide a > pointer > > to > > > the canonical minified version? > > > > Most common occurrence (off the top of my h

Re: Minified Javascript in source releases (was Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3)

2018-05-21 Thread Ted Dunning
On Mon, May 21, 2018, 21:12 Justin Mclean wrote: > Hi, > > > Why does this need to be included at all? Why not just provide a pointer > to > > the canonical minified version? > > Most common occurrence (off the top of my head) is a minified version of > bootstrap for project site / documentations

Re: Minified Javascript in source releases (was Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3)

2018-05-21 Thread Justin Mclean
Hi, > Why does this need to be included at all? Why not just provide a pointer to > the canonical minified version? Most common occurrence (off the top of my head) is a minified version of bootstrap for project site / documentations. So your view is that that shouldn’t be included in a source r

Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-21 Thread SHUANG SU
Mike and Ted, thanks a lot for your detailed explanation! I've been gradually understanding the way that the community thinks about the "release". And I will fix the artifact soon. Thanks, Shuang -- Su Shuang (100pah) -- 2018-05-21 14:

Re: Minified Javascript in source releases (was Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3)

2018-05-21 Thread Ted Dunning
On Mon, May 21, 2018 at 12:52 AM, Justin Mclean wrote: > Hi, > > > Javascript code that is minified or combined in any major way is much > more > > like binary code in that respect. It is true that somebody *could* > inspect > > the correlation, but it is not true that this inspection is either >

Minified Javascript in source releases (was Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3)

2018-05-21 Thread Justin Mclean
Hi, > Javascript code that is minified or combined in any major way is much more > like binary code in that respect. It is true that somebody *could* inspect > the correlation, but it is not true that this inspection is either normally > done or easily done. Thanks Ted I’ve not thought of it in t

Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-20 Thread Ted Dunning
The general meaning of source code is that it is the artifact that people will edit and which they can inspect by normal textual or graphical means to ensure that there are no surprises. Javascript code that is minified or combined in any major way is much more like binary code in that respect. It

Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-20 Thread Mike Jumper
On Sun, May 20, 2018 at 10:07 PM, SHUANG SU wrote: > Thanks, Justin, > > I think I should remove the jar about rat from the artifact, and then > there is no binary code anymore. > > But I am puzzled about the definition of the term "compiled code". > Generally, the JavaScript code does not need t

Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-20 Thread SHUANG SU
Thanks, Justin, I think I should remove the jar about rat from the artifact, and then there is no binary code anymore. But I am puzzled about the definition of the term "compiled code". Generally, the JavaScript code does not need to be compiled to binary. The code in "dist/**" is also JavaScript

Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-20 Thread Justin Mclean
Hi, Releases at the ASF must not contain compiled code. You can if you want also produce a conviance binary for users at the same time but the source release needs to contain no compiled code otherwise it's not open source. Thanks, Justin On Mon., 21 May 2018, 7:10 am SHUANG SU, wrote: > Thank

Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-20 Thread SHUANG SU
Thanks, Willem. But I will explain the reason that provides an all-in-one artifact. I understand that one of the reasons for separating src and binary files is that in some project the compilation is depending on the target runtime environment and thus the products cannot be enumerated completely

Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-20 Thread Kevin A. McGrail
If the release candidate isn't correct for the artifacts you need to roll an rc4 which might be two files not one and send that for a vote. On Sat, May 19, 2018, 22:34 Willem Jiang wrote: > Hi, > > As there is only one zip file, I guess it just the src distribution. > But after went through the

Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-19 Thread Willem Jiang
Hi, As there is only one zip file, I guess it just the src distribution. But after went through the file, I found lot of echart js files in the dist directory and the rat jar. So I'm confused what's the purpose of apache-echarts-4.1.0.rc3- incubating.zip Normally we distribute the src and binary

Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-18 Thread SHUANG SU
John, (1) The included jar file of license check tool (Rat) is for developers' convenience. Basically, the infrastructure about building, checking and releasing of the ECharts is based on Node.js, thus it is not convenient for the developers and users to find and install Java-based tool (Rat) to c

Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-17 Thread John D . Ament
Shuang, Apologies for my late reply. Only issue I see is the inclusion of compiled code in the source release. It's generally expected to not have class files, jars in the source release. We have good faith on what is included, but no way to confirm. Also, since you now include RAT in the d

Re: [VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-17 Thread SHUANG SU
BTW: >From the Apache Echarts (incubating) 4.1.0.rc1 to 4.1.0.rc3, the license of source code files has been added, including the licenses of the third-party libs, and the Apache Rat has been executed against the project. -- Su Shuang (100pah) ---

[VOTE] Release Apache ECharts (incubating) 4.1.0.rc3

2018-05-17 Thread SHUANG SU
I am pleased to be calling this vote for the release of Apache ECharts 4.1.0.rc3. Apache ECharts community has voted and approved the release. Vote thread: https://lists.apache.org/thread.html/67dffef28ecffd66689ac991ca027c0868d734629949d958e7b12dd3@%3Cdev.echarts.apache.org%3E Results thread: h