Uploaded image for project: 'Application Server 3  4  5 and 6'
  1. Application Server 3 4 5 and 6
  2. JBAS-7104

Typo in windows run.bat prevents inclusion of tools.jar

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Trivial
    • Resolution: Done
    • Affects Version/s: JBossAS-5.1.0.GA
    • Fix Version/s: 6.0.0.M1
    • Component/s: Other
    • Labels:
      None
    • Similar Issues:
      Show 10 results 

      Description

      Just noticed the following in the windows run.bat while investigating an issue.

      if "x%JAVAC_JAR%" == "x" (
      set "RUNJAR=%JAVAC_JAR%;%JBOSS_HOME%\bin\run.jar"
      ) else (
      set "RUNJAR=%JBOSS_HOME%\bin\run.jar"
      )

      The set commands should be reversed.

        Gliffy Diagrams

          Activity

          Hide
          smarlow Scott Marlow added a comment -

          Yes, this is clearly a typo. What is the impact of not having the Java compiler on the system classpath when starting the as? Is this impacting Tomcat jsp compiling? I'm just curious what the issue was.

          Show
          smarlow Scott Marlow added a comment - Yes, this is clearly a typo. What is the impact of not having the Java compiler on the system classpath when starting the as? Is this impacting Tomcat jsp compiling? I'm just curious what the issue was.
          Hide
          kconner Kevin Conner added a comment -

          We embed wise within the ESB (SOA platform) and this requires the compiler to compile WS proxies. Not an ideal situation but one we need for now.

          Show
          kconner Kevin Conner added a comment - We embed wise within the ESB (SOA platform) and this requires the compiler to compile WS proxies. Not an ideal situation but one we need for now.
          Hide
          smarlow Scott Marlow added a comment -

          Typo is fixed in Branch_5_x + trunk.

          Show
          smarlow Scott Marlow added a comment - Typo is fixed in Branch_5_x + trunk.

            People

            • Assignee:
              smarlow Scott Marlow
              Reporter:
              kconner Kevin Conner
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development