Re: [VOTE] Release of Apache Weex (incubating) 0.12.0-RC0

2017-04-16 Thread sospartan
I've review all dependencies ( javascript/iOS/android), and update the
NOTICE file
https://git-wip-us.apache.org/repos/asf?p=incubator-weex.git;a=blobdiff;f=NOTICE;h=7b45898e5cc193fa94f056139a43b9d343cdeec3;hp=21e0ee8b937f2653967b2311ae188227f53fa222;hb=b342eca1e6edd1cb66114794497cdcfead1c2b34;hpb=5ea996684b99c64456a480cfb7d02e3cd1006988

There are 31 of them. Most of it is used by Playground application.
And many of them ( 8 at least) is maintained by weex team, I'd like make a
proposal to move these project into weex-incubating project.



On Mon, Apr 17, 2017 at 11:28 AM, sospartan  wrote:

> My pleasure to help that.
>
> I've make some changes to include test code and xml file
> https://github.com/apache/incubator-weex/commit/
> 951f65492f2b37cd90bbac9fd0c30ba5866cc189
>
> And I create a script under 'scripts' folder, you can run
> './scripts/rat-scan.sh' to validate our src-header, the exclude rule is in
> ‘scripts/rat-ant-build.xml'.
>
> I'll call another vote (for RC1) after I fix the 'NOTICE' file with
> dependencies.
>
>
> On Sat, Apr 15, 2017 at 9:30 AM, Niclas Hedhman 
> wrote:
>
>> Ok, thanks for taking time to answer my concerns. I withdraw the veto (the
>> GPL concern).
>> I recommend cutting a new release with the changes made, but will not
>> block
>> if you decide to push this release out anyway. Repacking should be quick
>> and automatic...
>>
>> Cheers
>>
>> On Sat, Apr 15, 2017 at 9:03 AM, sospartan  wrote:
>>
>> > These weex-* project is removable, theoretically speaking.  We put
>> these in
>> > dependency for building javascript library more
>> > convenient to use. It's kind of like a  all-in-one javascript template
>> > framework.
>> > If you suggest moving these project to weex incubating project too, I
>> think
>> > we can discuss that, more things need to be settle. But for the next
>> > release I'll suggest keep current way.
>> >
>> > The 'animationjs' we use from here https://www.npmjs.com/package/
>> > animationjs
>> >
>> > flow-types; I mean these file can be generated by 'flow-type' tool, were
>> > not manual code by us. For saving the time build it, we put it in code
>> repo
>> > in advance.
>> > @Danz can we remove these files? Put it in .gitignore then you can keep
>> use
>> > these file in you local environment.
>> >
>> >
>> >
>> > On Fri, Apr 14, 2017 at 4:54 PM, Niclas Hedhman 
>> > wrote:
>> >
>> > > Well, the weexteam projects are listed as direct dependencies...
>> > >
>> > > "dependencies": {
>> > > "animationjs": "^0.1.5",
>> > > "core-js": "^2.4.0",
>> > > "cubicbezier": "^0.1.1",
>> > > "envd": "^0.1.1",
>> > > "httpurl": "^0.1.1",
>> > > "lazyimg": "^0.1.5",
>> > > "modals": "^0.1.6",
>> > > "query-string": "^4.2.3",
>> > > "scroll-to": "0.0.2",
>> > > "semver": "^5.1.0",
>> > > "weex-components": "^0.2.0",
>> > > "weex-picker": "^0.1.0",
>> > > "weex-rax-framework": "0.1.7",
>> > > "weex-styler":"0.1.8",
>> > > "weex-vue-framework": "2.1.8-weex.1"
>> > > },
>> > >
>> > > So, sorry if I don't see that as "not core parts of weex but some
>> > > derivative projects."
>> > >
>> > > animationJs ; my apologies. What showed up in my google search was
>> > > https://github.com/lathoub/AnimationJS.
>> > >
>> > > flow-types; Not sure what you mean. Please elaborate.
>> > >
>> > >
>> > > On Fri, Apr 14, 2017 at 3:24 PM, sospartan 
>> wrote:
>> > >
>> > > > I'll check every dependency, includes npm dependencies in
>> package.json/
>> > > > dependencies import by android's gradle or dependencies  imported by
>> > > ios's
>> > > > Pod, and update the NOTICE file.
>> > > >
>> > > > The weex-* project are side project maintain by weex or related
>> teams.
>> > > I'll
>> > > > check licenses of these projects. These projects are not core parts
>> of
>> > > weex
>> > > > but some derivative projects.
>> > > >
>> > > > I'll update other file type like XML/Shell scripts to have a proper
>> src
>> > > > header.
>> > > >
>> > > > Files under 'flow-typed/' are generated to help run lint against js
>> > code.
>> > > > It's not 'source code'. Including js file under android and ios
>> SDK's
>> > > > assets (android/sdk/assets,etc.)
>> > > > @Niclas How do we deal these generate file?
>> > > >
>> > > > 'animationjs' is a library maintained by Danz He. I'm notice this
>> > library
>> > > > is MIT, not GPL.
>> > > >
>> > > >
>> > > > On Fri, Apr 14, 2017 at 10:48 AM, Niclas Hedhman <
>> nic...@hedhman.org>
>> > > > wrote:
>> > > >
>> > > > > -1 ( VETO - dependency on GPL code)
>> > > > >
>> > > > >
>> > > > > The NOTICE file contains notices about 5 dependencies, but the
>> list
>> > of
>> > > > > dependencies in package.json is much bigger, and I would assume
>> that
>> > > most
>> > > > > of them are released under a license that requires attribution.
>> > > > >
>> > > > > Also, what is the relationship with the weex-* dependencies?
>> Looking
>> > at
>> > > > > weex-picker, it doesn't seem to be released with the same rigor
>

[GitHub] incubator-weex pull request #266: [weex-html5] weex-vue-render add slider-ne...

2017-04-16 Thread MrRaindrop
GitHub user MrRaindrop opened a pull request:

https://github.com/apache/incubator-weex/pull/266

[weex-html5] weex-vue-render add slider-neighbor & bugfix.

This PR has follows changed on weex-vue-render:

* support 'slider-neighbor' component (already supported by native renders).
* add gestures events, including 'panstart', 'panmove', 'panend', 'swipe', 
'longpress'.
* fix events bubble error.
* fix style merging error.
* fix appear bug.
* add createEventMap method to global.weex.
* fix transition defaults (duration by 0 and timingFunction by linear)
* npm release pkg contains no src folder. Change main path to the 
dist/index.js instead.
* add test cases.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/MrRaindrop/incubator-weex 0.12-dev

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/266.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #266


commit 01e3a574619f218a1da60fa501d3f4bbd14fff7e
Author: MrRaindrop 
Date:   2017-03-31T09:37:36Z

* [html5] add slider-neighbor

commit d3c3fc4db5dd90b169552ea7f2ba6340cac547bc
Author: MrRaindrop 
Date:   2017-04-06T08:52:28Z

Merge branch 'dev-web-scale' into dev-web-slider-neighbor

commit 65754c47d76d6bd8e8233e7607286ea350b8d01b
Author: MrRaindrop 
Date:   2017-04-10T02:27:25Z

Merge branch 'dev-web-scale' into dev-web-slider-neighbor

commit 09cfa4e1b1170215e374705f2b17d1878776ca9a
Author: MrRaindrop 
Date:   2017-04-10T07:47:14Z

+ [html5] add slider-neighbor.

commit 716a951a66288e839a26548ce8de41cc8335e204
Author: MrRaindrop 
Date:   2017-04-10T08:22:24Z

Merge branch 'dev-web-scale' into dev-web-slider-neighbor

commit 1a53bcdf1fbe3657a076b51ea627343f2ecc0038
Author: MrRaindrop 
Date:   2017-04-10T10:03:09Z

Merge branch 'dev-web-scale' into dev-web-slider-neighbor

commit 122b90e9a4bc8c7efdc859391f05066f291275e4
Author: MrRaindrop 
Date:   2017-04-11T07:24:19Z

+ [html5] add gesture.

commit bf99deb46ae2e54c51d9c0b6cbf70e67374e91ad
Author: MrRaindrop 
Date:   2017-04-13T03:46:27Z

Merge branch 'dev-web-scale' into 0.12-dev-web

commit b0404965abf2d4873ed9963ca52ff8ee31bc0f53
Author: MrRaindrop 
Date:   2017-04-13T03:49:34Z

Merge branch 'dev-web-slider-neighbor' into 0.12-dev-web

commit ca4f1997d2ff0677dd942b88724cd9095d06948e
Author: MrRaindrop 
Date:   2017-04-13T09:37:11Z

* [html5] fix appear events when components's v-if is false, & fix appear 
events in update hooks.

