You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@jmeter.apache.org by bu...@apache.org on 2018/02/23 07:33:36 UTC

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

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

            Bug ID: 62128
           Summary: JMETER_HOME not set correctly in jmeter.bat
           Product: JMeter
           Version: 4.0
          Hardware: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Main
          Assignee: issues@jmeter.apache.org
          Reporter: logox01@gmx.at
  Target Milestone: ---

Hi,

I'm very happy about the change to define the user specific settings in a
separate setenv.bat (no need to change the jmeter.bat anymore).

But currently I do need to update the jmeter.bat as well, because the
JMETER_HOME directory does not point to the jmeter directory.
That happens if you don't start the jmeter.bat directly, but from another
location (we use separate startup batch files within the testplan directory to
define testplan specific parameters).

It would be great if you could define another fallback for setting the correct
JMETER_HOME direcotry:

rem Guess JMETER_HOME if not defined
set "CURRENT_DIR=%cd%"
if not "%JMETER_HOME%" == "" goto gotHome
set "JMETER_HOME=%CURRENT_DIR%"
if exist "%JMETER_HOME%\bin\jmeter.bat" goto okHome
cd ..
set "JMETER_HOME=%cd%"
cd "%CURRENT_DIR%"
if exist "%JMETER_HOME%\bin\jmeter.bat" goto okHome
set JMETER_HOME=%~dp0\..
:gotHome

Thank you

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

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

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om

--- Comment #6 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
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

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

Felix Schumacher <fe...@internetallee.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|                            |All
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #1 from Felix Schumacher <fe...@internetallee.de> ---
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&view=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.

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

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

logox01@gmx.at changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #4 from logox01@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

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

Felix Schumacher <fe...@internetallee.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Felix Schumacher <fe...@internetallee.de> ---
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&view=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

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

--- Comment #5 from Andrey Pokhilko <ap...@ya.ru> ---
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

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

Philippe Mouawad <p....@ubik-ingenierie.com> 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

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=62128

logox01@gmx.at changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|FIXED                       |---
             Status|RESOLVED                    |REOPENED

--- Comment #2 from logox01@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.