uninstall stanzas to improve

[rolandwalker]

currently uses uninstall :delete

These would clearly be improved by using uninstall :pkgutil. If possible, the :delete directive should be removed.

• authbuddy.rb
• growl-fork.rb
• netbeans-cpp.rb [ID overlap with other Netbeans]
• netbeans-php.rb [ID overlap with other Netbeans]
• squirrel.rb
• whatpulse.rb
• ntfs-free.rb [appears not to have any ID]
• zoomus.rb [appears not to use BOMs]

currently lacks uninstall :delete or uninstall :script

This occurred because a :delete directive was moved to the zap stanza in #6164.

• mailfollowup.rb

currently uses uninstall :script

uninstall :script is fine and does not need to change. However, these Casks would still be improved by adding a :pkgutil directive. The :pkgutil directive would help robustness in corner cases, such as when the installation has been mangled and the expected uninstall script is missing.

• ax88179.rb
• ax88772.rb
• camtwist.rb
• cartao-de-cidadao.rb [nonspecific ID pt.eid.middleware?]
• controllermate.rb
• doublecommand.rb
• drobo-dashboard.rb
• fitbit-connect.rb
• fluke.rb
• gambit-c.rb
• haskell-platform.rb
• intel-haxm.rb [should be com.intel.kext.haxm.#{version}; confirmation needed]
• leap-motion.rb [bad url; must update cask before proceeding]
• logitech-control-center.rb
• logitech-gaming-software.rb
• nosleep.rb
• opennx.rb
• pdftk.rb
• power-manager.rb [bad pkg; must update cask before proceeding]
• razer-synapse.rb
• screens-connect.rb
• totalfinder.rb
• totalterminal.rb
• vagrant.rb
• virtualbox.rb
• zendserver.rb

References

#3322

[tapeinosyne]

I took the liberty of updating the opening message with information concerning the uninstall stanza of specific entries.

The Netbeans IDEs appear to share a common ID, org.netbeans.ide.baseide.#{version}, which should likely not be removed by our uninstall.

Tangential: growl-fork seems to be an unofficial build, which we should move to caskroom/unofficial.

[rolandwalker]

ntfs-free.rb may appear to have no bundle ID because it uses Apple bundle IDs which are filtered out by the development scripts. Nonetheless we shouldn't uninstall those IDs.

[vitorgalvao]

cartao-de-cidadao is a tricky one. I was the one to add it. It’s software to work together with a hardware device to interact with Portuguese (and perhaps other similar) ID cards. Trying to get everything correct for the uninstall was a mess, so it was great when I found out they had packaged a script. Not to say it can’t be done, naturally, but it does have some weirdnesses, like the aforementioned nonspecific ID.

[vitorgalvao]

Most of these are not needed or terribly outdated, :pkgutil isn’t perfect anyway so it shouldn’t be the standard for correctness, and others simply can’t be solved with it.