Ticket #31280: move_unload_to_deactivate.diff

fuse4x-kext/Portfile

@@ -66 +66 @@
     xinstall -o root -m 4755 ${worksrcpath}/build/${xcode.configuration}/load_fuse4x ${destroot}${kextdir}/fuse4x.kext/Support
 }
 
-# We don't need to load the kext, because the fuse4x library will
-# autoload it when needed. However, if there's an existing version
-# loaded, we'd better unload it because it might be a different
-# version.
+# If fuse4x kext is runnig let's unload it before deactivating the port.
 #
-# Thus, check whether the kext is already loaded, and if so attempt to
-# unload it. If that fails, the user probably needs to unmount some
+# If unload fails the user probably needs to unmount some
 # running filesystems; print a list.
-post-activate {
+pre-deactivate {
     set kextid "org.fuse4x.kext.fuse4x"
     if {[string length [exec /usr/sbin/kextstat -lb $kextid]] > 0} {
-        ui_msg "Another version of Fuse4X is already loaded. Attempting to unload it."
-        if {![catch {exec /sbin/kextunload -b $kextid}]} {
-            ui_msg "Successfully unloaded the previous installation of Fuse4X"
-        } else {
+        if {[catch {exec /sbin/kextunload -b $kextid}]} {
             set mounted [exec /sbin/mount -t fuse4x]
             ui_warn "Unable to unload the existing Fuse4X kernel extension."
-            ui_warn "To complete the upgrade. please unmount the following filesystems "
-            ui_warn "and then run `sudo kextunload -b $kextid`:"
+            ui_warn "To complete the deactivation please unmount the following filesystems:"
             foreach line [split $mounted "\n"] {
-                regexp {on (/.*) \(fuse4x} $line -> mountpoint
-                ui_warn "  - $mountpoint"
+                regexp {(.*)@fuse\d+ on (/.*) \(fuse4x} $line -> fsname mountpoint
+                ui_warn "     $fsname on $mountpoint"
             }
+            ui_warn "and then run `sudo kextunload -b $kextid`:"
         }
     }
 }