Use `&&` instead of `;` to properly combine commands in Dockerfile

DNSCrypt / dnscrypt-server-docker

A Docker image for a non-censoring, non-logging, DNSSEC-capable, DNSCrypt-enabled DNS resolver

https://dnscrypt.info

ISC License

670 stars 135 forks source link

Closed PeterDaveHello closed 3 years ago

PeterDaveHello commented 3 years ago

I believe they should be && but not ;, as the previous one should success, and the following command should be executed.

jedisct1 commented 3 years ago

Thanks!

The intent was to keep going if there was nothing to update. But apparently these commands exit with 0 even if they didn't do anything.