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

[Solaris] : Fix for 8071710 needs to be updated for build dependency checking.

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: P2
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 9
    • Component/s: infrastructure
    • Labels:
    • Subcomponent:
    • Introduced In Version:
      9
    • Resolved In Build:
      b52
    • OS:
      solaris

      Backports

        Description

        8071710: [solaris] libfontmanager should be linked against headless awt library
        https://bugs.openjdk.java.net/browse/JDK-8071710
        updated libfontmanager to link against headless AWT instead of XAWT.
        It also updated the dependent library.
        However sporadically Solaris builds will fail as if the dependency checking was not working.
        Re-ordering the declarations of the libs to build so that headless was not one of
        the last ones resolved it which seemed to support this theory
        Magnus worked out that the exact reason is that the HEADLESS lib being declared
        after FONTMANAGER meant that the dependency was indeed not working :-
        "
        As it turns out, the the call to SetupNativeCompilation for BUILD_LIBAWT_HEADLESS is placed lower in the makefile than BUILD_LIBFONTMANAGER. This means that the variable $(BUILD_LIBAWT_HEADLESS) is empty when the dependency criterion is evaluated, and no actual extra dependency between the two libraries are recorded by make. :-(

        The simplest way to solve this is to reorder the two blocks. I suggest placing BUILD_LIBFONTMANAGER below BUILD_LIBAWT_HEADLESS, I have verified that no other dependencies will be broken this way."

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  prr Philip Race
                  Reporter:
                  prr Philip Race
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: