This won't build on OpenBSD since the BSD version of setpgrp() takes arguments. It should be fine to just delete the setpgrp() call since setpgid(0, getpid()) goes the same thing, and is actually portable.
I would actually question why the setpgid or setpgrp call is needed at all if we are unconditionally calling setsid() now. It would seem to be a no-op with all the ifdefs removed.
This won't build on OpenBSD since the BSD version of setpgrp() takes arguments. It should be fine to just delete the setpgrp() call since setpgid(0, getpid()) goes the same thing, and is actually portable.
I would actually question why the setpgid or setpgrp call is needed at all if we are unconditionally calling setsid() now. It would seem to be a no-op with all the ifdefs removed.