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

Remove unused, obsolete iOS code

    Details

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

      Description

      The following src/test directories are obsolete and unused:

      extensions
      apps/toys/iOS
      tools/ios/Maven

      They are currently unused, are not hooked up to the build, and would not compile given the changes for modularization.

      The proposed solution is to remove these from the repository to avoid any confusion.

      Note that this supersedes JDK-8144130 which will be closed as a duplicate of this more general cleanup bug.

        Issue Links

          Activity

          Hide
          kcr Kevin Rushforth added a comment -
          Proposed fix:

          $ hg rm apps/toys/iOS
          $ hg rm tools/ios/Maven
          $ hg rm extensions
          Show
          kcr Kevin Rushforth added a comment - Proposed fix: $ hg rm apps/toys/iOS $ hg rm tools/ios/Maven $ hg rm extensions
          Hide
          jvos Johan Vos added a comment -
          I'm ok with this proposal.
          Show
          jvos Johan Vos added a comment - I'm ok with this proposal.
          Hide
          kcr Kevin Rushforth added a comment -
          Webrev: http://cr.openjdk.java.net/~kcr/8172216/webrev.00/

          In addition to the three directories deleted using "hg rm" as listed above, I removed a commented out reference to those now-deleted directories from build.gradle and from an IDE file.
          Show
          kcr Kevin Rushforth added a comment - Webrev: http://cr.openjdk.java.net/~kcr/8172216/webrev.00/ In addition to the three directories deleted using "hg rm" as listed above, I removed a commented out reference to those now-deleted directories from build.gradle and from an IDE file.
          Hide
          ckyang Chien Yang added a comment -
          +1
          Show
          ckyang Chien Yang added a comment - +1
          Hide
          kcr Kevin Rushforth added a comment -
          Changeset: 984e55cfdf6f
          Author: kcr
          Date: 2017-01-05 10:02 -0800
          URL: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/984e55cfdf6f

          8172216: Remove unused, obsolete iOS code
          Reviewed-by: ckyang
          Show
          kcr Kevin Rushforth added a comment - Changeset: 984e55cfdf6f Author: kcr Date: 2017-01-05 10:02 -0800 URL: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/984e55cfdf6f 8172216: Remove unused, obsolete iOS code Reviewed-by: ckyang

            People

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

              Dates

              • Due:
                Created:
                Updated:
                Resolved: