dnsmasq: install launchd plist on darwin #23578
Conversation
|
@nicknovitski, thanks for your PR! By analyzing the history of the files in this pull request, we identified @wkennington, @fpletz and @tailhook to be potential reviewers. |
| @@ -39,6 +39,11 @@ stdenv.mkDerivation rec { | |||
| # module can create it in Nix-land? | |||
| postInstall = '' | |||
| install -Dm644 trust-anchors.conf $out/share/dnsmasq/trust-anchors.conf | |||
| '' + optionalString stdenv.isDarwin '' | |||
| install -Dm644 contrib/MacOSX-launchd/uk.org.thekelleys.dnsmasq.plist \ | |||
| $out/System/Library/LaunchDaemons/uk.org.thekelleys.dnsmasq.plist | |||
There was a problem hiding this comment.
I'm pretty sure most other packages that have something like this put it in $out/Library/LaunchDaemons
There was a problem hiding this comment.
Huh, you're right, I found two that use Library and only one that used System/Library. Guess I didn't look closely enough the first time. I'll change my patch to match the majority.
nicknovitski
force-pushed
the
dnsmasq-plist
branch
from
a249b4d
to
44cf3c4
Compare
|
@LnL7 pointed out most packages put plist files in just |
|
Well that sucks. |
|
What's that from? |
|
Those seem to be the causes of the current build failures. |
It's just a static file in the
contribfolder of the project's source, added as an output inSystem/Library/LaunchDaemons(I copied this from a few other packages), with its path corrected.