Skip to content

Configuration support for /etc/aliases, see issue #988#1065

Merged
johansmitsnl merged 4 commits intodocker-mailserver:masterfrom
peter-hartmann:master
Nov 1, 2018
Merged

Configuration support for /etc/aliases, see issue #988#1065
johansmitsnl merged 4 commits intodocker-mailserver:masterfrom
peter-hartmann:master

Conversation

@peter-hartmann
Copy link
Copy Markdown
Contributor

@peter-hartmann peter-hartmann commented Oct 28, 2018

What it does:
[x] add postfix-aliases.cf to ./config (/tmp/docker-mailserver)
[x] in check-for-changes.sh monitor postfix-aliases.cf and apply changes to /etc/aliases
[x] in start-mailserver.sh apply postfix-aliases.cf to /etc/aliases
This solution addresses issue #988 .

@johansmitsnl
Copy link
Copy Markdown
Contributor

Restarted the build, to see if it was a "busy" error.

@johansmitsnl
Copy link
Copy Markdown
Contributor

@peter-hartmann do tests pass on your side? Now the same tests failed on Travis.

@peter-hartmann
Copy link
Copy Markdown
Contributor Author

@johansmitsnl Test fail on my side too for both my repo and yours.
E.g.

git clone --recurse-submodules https://github.com/tomav/docker-mailserver.git
cd docker-mailserver
make
...
...
...
 ✓ checking relay hosts: default auth entry is added
 ✓ checking that mail for root was delivered

192 tests, 1 failure
Makefile:285: recipe for target 'tests' failed
make: *** [tests] Error 1

Any idea?

@johansmitsnl johansmitsnl merged commit 30ed8fb into docker-mailserver:master Nov 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants