Subject: Re: pkg/35287: New pkgsrc entries for Perforce (p4d/p4/p4web)
To: None <schmonz@NetBSD.org, gnats-admin@netbsd.org,>
From: Amitai Schlair <schmonz@netbsd.org>
List: pkgsrc-bugs
Date: 01/18/2007 05:20:02
The following reply was made to PR pkg/35287; it has been noted by GNATS.

From: Amitai Schlair <schmonz@netbsd.org>
To: gnats-bugs@NetBSD.org, Marc Tooley <netbsd-helper@quake.ca>
Cc: 
Subject: Re: pkg/35287: New pkgsrc entries for Perforce (p4d/p4/p4web)
Date: Thu, 18 Jan 2007 00:15:54 -0500

 I'm looking at your changes now, and the rc.d script is  
 intimidatingly detailed to read through and feel confident in (not  
 least because sometimes it says "/usr/pkg/sbin" and sometimes it says  
 "/usr/local/sbin"... :-). My feelings are mixed: on the one hand, I  
 like the idea of managing multiple p4d instances with a single script  
 -- and something similar could help me solve PR pkg/30957 -- but on  
 the other hand, the rc.d script winds up being distressingly complex.
 
 Part of the problem, as you've encountered firsthand, is that rc.subr  
 doesn't provide free goodies for managing multiple instances with one  
 script. Maybe it should. Until then, as much as I like the idea in  
 the abstract, this approach feels wrong to me. And I say this as a  
 guy who's written his fair share of hairy, non-traditional rc.d  
 scripts (check out mail/qmail-run sometime).
 
 Your effort here was obviously significant and I'd like to see it  
 come to fruition. I propose that you bring up the issues you faced in  
 writing this rc.d script in a post on tech-pkg@, and I'll pitch in  
 where I can, and if and when a consensus is reached, I'll be glad to  
 act on it. Meanwhile, for purposes of this PR, how about I apply the  
 rest of your update to 2006.1?