[packages] Add support for pre- and post- scripts everywhere

- for remove and localInstall, add support for pre- and post-
   scripts like there already was for install.

This feels like there's code duplication going on, but I haven't
thought of an elegant way to distinguish the available operations
so that I can pass around functions instead.

src/modules/packages/main.py

@@ -126,7 +126,8 @@ class PackageManager(metaclass=abc.ABCMeta):
         """
         Install a package from a single entry in the install list.
         This can be either a single package name, or an object
-        with pre- and post-scripts.
+        with pre- and post-scripts. If @p packagedata is a dict,
+        it is assumed to follow the documented structure.
 
         @param packagedata: str|dict
         @param from_local: bool
@@ -139,6 +140,22 @@ class PackageManager(metaclass=abc.ABCMeta):
             self.install([packagedata["package"]], from_local=from_local)
             self.run(packagedata["post-script"])
 
+    def remove_package(self, packagedata):
+        """
+        Remove a package from a single entry in the remove list.
+        This can be either a single package name, or an object
+        with pre- and post-scripts. If @p packagedata is a dict,
+        it is assumed to follow the documented structure.
+
+        @param packagedata: str|dict
+        """
+        if isinstance(packagedata, str):
+            self.remove([packagedata], from_local=from_local)
+        else:
+            self.run(packagedata["pre-script"])
+            self.remove([packagedata["package"]], from_local=from_local)
+            self.run(packagedata["post-script"])
+
 
 class PMPackageKit(PackageManager):
     backend = "packagekit"
@@ -440,19 +457,27 @@ def run_operations(pkgman, entry):
                     libcalamares.utils.warning(warn_text)
         elif key == "remove":
             _change_mode(REMOVE)
+            if all([isinstance(x, str) for x in package_list]):
                 pkgman.remove(package_list)
+            else:
+                for package in package_list:
+                    pkgman.remove_package(package)
         elif key == "try_remove":
             _change_mode(REMOVE)
             for package in package_list:
                 try:
-                    pkgman.remove([package])
+                    pkgman.remove_package(package)
                 except subprocess.CalledProcessError:
                     warn_text = "Could not remove package "
-                    warn_text += package
+                    warn_text += str(package)
                     libcalamares.utils.warning(warn_text)
         elif key == "localInstall":
             _change_mode(INSTALL)
+            if all([isinstance(x, str) for x in package_list]):
                 pkgman.install(package_list, from_local=True)
+            else:
+                for package in package_list:
+                    pkgman.install_package(package, from_local=True)
 
         completed_packages += len(package_list)
         libcalamares.job.setprogress(completed_packages * 1.0 / total_packages)

src/modules/packages/packages.conf

@@ -91,9 +91,6 @@ update_system: false
 # "package: vi" with neither script option will trick Calamares into
 # trying to install a package named "package: vi", which is unlikely to work.
 #
-# Pre- and post-scripts are supported only in the install and try_install
-# operations.
-#
 # Any package name may be localized; this is used to install localization
 # packages for software based on the selected system locale. By including
 # the string `LOCALE` in the package name, the following happens: