X-Istence wrote (at Sun, Dec 14, 2003 at 08:44:53AM -0500):
> Casey Zacek wrote:
> 
> >'vadddomain -O' is broken in 5.4.0-pre1, and probably others.
> >
> >The problem is that vget_assign() looks in [/var/qmail/]users/cdb for
> >the domain, and it's not there yet for the vadduser(postmaster) call.
> >
> >vsetuserquota(), after vadduser(), also uses vget_assign(), as does
> >vadddomain's main() directly if a catchall has been chosen.
> >
> >It's not a big deal to me, but I thought I'd point it out because it
> >probably shouldn't be broken in 5.4.0.
> 
> Submit a patch on sourceforge or put it on the list if possible, as you 
> know why the problem happens.
> 
> If not, let me know where to look exactly, and ill take a look.

The problem is that I don't see how this ever worked.  The -O causes
vadddomain to skip steps that are used by vget_assign().  The solution
is to not use the -O option.  Either that, or make the -O option take
domain names on standard input, read from a file, or take multiple
domain names on the command-line, perhaps.  That would be cool.  Right
now, though, I don't think I have time to patch it; I'm under
time-crunch (which is why I was using -O to begin with). :)

Actually, reading domain names from a file or stdin should be pretty
easy.  Maybe I'll do that.

-- 
Casey Zacek
Senior Engineer
NeoSpire, Inc.

Reply via email to