[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#793331: RFS: postsrsd/1.2-1 [ITP]



Hi,

On 18-08-15 01:01, Tomasz Buchert wrote:
great! Just nit-picking here, really. And trying to understand
AppArmor :).
You seem to be right, I've committed a patch to remove the m flag.

Yes, I tried without postconf present and the unit failed.

Its output is silenced, and postsrsd fails
when the -d argument is empty anyway.

I don't think you want this systemd unit to fail *by default*.
Hmm, you're right about that. Would a Recommends be good enough, or do we really need a hard dependency? I'd like to avoid adding a hard dependency only for the systemd unit (since the daemon itself runs fine without postfix), and failing is in my opinion the only reasonable option when neither postconf nor a configured domain is present.

Sorry, I didn't notice that. I agree, though, that it would make sense
to put all configuation in one place.
I looked a bit further into this, and Debian Policy 9.3.2 says that removing the /etc/default file should be supported, so I don't think we can drop the defaults from the systemd unit.

Cheers,
Oxan


Reply to: