Skip to content

Mention submodule dependency#573

Merged
tomav merged 1 commit intodocker-mailserver:masterfrom
arneke:mention-submodule-dependency
Apr 17, 2017
Merged

Mention submodule dependency#573
tomav merged 1 commit intodocker-mailserver:masterfrom
arneke:mention-submodule-dependency

Conversation

@arneke
Copy link
Copy Markdown
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
Copy Markdown
Contributor

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
Copy Markdown
Contributor Author

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
Copy Markdown
Contributor

tomav commented Apr 15, 2017

Restarted

@tomav tomav merged commit 4e842c7 into docker-mailserver:master Apr 17, 2017
@tomav
Copy link
Copy Markdown
Contributor

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants