★ wanayoo — archive 1999 https://github.com/tomav/docker-mailserver/pull/573Nouvelle recherche | Portail wanayoo
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mention submodule dependency #573

Merged
merged 1 commit into from Apr 17, 2017
Merged

Conversation

@arneke
Copy link
Contributor

@arneke arneke commented Apr 15, 2017

Mention that you have to manually initialize and update submodules, otherwise you find out after a very long wait.

Mention that the integration tests will not work unless you manually initialize and update submodules
@kiliant
Copy link
Contributor

@kiliant kiliant commented Apr 15, 2017

👍 haha had to find that out myself.

ci should be just fine, i had those "hickups" as well. maybe @tomav can restart it

@arneke
Copy link
Contributor Author

@arneke arneke commented Apr 15, 2017

Actually, I had intermittent issues with that PGP key as well, the one that killed the build, as well as multiple failures in a row with db.local.clamav.net (round robin DNS afaik). So I have been trying to get clean build for two days now.

Pretty sure it's not an issue with the server I am building this on, it's a bit dated, but on a solid connection. Making the build more resilient, or making a base image to build on top of, may qualify for a separate ticket. But I haven't looked into what you have discussed previously.

@tomav
Copy link
Owner

@tomav tomav commented Apr 15, 2017

Restarted

@tomav tomav merged commit 4e842c7 into tomav:master Apr 17, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@tomav
Copy link
Owner

@tomav tomav commented Apr 17, 2017

Thanks

RichardFevrier pushed a commit to RichardFevrier/docker-mailserver that referenced this pull request Aug 26, 2019
Mention that the integration tests will not work unless you manually initialize and update submodules
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.