search

Lightstreamer / Docker

Docker Official Image packaging for Lightstreamer Server https://www.lightstreamer.com
Apache License 2.0
3 stars 2 forks source link

Add "set -e" so semicolons DTRT #3

Closed tianon closed 6 years ago

tianon commented 6 years ago

https://github.com/docker-library/official-images/pull/4065 looks fine, but will only fail during problems as a side effect (package install failing will cause the downloads to fail in the second semicolon'd expression, for example), so this simply closes that loophole (and allows for ; and && to be used mostly interchangeably).

$ help set | grep '  -e'
      -e  Exit immediately if a command exits with a non-zero status.
gfinocchiaro commented 6 years ago

Thanks a lot for such advice.