Opened 8 years ago
Closed 8 years ago
#49989 closed defect (fixed)
nedi: activation fails: /Library/LaunchDaemons/org.macports.nedisyslog.plist already exists and does not belong to a registered port
| Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | ryandesign (Ryan Carsten Schmidt) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | haspatch | Cc: | Ionic (Mihai Moldovan), mojca (Mojca Miklavec), dbevans (David B. Evans) |
| Port: | nedi |
Description
Mojca and Mihai emailed that "/Library/LaunchDaemons/org.macports.nedisyslog.plist already exists and does not belong to a registered port".
The nedi port used to install this file incorrectly, bypassing the destroot. This was reported in #40700 and fixed in r128981, however a pre-activate phase was not added to the port to automatically clean up the results of the former incorrect behavior, thus leading to the activation problem now. Such a block should be added. The revision of the port needs to be increased when doing this.
Attachments (1)
Change History (4)
Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)
comment:1 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)
| Keywords: | haspatch added |
|---|
I haven't tested this patch yet.
comment:2 Changed 8 years ago by mojca (Mojca Miklavec)
| Cc: | devans@… added |
|---|
comment:3 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Committed with changes in r143403. My patch needed adjustment because it was testing for file existence, but the file that had been installed was a symlink, which would be broken; [file exists] will return false for a broken symlink. I adjusted it to use [file type] instead.
I don't know nedi or this portfile so I'm not updating the port to a new version right now. You can file a new ticket for that it you like.
proposed patch