search

sclorg / spec2scl

MIT License
7 stars 7 forks source link

Add set -ex #17

Closed torsava closed 7 years ago

torsava commented 9 years ago

Originally reported by: Robert Kuska (Bitbucket: xkuska, GitHub: Unknown)

More info https://github.com/fedora-python/python3/issues/2

torsava commented 9 years ago

Original comment by Robert Kuska (Bitbucket: xkuska, GitHub: Unknown):

Also consider wrapping whole sections inside scl enable.

irushchyshyn commented 7 years ago

Whole commands are now wrapped in scl enable and are preceded with set -e according to Packaging Guide.

Closing the issue. Please reopen if necessary.

voxik commented 7 years ago

Sorry, but this does not appear to be fixed:

https://github.com/sclorg/spec2scl/blob/master/spec2scl/settings.py#L18

There is just "set -e" which is not sufficient ...

irushchyshyn commented 7 years ago

Thanks for pointing this out! Fixed in a96bd7e.