[Bug 188400] Review Request: ssmtp

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ssmtp


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=188400





------- Additional Comments From pertusus at free.fr  2006-04-11 10:51 EST -------
(In reply to comment #6)

> * the 'Provides: smtpdaemon' is right; afais, it is used by other
>   packages to express a requirement on a sendmail compatible CLI
>   tool. Interpreting it as something which listens on a TCP port does
>   not make sense because the daemon might be running on another host
>   which can not be controlled by the local package-manager

It is not exactly what I concluded after that:

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=66396
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=165957

There seems to be some disension and confusion on that matter. I 
don't have a firm opinion on that, but the fetchmail case seems to 
indicate that smtpdaemon is used to indicate a smtp daemon running
locally on port 25. I agree that it doesn't make that much sense 
because having such a daemon installed doesn't mean that it is 
actually started, nor that it has not been tuned to ignore messages 
for some users, or listen on another port (or not listen at all...). 
If there isn't such package, however, it is certain that there is 
no local smtp daemon...

>   | %install 
>   | rm -rf $RPM_BUILD_ROOT 
>   | %makeinstall etcdir='$(sysconfdir)' GEN_CONFIG=/bin/true
mandir="$RPM_BUILD_ROOT%_mandir/man8" 
>   | install -p -m644 ssmtp.conf $RPM_BUILD_ROOT%_sysconfdir/ssmtp/ssmtp.conf 

This should work. A longuer version with an explicit overwritting 
of used variables on a 
make install
line (instead on %makeinstall) would be even more suitable, in my 
opinion, given that the Makefile.in is done by hand. But 
%makeinstall and specific overriding seems fine to me too.
Something along (haven't tested)

make install bindir=%{buildroot}%{_sbindir} \
  etcdir=%{buildroot}%{_sysconfdir} GEN_CONFIG=/bin/true
     mandir="%{buildroot}%_mandir/man8"

>   I does not include your alternatives symlinks but is nevertheless a
>   little bit shorter.

There are also some more manpages, and the following lines are still usefull, do
it must be longer anyway :-/:

mkdir  %{buildroot}%{_bindir}/
install -D -m 644 debian/mailq.8 %{buildroot}%{_mandir}/man1/mailq.ssmtp.1
install -m 644 debian/newaliases.8 %{buildroot}%{_mandir}/man1/newaliases.ssmtp.1
install -D -m 644 ssmtp.conf.5 %{buildroot}%{_mandir}/man5/ssmtp.conf.5
ln -s %{_sbindir}/ssmtp %{buildroot}%{_sbindir}/sendmail.ssmtp
ln -s %{_sbindir}/ssmtp %{buildroot}%{_bindir}/newaliases.ssmtp
ln -s %{_sbindir}/ssmtp %{buildroot}%{_bindir}/mailq.ssmtp



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.