[Bug 62128] JMETER_HOME not set correctly in jmeter.bat

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

Philippe Mouawad  changed:

   What|Removed |Added

   Target Milestone|--- |JMETER_4.1
   Hardware|PC  |All
   Keywords||FixedInTrunk

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 62128] JMETER_HOME not set correctly in jmeter.bat

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

Philippe Mouawad  changed:

   What|Removed |Added

 CC||p.mouawad@ubik-ingenierie.c
   ||om

--- Comment #6 from Philippe Mouawad  ---
Hi Andrei,
What is so critical about it ?
I ‘ve tested jmeter 4.0 on windows and it didn’t hurt me that much.

but I may have missed something.

If we release a minor version, there is a risk if it’s nightly as we have
hugely changed Http following removal of deprecated APIs and although we have
been using it a lot in our team in scripting, we didn’t have yet the occasion
to do a massive load test to ensure no issue appears.

Or maybe you’d like to only release this hotfix ?

Regards

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 62128] JMETER_HOME not set correctly in jmeter.bat

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

--- Comment #5 from Andrey Pokhilko  ---
IMO this issue is so critical that worth immediate minor release of JMeter...

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 62128] JMETER_HOME not set correctly in jmeter.bat

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

logo...@gmx.at changed:

   What|Removed |Added

 Status|RESOLVED|CLOSED

--- Comment #4 from logo...@gmx.at ---
Thanks for fixing the issue.
I successfully retested it with 4.1 r1825385.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 62128] JMETER_HOME not set correctly in jmeter.bat

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

Felix Schumacher  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED

--- Comment #3 from Felix Schumacher  ---
Hopefully correct now. Thanks for the review.

Date: Mon Feb 26 17:23:57 2018
New Revision: 1825385

URL: http://svn.apache.org/viewvc?rev=1825385=rev
Log:
Set JMETER_HOME variable correctly.

Bugzilla Id: 62128

Modified:
jmeter/trunk/bin/jmeter.bat

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 62128] JMETER_HOME not set correctly in jmeter.bat

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

logo...@gmx.at changed:

   What|Removed |Added

 Resolution|FIXED   |---
 Status|RESOLVED|REOPENED

--- Comment #2 from logo...@gmx.at ---
I just tested the new jmeter.bat (r1825328).

This line:

set "%JMETER_HOME%=%~dp0\.."

needs to be replaced by that:

set "JMETER_HOME=%~dp0\.."

Best regards

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 62128] JMETER_HOME not set correctly in jmeter.bat

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

Felix Schumacher  changed:

   What|Removed |Added

 OS||All
 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #1 from Felix Schumacher  ---
Thanks for your contribution. This will be included in v4.1

It would be nice, if you could verify, that this fixes your problems.

Date: Sat Feb 24 09:57:38 2018
New Revision: 1825203

URL: http://svn.apache.org/viewvc?rev=1825203=rev
Log:
Try to guess JMETER_HOME correctly, when jmeter.bat is called from a batch file
in another directory.

Contributed by logox01 (logox01 at gmx.at)

Bugzilla Id: 62128

Modified:
jmeter/trunk/bin/jmeter.bat
jmeter/trunk/xdocs/changes.xml

-- 
You are receiving this mail because:
You are the assignee for the bug.