Re: [PATCH 2/2] Core94: fix for dma when postfix is installed

[Michael Tremer <michael.tremer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 1487 bytes --]

Hi,

the sed command is a bit messy. You can also use a difference character
as a delimiter for the two fields like @. That would look like this:

  sed -e "s(a)usr/bin/sendmail(a)usr/bin/sendmail.postfix(a)g" ...

That makes it more readable.

You can also use the & character in the second half to get the matched
string from the first half. That would look like this:

  sed -e "s(a)usr/bin/sendmail@&.postfix@" ...

...which is even shorter.

I will merge this because you version is working, too.

-Michael

On Thu, 2015-10-15 at 12:18 +0200, Alexander Marx wrote:
> ---
>  config/rootfiles/core/94/update.sh | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/config/rootfiles/core/94/update.sh
> b/config/rootfiles/core/94/update.sh
> index 1b9eed7..7b645e9 100644
> --- a/config/rootfiles/core/94/update.sh
> +++ b/config/rootfiles/core/94/update.sh
> @@ -70,7 +70,13 @@ EOF
>  
>  fcrontab -z &>/dev/null
>  
> -# dma files
> +# DMA - reconfigure Postfix if exists
> +if [ -e /etc/postfix/main.cf ]; then
> +	mv /usr/sbin/sendmail /usr/sbin/sendmail.postfix
> +	/usr/sbin/alternatives --install /usr/sbin/sendmail sendmail
> /usr/sbin/sendmail.postfix 15
> +	sed -i 's/usr\/sbin\/sendmail/usr/sbin/sendmail.postfix/'
> /opt/pakfire/db/rootfiles/postfix
> +fi
> +# DMA - configure dma as default mta
>  mkdir /etc/alternatives
>  mkdir /var/lib/alternatives
>  /usr/sbin/alternatives --install /usr/sbin/sendmail sendmail
> /usr/sbin/sendmail.dma 20

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]