Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 634860 - mail-filter/amavisd-new init script doesn't use start-stop-daemon, can't change NICELEVEL
Summary: mail-filter/amavisd-new init script doesn't use start-stop-daemon, can't chan...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal minor (vote)
Assignee: Net-Mail Packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-10-20 08:56 UTC by Marcin Mirosław
Modified: 2018-11-18 23:56 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marcin Mirosław 2017-10-20 08:56:33 UTC
Init script for mail-filter/amavisd-new starts daemon without using start-stop-daemon. So any configuration specific for start-stop-daemon, like SSD_NICELEVEL, is completely ignored.

Reproducible: Always
Comment 1 Michael Orlitzky gentoo-dev 2018-11-15 19:56:27 UTC
Does this work for you?

  https://507352.bugs.gentoo.org/attachment.cgi?id=555284

I'm planning to commit it along with a version bump for amavisd-new-2.11.1.
Comment 2 Marcin Mirosław 2018-11-16 09:23:00 UTC
Michael, I don't use amavisd now (I've changed my job).
Comment 3 Larry the Git Cow gentoo-dev 2018-11-18 23:56:50 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c8fe3982751aa4881b42f89ed080b210c4529c81

commit c8fe3982751aa4881b42f89ed080b210c4529c81
Author:     Michael Orlitzky <mjo@gentoo.org>
AuthorDate: 2018-11-17 15:09:21 +0000
Commit:     Michael Orlitzky <mjo@gentoo.org>
CommitDate: 2018-11-18 23:23:57 +0000

    mail-filter/amavisd-new: new version 2.11.1.
    
    A mostly standard version bump (bug 668494) with some improvements and
    bug fixes piled on:
    
      * The sys-apps/file dependency is replaced by dev-perl/File-LibMagic
        to improve performance a bit (bug 592802).
    
      * New user creation has been moved to pkg_setup(). This allows us to
        set permissions and ownership properly in src_install(), so that
        we don't have to "fix" them later and cause big ol' security
        problems (bug 630836).
    
      * The OpenRC service script has been rewritten to use start-stop-daemon.
        This fixes outstanding bugs 507352, 634860, and 646336.
    
      * The systemd service dependencies have been updated (bug 581452). We
        never really needed spamassassin (spamd) at all, and we shouldn't fail
        to start if postfix/clamav are absent entirely.
    
      * As part of the previous item, removed two failing "sed" calls
        (that were missing die() statements!) intended to modify a service
        file that no longer lives where it used to.
    
    Thanks are due to,
    
      * Marcin Mirosław who reported an OpenRC issue,
      * Nick Wiltshire for reporting and testing the OpenRC fixes,
      * Robin Lutz for reporting the OpenRC restart issue,
      * Timo Rothenpieler for reporting the systemd dependency issue.
    
    Bug: https://bugs.gentoo.org/630836
    Closes: https://bugs.gentoo.org/507352
    Closes: https://bugs.gentoo.org/581452
    Closes: https://bugs.gentoo.org/592802
    Closes: https://bugs.gentoo.org/634860
    Closes: https://bugs.gentoo.org/646336
    Closes: https://bugs.gentoo.org/668494
    Signed-off-by: Michael Orlitzky <mjo@gentoo.org>
    Package-Manager: Portage-2.3.51, Repoman-2.3.11

 mail-filter/amavisd-new/Manifest                   |  1 +
 ...-2.11.0-r4.ebuild => amavisd-new-2.11.1.ebuild} | 68 ++++++++++++----------
 mail-filter/amavisd-new/files/amavisd.initd-r2     | 42 +++++++++++++
 mail-filter/amavisd-new/files/amavisd.service-r1   | 21 +++++++
 4 files changed, 101 insertions(+), 31 deletions(-)