Re: vsftpd doesn't compile on DragonFlyBSD

[TimH <tim%130collective.org@localhost>]

On Sat, 3 Nov 2012 14:45:24 +0100
Thomas Klausner <wiz%NetBSD.org@localhost> wrote:

> On Fri, Nov 02, 2012 at 08:06:32PM -0700, TimH wrote:
> > Testing on 3.2.1 and a current snap.
> > 
> > I made this change and it compiles and runs (both ipv6 and ipv4).  I'm
> > very far from an expert, so there may be something else needing doing.
> > 
> > 
> > --- work/vsftpd-2.3.4/sysdeputil.c.orig 2012-11-01 19:24:36.210186000 -0700
> > +++ work/vsftpd-2.3.4/sysdeputil.c      2012-11-01 19:25:00.250549000 -0700
> > @@ -116,6 +116,11 @@
> >    #endif
> >  #endif
> >  
> > +#if defined(__DragonFly__)
> > +  #define VSF_SYSDEP_HAVE_SETPROCTITLE
> > +  #define WTMPX_FILE _PATH_WTMPX
> > +#endif
> > +
> >  #ifdef __hpux
> >    #include <sys/socket.h>
> >    #ifdef SF_DISCONNECT
> > 
> 
> Committed, thanks. Please send this patch upstream!
>  Thomas

        I'm going to grab their latest code and try to work up a patch
against that.

        The other thing about this program is that you need to run two
different instances of it in order to have ipv4 and ipv6.

        I set up rc.donf like so:

vsftpd_enable="YES"
vsftpdv6_enable="YES"

        I created an additional rc.d script like so:

#!/bin/sh
#
# $NetBSD: vsftpd.sh,v 1.2 2004/11/11 13:51:31 xtraeme Exp $
#

# PROVIDE: vsftpdv6
# REQUIRE: DAEMON LOGIN

. /etc/rc.subr

name="vsftpd"
rcvar=$name
command="/usr/pkg/libexec/${name}"
command_args="/usr/pkg/etc/vsftpd/vsftpdv6.conf"
required_dirs="/var/chroot/vsftpd"
required_files="/usr/pkg/etc/vsftpd/vsftpdv6.conf"
pidfile="/var/run/vsftpdv6.pid"

load_rc_config $name
run_rc_command "$1"


        This runs a second instance of vsftpd that reads the v6 conf
file and uses a pidfile so that there is no confusion about seeing the
v4 instance's process and thinking it's already running.  Also note the
lack of the "&" in command_args which is not needed since I have
background set in the conf file.

        I'm not sure how pkgsrc would deal with needing two rc.d
scripts for one package...

Cheers!
--TimH