[MacPorts] #14223: p5-mail-spamassassin upgrade overwrites the user's /Library/LaunchDaemons/org.macports.spamd.plist
#14223: p5-mail-spamassassin upgrade overwrites the user's /Library/LaunchDaemons/org.macports.spamd.plist ---------------------------------+------------------------------------------ Reporter: vinc17@macports.org | Owner: simon@macports.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Keywords: | ---------------------------------+------------------------------------------ After upgrading p5-mail-spamassassin, my /Library/LaunchDaemons/org.macports.spamd.plist file has been overwritten. This is a config file and the port shouldn't undo the user's changes. -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14223> MacPorts </projects/macports> Ports system for Mac OS
#14223: p5-mail-spamassassin upgrade overwrites the user's /Library/LaunchDaemons/org.macports.spamd.plist ----------------------------------+----------------------------------------- Reporter: vinc17@macports.org | Owner: simon@macports.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: | Keywords: ----------------------------------+----------------------------------------- Comment (by dluke@macports.org): It's not a config file, it's the automatically generated startup item (which happens to be a launchd plist in this case). Perhaps we should document that you shouldn't modify it? You can always not enable the macports-provided one and create your own plist based off of it and use it instead. -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14223#comment:1> MacPorts </projects/macports> Ports system for Mac OS
#14223: p5-mail-spamassassin upgrade overwrites the user's /Library/LaunchDaemons/org.macports.spamd.plist ----------------------------------+----------------------------------------- Reporter: vinc17@macports.org | Owner: simon@macports.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: | Keywords: ----------------------------------+----------------------------------------- Comment (by vinc17@macports.org): The file is under /opt/local/etc/ thus it is a config file, like all files in /opt/local/etc/. If the user mustn't modify it, the file should be somewhere else, and the symbolic link /Library/LaunchDaemons/org.macports.spamd.plist should not be created (otherwise the user can't create his own plist file as there would be two plist files for the same service, which is very bad). -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14223#comment:2> MacPorts </projects/macports> Ports system for Mac OS
#14223: p5-mail-spamassassin upgrade overwrites the user's /Library/LaunchDaemons/org.macports.spamd.plist ----------------------------------+----------------------------------------- Reporter: vinc17@macports.org | Owner: simon@macports.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: | Keywords: ----------------------------------+----------------------------------------- Comment (by dluke@macports.org): Having two plist files (or 100) there for the same service doesn't break anything, especially since the ones installed by macports are not active. -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14223#comment:3> MacPorts </projects/macports> Ports system for Mac OS
#14223: p5-mail-spamassassin upgrade overwrites the user's /Library/LaunchDaemons/org.macports.spamd.plist ----------------------------------+----------------------------------------- Reporter: vinc17@macports.org | Owner: simon@macports.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: | Keywords: ----------------------------------+----------------------------------------- Comment (by vinc17@macports.org): Since it is not active, there is no need to install it, and having it installed can confuse the user. If this is just for an example, then it should be under the documentation directory. -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14223#comment:4> MacPorts </projects/macports> Ports system for Mac OS
#14223: p5-mail-spamassassin upgrade overwrites the user's /Library/LaunchDaemons/org.macports.spamd.plist ----------------------------------+----------------------------------------- Reporter: vinc17@macports.org | Owner: simon@macports.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: | Keywords: ----------------------------------+----------------------------------------- Comment (by dluke@macports.org): Replying to [comment:4 vinc17@macports.org]:
Since it is not active, there is no need to install it,
It is installed to enable the end user to easily 'activate' the automatic startup (launchctl load -w foo.plist). The way the current startupitem code works, the plist will be recreated/replaced upon upgrade. The plists that are generated should be sufficient for people's needs, but if you wanted to use it as a base for a plist that does something differently, that would be reasonable as well. Perhaps we should update the documentation to make this more clear? In any event, it's not a bug for this particular port (it's either a documentation bug, or a base bug). -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14223#comment:5> MacPorts </projects/macports> Ports system for Mac OS
#14223: p5-mail-spamassassin upgrade overwrites the user's /Library/LaunchDaemons/org.macports.spamd.plist ----------------------------------+----------------------------------------- Reporter: vinc17@macports.org | Owner: simon@macports.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.6.0 Resolution: | Keywords: ----------------------------------+----------------------------------------- Comment (by vinc17@macports.org): Replying to [comment:5 dluke@macports.org]:
It is installed to enable the end user to easily 'activate' the automatic startup (launchctl load -w foo.plist).
But the {{{launchctl load -w foo.plist}}} is not remembered after a reboot. So, I don't think this is easy. Also, if the user wants to do {{{launchctl load -w foo.plist}}}, I suppose he can still do it if the plist is in the doc directory (or in some other directory that makes clear that the files must not be modified by the user).
The way the current startupitem code works, the plist will be recreated/replaced upon upgrade.
The replace script should be able to detect when the file has been modified. The current solution is broken. BTW, there has been a similar discussion in the macports-dev (''Updating macports.conf'').
Perhaps we should update the documentation to make this more clear?
The current way is still confusing as it breaks conventions.
In any event, it's not a bug for this particular port (it's either a documentation bug, or a base bug).
Yes, perhaps a base bug. -- Ticket URL: <http://trac.macosforge.org/projects/macports/ticket/14223#comment:6> MacPorts </projects/macports> Ports system for Mac OS
participants (1)
-
MacPorts