jakarta-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: [VOTE] Release JMeter 2.5 RC1
Date Tue, 02 Aug 2011 14:18:20 GMT
On 2 August 2011 14:05, sebb <sebbaz@gmail.com> wrote:
> On 1 August 2011 06:46, Stefan Bodewig <bodewig@apache.org> 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.

I see that Milamber has fixed that.

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

Looks like it can be made to work.

The procedure that worked for me was:

Update the file containing the $Revision keyword. It's not sufficient
to edit the text within the keywork field, as SVN does not consider
that to be an update. Probably the easiest would be to add the current
date as a comment to the version file.

The SVN tag can then be created using svn copy from the workspace.

See my test commit:

URL: http://svn.apache.org/viewvc?rev=1153135&view=rev
Log:
Test tag / keyword creation

There's a bit more work still needed to incorporate this into the
JMeter build process.
For example, the keyword expansion produces "$Revision: 1153135 $",
but ideally we only want the revision number.

I can make a start on this.

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

I see Milamber has done that too.

>> * 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: dev-unsubscribe@jakarta.apache.org
>> For additional commands, e-mail: dev-help@jakarta.apache.org
>>
>>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: dev-help@jakarta.apache.org


Mime
View raw message