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

Remove obsolete properties from build.properties

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: P3
    • Resolution: Fixed
    • Affects Version/s: 9
    • Fix Version/s: 9
    • Component/s: javafx
    • Labels:

      Description

      The following properties in build.properties are now obsolete and should be removed to avoid confusion:

      jfx.build.jdk.version=9
      jfx.build.jdk.buildnum=144

      The other two properties, jfx.build.jdk.version.min and jfx.build.jdk.buildnum.min are used and need to remain.

        Activity

        Hide
        kcr Kevin Rushforth added a comment -
        diff --git a/build.properties b/build.properties
        --- a/build.properties
        +++ b/build.properties
        @@ -68,5 +68,3 @@
         
         jfx.build.jdk.version.min=9
         jfx.build.jdk.buildnum.min=144
        -jfx.build.jdk.version=9
        -jfx.build.jdk.buildnum=144
        Show
        kcr Kevin Rushforth added a comment - diff --git a/build.properties b/build.properties --- a/build.properties +++ b/build.properties @@ -68,5 +68,3 @@    jfx.build.jdk.version.min=9  jfx.build.jdk.buildnum.min=144 -jfx.build.jdk.version=9 -jfx.build.jdk.buildnum=144
        Hide
        ddhill David Hill (Inactive) added a comment -
        +1
        Show
        ddhill David Hill (Inactive) added a comment - +1
        Hide
        kcr Kevin Rushforth added a comment -
        Changeset: 3d0af5519e26
        Author: kcr
        Date: 2016-11-17 09:39 -0800
        URL: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/3d0af5519e26

        8169899: Remove obsolete properties from build.properties
        Reviewed-by: ddhill
        Show
        kcr Kevin Rushforth added a comment - Changeset: 3d0af5519e26 Author: kcr Date: 2016-11-17 09:39 -0800 URL: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/3d0af5519e26 8169899: Remove obsolete properties from build.properties Reviewed-by: ddhill

          People

          • Assignee:
            kcr Kevin Rushforth
            Reporter:
            kcr Kevin Rushforth
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: