Re: [VOTE] Release JMeter 2.5 RC1

On 1 August 2011 06:46, Stefan Bodewig <[email protected]> wrote:
> On 2011-08-01, Milamber wrote:
>
>> The first release candidate for JMeter 2.5 has been prepared, and your
>> votes are solicited.
>
> I haven't performed any sort of test, not even tried to build JMeter.
>
> * hashes and sigs are fine
>
> * source tarball and svn tag differ in some ways, some minor like no
>  checkstyle.xml in the source archive but at least one that might be
>  significant:

We ought to include checkstyle.xml in the source archive - I think
that must be new since 2.4.

> diff -ur v2_5_RC1/src/core/org/apache/jmeter/util/JMeterVersion.java jakarta-jmeter-2.5/src/core/org/apache/jmeter/util/JMeterVersion.java
> --- v2_5_RC1/src/core/org/apache/jmeter/util/JMeterVersion.java 2011-08-01 07:26:20.174097342 +0200
> +++ jakarta-jmeter-2.5/src/core/org/apache/jmeter/util/JMeterVersion.java       2011-08-01 01:18:26.000000000 +0200
> @@ -44,7 +44,7 @@
>      * This ensures that JMeterUtils always gets the correct
>      * version, even if JMeterUtils is not re-compiled during the build.
>      */
> -    private static final String VERSION = "2.5";
> +    private static final String VERSION = "2.5 r1152655";
>
>  For me this is not big enough to cast a negative vote, this really
>  depends on how closely the JMeter devs want the tag and the released
>  code to match.

This is tricky. The idea is that the displayed version includes the
SVN revision number from which the binary was built.

The file would need to be updated when creating the tag so it gets the
same id as the tag.
The existing post-tag update phase could be changed to check that the
correct rev number had been included.

I'll see if I can get this working, but it may take a while as I don't
have access to my normal build PC at present.

> * in addition the source distribution is missing the docs subdirectory.
>  Is this intentional?

The docs subdirectory is generated from xdocs, so is not needed in the
source archive.

It's only stored in SVN for use when updating the website.

> * RAT only flags some testfiles, which is good enough for me
>
> * NOTICE should contain 2011 rather than 2010

Good point; needs to be fixed.

> * ApacheJMeter.jar should probably not contain the tird party notice and
>  licenses but only the ASF ones as the files inside the jar only apply
>  to the jar and not the binary distribution.  No biggy for me.

Probably should be fixed.

> Not a +1, yet.
>
> Stefan
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>