måndagen den 26 januari 2004 19.37 skrev Tom Collins:
> On Jan 26, 2004, at 11:26 AM, Oden Eriksson wrote:
> > Ok. will do.
> >
> > Maybe qmailadmin should obey ~/etc/vlimits.default ?
>
> It does -- that's where it gets the default quota when adding a new
> user.

Oh!, I didn't know that, very cool!

> > nuked my db and started fresh. This time the postmaster account was
> > getting
> > the default 20mb quota I defined in ~/etc/vlimits.default, nice!. I
> > didn't
> > had any values in there before. And let me emphasize I'm not used to
> > mysql
> > auth at all..., this is actually my second go at it.
> >
> > Yes I used your patch, but had to apply it by hand.
>
> My guess is that you made a mistake when applying the patch.

Yes, I did, he he he..., must have been tired or stressed, sorry about that:

--- qmailadmin-1.2.0-rc3.old/user.c     2004-01-26 07:34:14.000000000 +0100
+++ qmailadmin-1.2.0-rc3/user.c 2004-01-26 19:43:19.000000000 +0100
@@ -476,15 +476,14 @@

 #ifdef MODIFY_QUOTA
     if (strcmp (Quota, "NOQUOTA") == 0) {
-       vsetuserquota (Newu, Domain, "NOQUOTA");
+      vsetuserquota (Newu, Domain, "NOQUOTA");
     } else if ( Quota[0] != 0 ) {
       if(quota_to_bytes(qconvert, Quota)) {
         sprintf(StatusMessage, get_html_text("314"));
       } else {
-         vsetuserquota (Newu, Domain, qconvert);
+        vsetuserquota (Newu, Domain, qconvert);
       }
     }
-         vsetuserquota (Newu, Domain, qconvert);
 #endif

 #ifdef MODIFY_SPAM
@@ -497,13 +496,13 @@
     }
 #endif

-     /* report success */
-     sprintf(StatusMessage, "%s [EMAIL PROTECTED] (%s) %s",
-       get_html_text("002"), Newu, Domain, Gecos,
-       get_html_text("119"));
+    /* report success */
+    sprintf(StatusMessage, "%s [EMAIL PROTECTED] (%s) %s",
+      get_html_text("002"), Newu, Domain, Gecos,
+      get_html_text("119"));

   } else {
-     /* otherwise, report error */
+    /* otherwise, report error */
     sprintf(StatusMessage, "<font color=\"red\">%s [EMAIL PROTECTED] (%s) %s</font>",
       get_html_text("002"), Newu, Domain, Gecos, get_html_text("120"));
   }


> > I would grab from cvs but your latest change isn't in there.
>
> I just added it.  I'll attach it as a gz file to another post as well.

Thanks! I saw the cvs changelog 10 seconds after my last mail.

Now it works, thanks Tom!


Reply via email to