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