Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-6818565

Regression with fix 6816311: COMPILER_VERSION -> REQUIRED_COMPILER_VERSION

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: P2 P2
    • 7
    • 7
    • infrastructure
    • None
    • b51
    • generic
    • generic

      6816311 fix broke solaris builds when ALT_JDK_DEVTOOLS was used (used by RE).

      When ALT_JDK_DEVTOOLS is used the COMPILER_PATH was being defined using the value of COMPILER_VERSION, but the changes in 6816311 made the COMPILER_NAME and COMPILER_VERSION (what is being used) defined via CC_VER (set with cc -version), but CC_VER dependeded on COMPILER_PATH. :^( Horrible cycle of make variable usage.

      1. The use of COMPILER_VERSION in COMPILER_PATH really should be REQUIRED_COMPILER_VERSION.

      2. The Defs-versions.gmk file that defines the REQUIRED* variables, needs to be included before the Compiler*.gmk files and after CC_VERSION is defined. (The names of all these CC*VER* variables should probably be re-organized.)

            ohair Kelly Ohair (Inactive)
            ohair Kelly Ohair (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved:
              Imported:
              Indexed: