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
======================================================================