Griffon
  1. Griffon
  2. GRIFFON-603

Force all dependent plugins to be uninstalled when a plugin is uninstalled

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.2.0
    • Fix Version/s: 1.3.0
    • Component/s: cli
    • Labels:
      None
    • Number of attachments :
      0

      Description

      Say you have a plugin A that depends on B a C. If A is uninstalled B and C remain still as they may be needed by other parts of the application. This is feature will instruct the command line to uninstall B and C as part of the uninstall sequence of A. Uninstalling B and C will remove their dependent plugins too.

      Caveat: toolkit plugins should not be removed (e.g, dialogs depends on miglayout and glazedlists, which both depend on swing. Uninstalling dialogs should uninstall miglayout and glazedlists only).

        Activity

        Hide
        Davide Cavestro added a comment -

        As a further refinement one could expect that B and C are uninstalled only if there are no other plugins that depends on them.
        If that's not feasible, the proposed solution would still be very useful, provided that we have a way to display plugin dependency graph and to try uninstalling without the changes being applied.

        Show
        Davide Cavestro added a comment - As a further refinement one could expect that B and C are uninstalled only if there are no other plugins that depends on them. If that's not feasible, the proposed solution would still be very useful, provided that we have a way to display plugin dependency graph and to try uninstalling without the changes being applied .
        Show
        Andres Almiray added a comment - Fixed with https://github.com/griffon/griffon/commit/71598915bdb4dc4b666595d643af4829256e50b9

          People

          • Assignee:
            Andres Almiray
            Reporter:
            Andres Almiray
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: