Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 528032 - mail-filter/postgrey - add systemd service files
Summary: mail-filter/postgrey - add systemd service files
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Server (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Stefan G. Weichinger
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: install-systemd-unit
  Show dependency tree
 
Reported: 2014-11-02 18:51 UTC by Stefan G. Weichinger
Modified: 2015-10-03 23:47 UTC (History)
5 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 Stefan G. Weichinger 2014-11-02 18:51:02 UTC
Gentoo doesn't bring systemd service files for postgrey.

Suggested file postgrey.service:

[Unit]
Description=Postfix Greylisting Service
Before=postfix.service

[Service]
Type=simple
ExecStart=/usr/sbin/postgrey --inet=127.0.0.1:10030 \
	--pidfile=/var/run/postgrey.pid \
	--group=postgrey --user=postgrey \
	--greylist-text="Greylisted for %%s seconds"
Restart=always
RestartSec=5
TimeoutSec=10

[Install]
WantedBy=multi-user.target


Reproducible: Always
Comment 1 Stefan G. Weichinger 2015-02-23 22:28:26 UTC
This bug is nearly 5 months old. pls act.
Comment 2 Stefan G. Weichinger 2015-09-18 09:15:38 UTC
BUMP
Comment 3 Manuel Rüger (RETIRED) gentoo-dev 2015-09-18 12:19:41 UTC
@qnikst: Could you check if your proxied maintainer is still active?

@systemd: Could you check the systemd unit for correctness?
Comment 4 Ian Delaney (RETIRED) gentoo-dev 2015-09-22 14:42:14 UTC
(In reply to Stefan G. Weichinger from comment #2)
> BUMP

I'll try get this in soon considering it has also slipped thru the cracks. Response from @systemd would be in order.
Comment 5 Pacho Ramos gentoo-dev 2015-09-22 19:06:58 UTC
The .service file looks ok to me
Comment 6 Ian Delaney (RETIRED) gentoo-dev 2015-09-23 14:11:00 UTC
Stefan G. Weichinger do you wish to be substitute proxy or co-maintainer? No sign of activity by designated maintainer.
Comment 7 Ian Delaney (RETIRED) gentoo-dev 2015-09-23 14:33:18 UTC
commit c8244c3de06e6a1b6ba8d3d179b1f77d39fce964
Author: Ian Delaney <idella4@gentoo.org>
Date:   Wed Sep 23 22:29:18 2015 +0800

    mail-filter/postgrey: revbump to 1.35-r2
    
    Inherit systemd eclass, add service for systemd, rm old versions,
    ack to Stefan G. Weichinger for submitting service file,
    fixes Bug 528032
Comment 8 Stefan G. Weichinger 2015-09-23 15:36:19 UTC
(In reply to Ian Delaney from comment #6)
> Stefan G. Weichinger do you wish to be substitute proxy or co-maintainer? No
> sign of activity by designated maintainer.

I can only try to support stuff like that service file and maybe updated ebuild --> when I check upstream there is 1.36 available, I can test the ebuild for that on my server.

I don't know any internals of postgrey ...
Comment 9 Ian Delaney (RETIRED) gentoo-dev 2015-09-26 09:19:29 UTC
That's enough to start with. Is that a yes?
Comment 10 Stefan G. Weichinger 2015-09-26 10:46:11 UTC
(In reply to Ian Delaney from comment #9)
> That's enough to start with. Is that a yes?

yes
Comment 11 Stefan G. Weichinger 2015-09-26 11:45:57 UTC
not exactly belonging here, but ...

copied the postgrey-1.35-r2.ebuild to postgrey-1.36.ebuild locally ... merged fine and runs withouth problems so far.
Comment 12 Ian Delaney (RETIRED) gentoo-dev 2015-10-03 09:36:59 UTC
commit 39c4aff1f0ba28dcf32860b3fe56fe604d46c44e
Author: Ian Delaney <idella4@gentoo.org>
Date:   Sat Oct 3 17:32:15 2015 +0800

    mail-filter/postgrey:  bump to -1.36, remove outdated -1.35-r1
    
    add new maintainer under direction of proxy-maint herd, request
    for bump from new maintainer wrt gentoo bug
    
    Gentoo bug: #528032

The maintainer Andreis Vinogradovs has been inactive and is prone now to removal from metadata.xml
Comment 13 Ian Delaney (RETIRED) gentoo-dev 2015-10-03 23:47:54 UTC
I don't see how this package or bug depends on bug 529188 given it is not in (R)DEPEND. Shifted to See Also: