[jboss-jira] [JBoss JIRA] (AS7-4387) CLONE - standalone.bat doesn't modify JAVA_OPTS correctly

[Rostislav Svoboda (JIRA)]

     [ https://issues.jboss.org/browse/AS7-4387?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rostislav Svoboda moved JBPAPP-8619 to AS7-4387:
------------------------------------------------

              Project: Application Server 7  (was: JBoss Enterprise Application Platform)
                  Key: AS7-4387  (was: JBPAPP-8619)
             Workflow: GIT Pull Request workflow   (was: jira)
    Affects Version/s: 7.1.1.Final
                           (was: EAP 6.0.0 ER 4)
          Component/s: Scripts
                           (was: Scripts and Commands)
             Security:     (was: Public)
        Fix Version/s: 7.1.2.Final-redhat1
                           (was: EAP 6.0.0 ER 5)
       Docs QE Status:   (was: NEW)

    
> CLONE - standalone.bat doesn't modify JAVA_OPTS correctly
> ---------------------------------------------------------
>
>                 Key: AS7-4387
>                 URL: https://issues.jboss.org/browse/AS7-4387
>             Project: Application Server 7
>          Issue Type: Bug
>          Components: Scripts
>    Affects Versions: 7.1.1.Final
>            Reporter: Rostislav Svoboda
>            Assignee: Paul Gier
>            Priority: Critical
>             Fix For: 7.1.2.Final-redhat1
>
>
> standalone.bat doesn't modify JAVA_OPTS correctly because Only last change of JAVA_OPTS is reflected. JAVA_OPTS isn't dynamically changed withing 'if' part.
> {code}
> if not "%PRESERVE_JAVA_OPTS%" == "true" (
>   rem Add -client to the JVM options, if supported (32 bit VM), and not overriden
>   echo "%JAVA_OPTS%" | findstr /I \-server > nul
>   if errorlevel == 1 (
>     "%JAVA%" -client -version 2>&1 | findstr /I /C:"Client VM" > nul
>     if not errorlevel == 1 (
>       set "JAVA_OPTS=-client %JAVA_OPTS%"
>     )
>   )
>   rem Add compressed oops, if supported (64 bit VM), and not overriden
>   echo "%JAVA_OPTS%" | findstr /I "\-XX:\-UseCompressedOops \-client" > nul
>   if errorlevel == 1 (
>     "%JAVA%" -XX:+UseCompressedOops -version > nul 2>&1
>     if not errorlevel == 1 (
>       set "JAVA_OPTS=-XX:+UseCompressedOops %JAVA_OPTS%"
>     )
>   )
>   rem Add tiered compilation, if supported (64 bit VM), and not overriden
>   echo "%JAVA_OPTS%" | findstr /I "\-XX:\-TieredCompilation \-client" > nul
>   if errorlevel == 1 (
>     "%JAVA%" -XX:+TieredCompilation -version > nul 2>&1
>     if not errorlevel == 1 (
>       set "JAVA_OPTS=-XX:+TieredCompilation %JAVA_OPTS%"
>     )
>   )
> )
> {code}
> must be changed to
> {code}
> if not "%PRESERVE_JAVA_OPTS%" == "true" (
>   rem Add -client to the JVM options, if supported (32 bit VM), and not overriden
>   echo "%JAVA_OPTS%" | findstr /I \-server > nul
>   if errorlevel == 1 (
>     "%JAVA%" -client -version 2>&1 | findstr /I /C:"Client VM" > nul
>     if not errorlevel == 1 (
>       set "JAVA_OPTS=-client %JAVA_OPTS%"
>     )
>   )
> )
> if not "%PRESERVE_JAVA_OPTS%" == "true" (
>   rem Add compressed oops, if supported (64 bit VM), and not overriden
>   echo "%JAVA_OPTS%" | findstr /I "\-XX:\-UseCompressedOops \-client" > nul
>   if errorlevel == 1 (
>     "%JAVA%" -XX:+UseCompressedOops -version > nul 2>&1
>     if not errorlevel == 1 (
>       set "JAVA_OPTS=-XX:+UseCompressedOops %JAVA_OPTS%"
>     )
>   )
> )
> if not "%PRESERVE_JAVA_OPTS%" == "true" (
>   rem Add tiered compilation, if supported (64 bit VM), and not overriden
>   echo "%JAVA_OPTS%" | findstr /I "\-XX:\-TieredCompilation \-client" > nul
>   if errorlevel == 1 (
>     "%JAVA%" -XX:+TieredCompilation -version > nul 2>&1
>     if not errorlevel == 1 (
>       set "JAVA_OPTS=-XX:+TieredCompilation %JAVA_OPTS%"
>     )
>   )
> )
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira