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

Re: Bug#201234: ITP: esmtp -- User configurable relay-only MTA

José Fonseca <j_r_fonseca@yahoo.co.uk> wrote:
>> It would be nice if the debian-diff to the sources was available
>> somewhere. It is not contained in the normal tarball and I could not
>> find it on sf.net. CVS seems to be overloaded.

> Get it from http://jrfonseca.dyndns.org/debian/esmtp_0.4.1-1.diff.gz

Just browsing over the diff, there is not missing much. :-)

* debian/control:
Depends: ${shlibs:Depends}, debconf
As you are using dh_installdebconf you can replace "debconf" with
${misc:Depends}, which will insert the correct versioned Depends.

Your Build-Depends are insufficient. Afaict you need to add flex,
bison and libesmtp-dev.

* Debconf.
|Quoting esmtprc(5)
| The options format is:
|            keyword=value
| The equal sign is optional and can be replaced by whitespace. The
| value may be enclosed in simple or double quotes, in which case
| special characters can be escaped as in normal C strings.

If you used single-quotes you should be able to  _parse_ this file
(*basically* just source everthing up to the first "identity" tag
after doublechecking quoting and format) in your postinst and do proper
configuration file handling as described in the debconf-devel manpage,
getting rid of the ugly:

| Template: esmtp/overwriteconfig
| Description: Automatically overwrite configuration files?

              tia, cu andreas

/etc/esmtprc should not be publically readable per default.

Reply to: