Permalink
Browse files

Return message if mail-server is not installed

- provide a template fragment to expand if no local mail server is installed
  • Loading branch information...
markVnl committed Aug 10, 2018
1 parent 24aa635 commit 55740cae54daf41929ceb1a71089b5b276ee4d7e
@@ -0,0 +1,15 @@
{
if ( -f "/etc/e-smith/db/configuration/defaults/dovecot/type"){
return ' ';
}
$OUT .= qq(
#
# 90 mail-server not installed
#
);
}

2 comments on commit 55740ca

@markVnl

This comment has been minimized.

Show comment
Hide comment
@markVnl

markVnl Aug 10, 2018

Owner

NethServer@24aa635

to mitigate: (not tested yet, for discussion)

the filter package “wants” to expand the/var/lib/nethserver/sieve-scripts/before.sieve as suggested by me;
but if the server package is not installed (lets say you want to run a MTA for a server elsewhere) there is no template to expand for the filter package.

form journalctl

... esmith::event[2650]: expanding /var/lib/nethserver/sieve-scripts/before.sieve
... esmith::event[2650]: ERROR: Cannot create output file //var/lib/nethserver/sieve-scripts/before.sieve.2652 No such file or directory
... esmith::event[2650]:  at /etc/e-smith/events/actions/generic_template_expand line 64.
... esmith::event[2650]: [WARNING] expansion of /var/lib/nethserver/sieve-scripts/b
Owner

markVnl replied Aug 10, 2018

NethServer@24aa635

to mitigate: (not tested yet, for discussion)

the filter package “wants” to expand the/var/lib/nethserver/sieve-scripts/before.sieve as suggested by me;
but if the server package is not installed (lets say you want to run a MTA for a server elsewhere) there is no template to expand for the filter package.

form journalctl

... esmith::event[2650]: expanding /var/lib/nethserver/sieve-scripts/before.sieve
... esmith::event[2650]: ERROR: Cannot create output file //var/lib/nethserver/sieve-scripts/before.sieve.2652 No such file or directory
... esmith::event[2650]:  at /etc/e-smith/events/actions/generic_template_expand line 64.
... esmith::event[2650]: [WARNING] expansion of /var/lib/nethserver/sieve-scripts/b
@DavidePrincipi

This comment has been minimized.

Show comment
Hide comment
@DavidePrincipi

DavidePrincipi Aug 10, 2018

Great idea! Please consider that a static piece of (commented) text does the job: that can simplify the code

We must ensure the path components exist though

DavidePrincipi replied Aug 10, 2018

Great idea! Please consider that a static piece of (commented) text does the job: that can simplify the code

We must ensure the path components exist though

Please sign in to comment.