Hmm. This PR was green for a few days as well. Seems like something a repo owner would need to intervene on.
On Feb 20, 2025, at 10:01 AM, grey <artki...@gmail.com> wrote:
I encountered something similar recently with the OpenSSH 9.9p2 PR I submitted.
I took screen shots and documented it in the Pull Request:
I have absolutely no idea what was causing that, reverting to classic view (even though I never enabled anything different) seemed to resolve it? I am vaguely relieved to see someone else encountering something similar, but my best guess is: GitHub effed something up, again. 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 blockedMerge is not an allowed merge method in this repository. - This branch must not contain merge commits.
Please Advise, Blake 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.
|
