Re: new package: wip/dcc (Distributed Checksum Clearinghouses)

[Petar Bogdanovic <petar%smokva.net@localhost>]

On Tue, Dec 23, 2008 at 10:46:07AM -0500, Greg Troxel wrote:
> 
> Petar Bogdanovic <petar%smokva.net@localhost> writes:
> 
> > On Tue, Dec 23, 2008 at 09:33:44AM -0500, Greg Troxel wrote:
> >> 
> >> Petar Bogdanovic <petar%smokva.net@localhost> writes:
> >> So, the question is whether there should even be options, or just build
> >> it all.  What bad thing would happen if all three options were defaulted
> >> to on?
> >> 
> >> I suspect Vernon would prefer that we always include dccd, so as to
> >> maximally encourage running a server.  It seems that dccd would be ok,
> >> just a few more bits.  Leaving the option lets someone who is for some
> >> reason both filtering mail and tight on space cope, but it seems dccd
> >> should be present in default builds.
> >
> > The only disadvantage when adding dccd is that you'll pollute your
> > dcc_home with configuration files only used by dccd.. which is a bit
> > disencouraging if you just want to use one of the clients. At least
> > that's what my impression was when I first did a default install. :)
> 
> I see - but with the dccproc man page and the spamassassin help, one
> doesn't need to look :-)
> 
> >> (A nit is that --disable-server should also omit the server man pages.)
> >
> > I tried to keep it default and follow what `make install' does. And it
> > leaves not just the man pages but also the configuration examples for
> > dccd. (and maybe a few other things)
> 
> That's totally reasonable.  pkgsrc is really not the place to fix things
> like that - should be reported/fixed upstream.
> 
> > Also there is no way to disable the cgi-bin stuff which is the only
> > reason why the package depends on perl.
> 
> Sure there is: add a patch that removes it from Makefile.  But I'm not
> sure one should do that - perl is big, but the odds of someone wanting
> dcc and not already having perl seem low.

Sorry, what I wanted to say is that there is no way to disable the
cgi-bin stuff over configure.


Let me sum up:

        - Leave the options and do not split the package until someone
          complains. Default enabled options will be `dccd dccifd' while
          `dccm' will be disabled.

        - Add a examples/dcc_spamassassin.cf file which can easily be
          copied to /etc/pkg/spamassassin in order to enable DCC.pm
          properly.

        - Mention examples/dcc_spamassassin.cf in MESSAGES.


If that's ok, I'll modify the package somewhen during the next few days.

------------------------------------------------------------------------------
_______________________________________________
pkgsrc-wip-review mailing list
pkgsrc-wip-review%lists.sourceforge.net@localhost
https://lists.sourceforge.net/lists/listinfo/pkgsrc-wip-review