commit a303f4fab7ceaf76c00d15026889914eac624c57
Author: MrRaindrop 
Date:   2017-04-13T09:38:13Z

* [html5] fix events bubble error.

commit 8dd79bd7577b76118ad3b9bd3ddf0d94c2f08453
Author: MrRaindrop 
Date:   2017-04-14T03:24:25Z

* [html5] fix style merging.

commit 8e0534589a76e75132075c43b675be179c480740
Author: MrRaindrop 
Date:   2017-04-14T03:52:44Z

+ [html5] add web platform only event: firstscreenfinish.

commit eb0d6cf5b511c5ddc4b6b1724d72f8f5331375cf
Author: MrRaindrop 
Date:   2017-04-14T04:54:30Z

* [html5] ignore build file.

commit 6e08521a2dd544fbd05b51f1866689956bd26b53
Author: MrRaindrop 
Date:   2017-04-14T05:51:54Z

* [html5] udpate pkg main.

commit 5215298630922e3d92f1852ad40926657631a5bd
Author: MrRaindrop 
Date:   2017-04-14T05:52:15Z

* [html5] rm src from dist pkg.

commit a68dab16dd953a5979bfccbbf6eafbef90737d04
Author: MrRaindrop 
Date:   2017-04-14T05:53:21Z

* [html5] release 0.11.6

commit 06b444b8642214d4177a3ccc06f12985ffa1b995
Author: MrRaindrop 
Date:   2017-04-14T05:56:01Z

Merge branch '0.12-dev' into 0.12-dev-web

commit f96ad2745f3a8048f1e46a70f9ac5b0859756b05
Author: MrRaindrop 
Date:   2017-04-14T09:12:43Z

* [html5] fix event bubbles.

commit 57afc02c41f4c6d2294c7110dc0ba4a667677269
Author: MrRaindrop 
Date:   2017-04-14T09:15:37Z

* [html5] release 0.11.7

commit a93240c9bbbc85d1e8090110caa3e3266b7f7510
Author: MrRaindrop 
Date:   2017-04-14T09:21:48Z

* [html5] fix lint.

commit 7932b237b39c38e09f4e613d711f28ee75322ff8
Author: MrRaindrop 
Date:   2017-04-14T09:59:47Z

* [html5] add global event createEventMap.

commit eb79187ced7ecf314900bb9f11e6c1a9dc17cb2f
Author: MrRaindrop 
Date:   2017-04-17T03:13:11Z

* [html5] fix transition defaults.

commit 01d5ee2e58cde882d93583e0b784a52968ab2702
Author: MrRaindrop 
Date:   2017-04-17T03:34:26Z

* [html5] rm test vue libs.

commit ae9b7a3e0053289b655a4a588e31de0c8eaa491d
Author: MrRaindrop 
Date:   2017-04-17T03:47:25Z

Merge branch '0.12-dev' into 0.12-dev-web




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure

Re: [VOTE] Release of Apache Weex (incubating) 0.12.0-RC0

2017-04-16 Thread sospartan
My pleasure to help that.

I've make some changes to include test code and xml file
https://github.com/apache/incubator-weex/commit/951f65492f2b37cd90bbac9fd0c30ba5866cc189

And I create a script under 'scripts' folder, you can run
'./scripts/rat-scan.sh' to validate our src-header, the exclude rule is in
‘scripts/rat-ant-build.xml'.

I'll call another vote (for RC1) after I fix the 'NOTICE' file with
dependencies.


On Sat, Apr 15, 2017 at 9:30 AM, Niclas Hedhman  wrote:

> Ok, thanks for taking time to answer my concerns. I withdraw the veto (the
> GPL concern).
> I recommend cutting a new release with the changes made, but will not block
> if you decide to push this release out anyway. Repacking should be quick
> and automatic...
>
> Cheers
>
> On Sat, Apr 15, 2017 at 9:03 AM, sospartan  wrote:
>
> > These weex-* project is removable, theoretically speaking.  We put these
> in
> > dependency for building javascript library more
> > convenient to use. It's kind of like a  all-in-one javascript template
> > framework.
> > If you suggest moving these project to weex incubating project too, I
> think
> > we can discuss that, more things need to be settle. But for the next
> > release I'll suggest keep current way.
> >
> > The 'animationjs' we use from here https://www.npmjs.com/package/
> > animationjs
> >
> > flow-types; I mean these file can be generated by 'flow-type' tool, were
> > not manual code by us. For saving the time build it, we put it in code
> repo
> > in advance.
> > @Danz can we remove these files? Put it in .gitignore then you can keep
> use
> > these file in you local environment.
> >
> >
> >
> > On Fri, Apr 14, 2017 at 4:54 PM, Niclas Hedhman 
> > wrote:
> >
> > > Well, the weexteam projects are listed as direct dependencies...
> > >
> > > "dependencies": {
> > > "animationjs": "^0.1.5",
> > > "core-js": "^2.4.0",
> > > "cubicbezier": "^0.1.1",
> > > "envd": "^0.1.1",
> > > "httpurl": "^0.1.1",
> > > "lazyimg": "^0.1.5",
> > > "modals": "^0.1.6",
> > > "query-string": "^4.2.3",
> > > "scroll-to": "0.0.2",
> > > "semver": "^5.1.0",
> > > "weex-components": "^0.2.0",
> > > "weex-picker": "^0.1.0",
> > > "weex-rax-framework": "0.1.7",
> > > "weex-styler":"0.1.8",
> > > "weex-vue-framework": "2.1.8-weex.1"
> > > },
> > >
> > > So, sorry if I don't see that as "not core parts of weex but some
> > > derivative projects."
> > >
> > > animationJs ; my apologies. What showed up in my google search was
> > > https://github.com/lathoub/AnimationJS.
> > >
> > > flow-types; Not sure what you mean. Please elaborate.
> > >
> > >
> > > On Fri, Apr 14, 2017 at 3:24 PM, sospartan 
> wrote:
> > >
> > > > I'll check every dependency, includes npm dependencies in
> package.json/
> > > > dependencies import by android's gradle or dependencies  imported by
> > > ios's
> > > > Pod, and update the NOTICE file.
> > > >
> > > > The weex-* project are side project maintain by weex or related
> teams.
> > > I'll
> > > > check licenses of these projects. These projects are not core parts
> of
> > > weex
> > > > but some derivative projects.
> > > >
> > > > I'll update other file type like XML/Shell scripts to have a proper
> src
> > > > header.
> > > >
> > > > Files under 'flow-typed/' are generated to help run lint against js
> > code.
> > > > It's not 'source code'. Including js file under android and ios SDK's
> > > > assets (android/sdk/assets,etc.)
> > > > @Niclas How do we deal these generate file?
> > > >
> > > > 'animationjs' is a library maintained by Danz He. I'm notice this
> > library
> > > > is MIT, not GPL.
> > > >
> > > >
> > > > On Fri, Apr 14, 2017 at 10:48 AM, Niclas Hedhman  >
> > > > wrote:
> > > >
> > > > > -1 ( VETO - dependency on GPL code)
> > > > >
> > > > >
> > > > > The NOTICE file contains notices about 5 dependencies, but the list
> > of
> > > > > dependencies in package.json is much bigger, and I would assume
> that
> > > most
> > > > > of them are released under a license that requires attribution.
> > > > >
> > > > > Also, what is the relationship with the weex-* dependencies?
> Looking
> > at
> > > > > weex-picker, it doesn't seem to be released with the same rigor as
> is
> > > > > expected from Apache projects, such as not stating the license
> terms
> > > and
> > > > > not respecting notices of dependencies.
> > > > >
> > > > > Why are these projects not part of Apache Weex (incubating)?
> > > > >
> > > > > Furthermore, it is not only .java files that are expected to have
> the
> > > > > license headers. We expect that in every source file that allows
> for
> > > > > comments. And it is missing all over the place, such as
> javascripts,
> > > XML
> > > > > files, build files, shell script files and so on.
> > > > >
> > > > > In the incubator-weex-1f7ed88/flow-typed/npm directory, which is
> > some
> > > > type
> > > > > of "replacements" for other things. What is this, because some of
> > those
> > > > > stubs are for GPL'd

[GitHub] incubator-weex issue #260: Wrong a word in the documentation

2017-04-16 Thread sospartan
Github user sospartan commented on the issue:

https://github.com/apache/incubator-weex/pull/260
  
@toxichl sorry the CI is broken, I'll fix that soon.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-weex issue #260: Wrong a word in the documentation

2017-04-16 Thread toxichl
Github user toxichl commented on the issue:

https://github.com/apache/incubator-weex/pull/260
  
Why still can not pass, I just modified a word?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-weex pull request #195: fix method onActivityResult bug,resultCode...

2017-04-16 Thread lltemplar
Github user lltemplar closed the pull request at:

https://github.com/apache/incubator-weex/pull/195


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---