Re: Netatalk 4.x

Thu, 20 Feb 2025 09:32:28 -0800

This PR is looking ready from my end however I'm not sure if I need to do anything about this message on github? 

Merging is blocked

  • Merge is not an allowed merge method in this repository.
  • This branch must not contain merge commits.

Please Advise,
Blake

On Feb 12, 2025, at 7:56 AM, bl...@netjibbing.com wrote:

The quoting on startupitem.executable was my main issue. I’ll incorporate these other tips as well.

Thanks!


On Feb 11, 2025, at 9:30 PM, Ryan Carsten Schmidt <ryandes...@macports.org> wrote:

On Feb 11, 2025, at 20:19, Blake wrote:

startupitem.name    ${name}

That's the default so you should omit this line.

startupitem.executable  "${prefix}/sbin/netatalk -d -F ${prefix}/etc/afp.conf"

Per guide.macports.org:

Do not wrap values in quotes if passing arguments to the daemon; “executable” StartupItem elements must be tagged individually so the spaces between arguments serve as delimiters for “string” tags.

startupitem.pidfile auto /var/run/netatalk.pid

We usually put these inside ${prefix} but since you're using an executable not a script you don't need a pidfile at all.

Reply via email to