Re: [VOTE] Release Apache Sling Scripting HTL Compiler 1.0.0, Apache Sling Scripting HTL Java Compiler 1.0.0, Apache Sling Scripting HTL Engine 1.0.20, Apache Sling Scripting HTL JS Use Provider 1.0.1

2016-09-07 Thread Bertrand Delacretaz 
Hi,

On Mon, Sep 5, 2016 at 6:59 PM, Radu Cotescu  wrote:
> ...Staging repository:
> https://repository.apache.org/content/repositories/orgapachesling-1517 ...

+1 to release the artifacts listed below - checked signatures, builds
and that the svn tags match.

Although the renaming is infortunate, considering Roy's comment in the
"What's wrong with Sightly?" thread, the above comments indicating
that there should be no backwards compatibility issues, and
https://svn.apache.org/r1759438 which IIUC fixes the website issues I
think it's ok to proceed.

Artifacts:
SHA1(./htl-maven-plugin/1.0.0/htl-maven-plugin-1.0.0-source-release.zip)=
c6293c9f7b2aa819b3fe22b5d2c6fd8f7fa0f277
SHA1(./org.apache.sling.scripting.sightly/1.0.20/org.apache.sling.scripting.sightly-1.0.20-source-release.zip)=
7607cefd33ab699632dca8e5f839736f837f3f9b
SHA1(./org.apache.sling.scripting.sightly.compiler/1.0.0/org.apache.sling.scripting.sightly.compiler-1.0.0-source-release.zip)=
585b92cd0ea35ad33965db295524b175de1eb69a
SHA1(./org.apache.sling.scripting.sightly.compiler.java/1.0.0/org.apache.sling.scripting.sightly.compiler.java-1.0.0-source-release.zip)=
91dd1e79c51dad2e10c884c6111ca9809466122e
SHA1(./org.apache.sling.scripting.sightly.js.provider/1.0.12/org.apache.sling.scripting.sightly.js.provider-1.0.12-source-release.zip)=
ae8f1b2e68c9406cf727839942f47f6c9add2701
SHA1(./org.apache.sling.scripting.sightly.models.provider/1.0.2/org.apache.sling.scripting.sightly.models.provider-1.0.2-source-release.zip)=
6c8200ccea3d7a27590e730a7f1a5086cea26c51
SHA1(./org.apache.sling.scripting.sightly.repl/1.0.4/org.apache.sling.scripting.sightly.repl-1.0.4-source-release.zip)=
f4798a33cac848c60e7e12a4955d0f12f519095d

-Bertrand

Re: [VOTE] Release Apache Sling Scripting HTL Compiler 1.0.0, Apache Sling Scripting HTL Java Compiler 1.0.0, Apache Sling Scripting HTL Engine 1.0.20, Apache Sling Scripting HTL JS Use Provider 1.0.1

2016-09-07 Thread Robert Munteanu 
On Mon, 2016-09-05 at 16:59 +, Radu Cotescu wrote:
> Please vote to approve this release:

+1

Robert

signature.asc
Description: This is a digitally signed message part

Re: [VOTE] Release Apache Sling Scripting HTL Compiler 1.0.0, Apache Sling Scripting HTL Java Compiler 1.0.0, Apache Sling Scripting HTL Engine 1.0.20, Apache Sling Scripting HTL JS Use Provider 1.0.1

2016-09-06 Thread Radu Cotescu 
Hi,

I've just replied in the other thread - there are no backwards
compatibility issues introduced by the rename as this was mostly cosmetic:
I've only changed projects' names and descriptions. None of the tests were
changed for these releases.

Thanks,
Radu

On Tue, 6 Sep 2016 at 09:41 Bertrand Delacretaz 
wrote:

> Hi,
>
> On Mon, Sep 5, 2016 at 6:59 PM, Radu Cotescu  wrote:
> > ...1. the modularisation of the Sightly engine - now renamed to HTL...
>
> Before voting on this release I have concerns about the name change
> and related backwards compatibility issues, added my thoughts in the
> "What's wrong with Sightly?" thread here.
>
> -Bertrand
>

Re: [VOTE] Release Apache Sling Scripting HTL Compiler 1.0.0, Apache Sling Scripting HTL Java Compiler 1.0.0, Apache Sling Scripting HTL Engine 1.0.20, Apache Sling Scripting HTL JS Use Provider 1.0.1

2016-09-06 Thread Bertrand Delacretaz 
Hi,

On Mon, Sep 5, 2016 at 6:59 PM, Radu Cotescu  wrote:
> ...1. the modularisation of the Sightly engine - now renamed to HTL...

Before voting on this release I have concerns about the name change
and related backwards compatibility issues, added my thoughts in the
"What's wrong with Sightly?" thread here.

-Bertrand

Re: [VOTE] Release Apache Sling Scripting HTL Compiler 1.0.0, Apache Sling Scripting HTL Java Compiler 1.0.0, Apache Sling Scripting HTL Engine 1.0.20, Apache Sling Scripting HTL JS Use Provider 1.0.1

2016-09-05 Thread Radu Cotescu 
+1

On Mon, 5 Sep 2016 at 18:59 Radu Cotescu  wrote:

> Please vote to approve this release:
>
>   [ ] +1 Approve the release
>   [ ]  0 Don't care
>   [ ] -1 Don't release, because ...
>
> This majority vote is open for at least 72 hours.
>
> Cheers,
> Radu
>