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

OpenJDK build can't handle commas in LDFLAGS

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • P4
    • 8u92
    • 8u65
    • infrastructure
    • None
    • b03

    Backports

      Description

        If additional LDFLAGS containing commas are passed to the build e.g.

        --with-extra-ldflags="-Wl,--as-needed -Wl,-O1 -Wl,--hash-style=gnu -Wl,--sort-common"

        (which are the default flags on Fedora), then the build will fail as the call to SetupNativeCompilation expands LDFLAGS_JDKLIB early and the commas are interpreted to denote separate arguments to this macro.

        On 8, this only affects the demo code (BUILD_DEMO_JVMTI_$1). It also affected the main JDK build on 9 until 8142907 which fixed both cases there. For 8, we should backport the relevant part of 8142907 under this bug.

        Attachments

          Issue Links

            backported by

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8149293 OpenJDK build can't handle commas in LDFLAGS

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8155438 OpenJDK build can't handle commas in LDFLAGS

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved
            relates to

            Bug - A problem which impairs or prevents the functions of the product. JDK-8142907 Integration of minor fixes from the build-infra project

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved

            Activity

              People

                andrew Andrew Hughes
                andrew Andrew Hughes
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: