The following bug has been CLOSED
======================================================================
http://dbmail.org/mantis/bug_view_advanced_page.php?bug_id=0000103
======================================================================
Reported By:                blake
Assigned To:                aaron
======================================================================
Project:                    DBMail
Bug ID:                     103
Category:                   Command-Line programs (dbmail-users, dbmail-util)
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
======================================================================
Date Submitted:             15-Oct-04 09:18 CEST
Last Modified:              13-Dec-04 11:07 CET
======================================================================
Summary:                    dbmail-util -q isn't quiet
Description: 
running dbmail-util with the -q option has no effect
======================================================================

----------------------------------------------------------------------
 aaron - 15-Oct-04 18:16 CEST 
----------------------------------------------------------------------
Quite right. Most of the printf's were not changed over to the quiet printf
macro.

----------------------------------------------------------------------
 blake - 19-Oct-04 01:47 CEST 
----------------------------------------------------------------------
Here's a diff vs: 2.0.0 that appears to fix evertything for me. Aaron, you
may want to look this over before commiting it. All my comments begin with
"BAM" and I put one everywhere I had a question.

----------------------------------------------------------------------
 aaron - 31-Oct-04 12:36 CET 
----------------------------------------------------------------------
This patch is spot-on. The stderr stuff I just put into the ordinary
qprintf() because it's nothing too special. Nice work! Sorry it took me so
long to get to it :-[

----------------------------------------------------------------------
 blake - 30-Nov-04 00:59 CET 
----------------------------------------------------------------------
Thanks for fixing this Aaron, unfortunately one bad line crept into 2.0.1:

--- maintenance.c.orig  Mon Nov 29 18:51:18 2004
+++ maintenance.c       Mon Nov 29 18:55:17 2004
@@ -373,8 +373,11 @@
                return -1;
        }

-       qerrorf("Ok. Found [%ld] physmessages without messageblocks.\n",
-            lostlist.total_nodes);
+       if (lostlist.total_nodes > 0)
+               qerrorf("Ok. Found [%ld] physmessages without
messageblocks.\n",
+                       lostlist.total_nodes);
+       else
+               qprintf("Ok. Found no physmessages without
messageblocks.\n");

        if (lostlist.total_nodes > 0) {
                el = lostlist.start;

----------------------------------------------------------------------
 aaron - 30-Nov-04 03:00 CET 
----------------------------------------------------------------------
Basically you want absolutely no output from dbmail-util -q unless
something really is wrong. Is this the only message that made it through?

----------------------------------------------------------------------
 blake - 30-Nov-04 21:16 CET 
----------------------------------------------------------------------
Yes, that's exactly what I want. And yes, that's the only one coming
through right now. My situation is that I run dbmail-util -aq in cron once
a day, and I don't want to receive an email from cron unless something is
wrong.

----------------------------------------------------------------------
 aaron - 30-Nov-04 21:34 CET 
----------------------------------------------------------------------
And I left NULL messages out. So a more complete fix is now in CVS for
dbmail_2_0_branch. I'll keep the bug open until I get this into HEAD, too.

----------------------------------------------------------------------
 aaron - 07-Dec-04 01:09 CET 
----------------------------------------------------------------------
Ok, it's in HEAD now, too.

Bug History
Date Modified  Username       Field                    Change              
======================================================================
15-Oct-04 09:18blake          New Bug                                      
15-Oct-04 18:16aaron          Assigned To               => aaron           
15-Oct-04 18:16aaron          Status                   new => assigned     
15-Oct-04 18:16aaron          Bugnote Added: 0000313                       
19-Oct-04 01:47blake          File Added: maintenance.diff                    
19-Oct-04 01:47blake          Bugnote Added: 0000316                       
31-Oct-04 12:36aaron          Bugnote Added: 0000326                       
31-Oct-04 12:36aaron          Resolution               open => fixed       
31-Oct-04 12:36aaron          Status                   assigned => resolved
30-Nov-04 00:59blake          Bugnote Added: 0000389                       
30-Nov-04 00:59blake          Resolution               fixed => reopened   
30-Nov-04 00:59blake          Status                   resolved => feedback
30-Nov-04 03:00aaron          Bugnote Added: 0000390                       
30-Nov-04 21:16blake          Bugnote Added: 0000400                       
30-Nov-04 21:34aaron          Bugnote Added: 0000401                       
07-Dec-04 01:09aaron          Bugnote Added: 0000426                       
07-Dec-04 01:09aaron          Resolution               reopened => fixed   
07-Dec-04 01:09aaron          Status                   feedback => resolved
13-Dec-04 11:07ilja           Status                   resolved => closed  
======================================================================

Reply via email to