Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226407 --- Comment #65 from Paul Howarth <paul@xxxxxxxxxxxx> 2010-08-09 10:47:03 EDT --- (In reply to comment #64) > >The "touch" and initial "chmod" parts of "create db ghosts" in %install are > >redundant because the files were created in the immediately preceding part of > >the spec, and with the correct permissions. > > Looks OK for me (line 395+). The previous parts are for sources, these are for > .db. Quite right. > >%description for the doc subpackage needs rewriting. > > I did my best, but english is not my native language. Feel free to propose > something more appropriate. I'd suggest this: This package contains the Sendmail Installation and Operation Guide (PDF), text files containing configuration documentation, plus a number of contributed scripts and tools for use with Sendmail. A couple of other quick points: * the "README.redhat" (newconfig patch) describes changes that happened for sendmail 8.12.x back in July 2002 and reflects what is now standard practice in Sendmail installations. I wouldn't have thought it was needed any more. * README.sendmail doesn't appear to contain anything of use to people installing the package. Robert, do you have any objections to this package being approved now? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review