On Monday, May 1, 2023 1:01:17 PM EDT Einhard Leichtfuß wrote: > On 01/05/2023 18:14, Scott Kitterman wrote: > > On Monday, May 1, 2023 11:06:07 AM EDT Einhard Leichtfuß wrote: > >> Package: postfix ... > >> In `main.cf`, the following lines were appended: > >>> readme_directory = /usr/share/doc/postfix > >>> html_directory = /usr/share/doc/postfix/html > >> > >> If I understand the postinst script correctly, this modification of > >> `main.cf` should only have happened upon first installation, which this > >> was not. I was unable to reproduce this. So maybe this modification > >> was indeed done earlier. > >> > >> However, even upon initial installation (with pre-existing > >> configuration), this should, in my opinion, not happen. ... > > Also, note that the message about is about main.cf not being modified. > > These changes are in master.cf, so I don't understand the concern with > > the message? > The second modification (readme_directory, html_directory) was to > `main.cf`. While this modification should only happen for initial > installations (with pre-existing configuration), the message is > displayed even then. > > Steps to reproduce (assuming postfix is not installed): > > $ apt install postfix-doc > $ echo > /etc/postfix/main.cf > $ apt install postfix
To focus in on the main.cf part of this, I believe that's per policy. First, it's a change made by postfix-doc, not postifx, so the postfix package statement that main.cf was not modified by it is correct and unrelated to the main.cf change. For the postfix-doc change to main.cf, Policy 10.7.4 is the relevant portion. Postfix-doc uses the provided interface (postfconf), when available. I checked and this goes back at least to when the postfix packaging was first kept in git in 2007. I think this part is not a bug. Please let me know if I'm misunderstanding the issue. I suspect the master.cf fix_master can be removed entirely, but I'm not 100% certain yet. Scott K
signature.asc
Description: This is a digitally signed message part.
