| From: | Thomas Munro <tmunro(at)postgresql(dot)org> |
|---|---|
| To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | pgsql: ci: Add missing "set -e" to scripts run by su. |
| Date: | 2025-11-06 01:28:35 |
| Message-ID: | E1vGonj-005Dcc-2I@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
ci: Add missing "set -e" to scripts run by su.
If any shell command fails, the whole script should fail. To avoid
future omissions, add this even for single-command scripts that use su
with heredoc syntax, as they might be extended or copied-and-pasted.
Extracted from a larger patch that wanted to use #error during
compilation, leading to the diagnosis of this problem.
Reviewed-by: Tristan Partin <tristan(at)partin(dot)io> (earlier version)
Discussion: https://postgr.es/m/DDZP25P4VZ48.3LWMZBGA1K9RH%40partin.io
Backpatch-through: 15
Branch
------
REL_16_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/4da1f66fae836c4579ea38fe6d1d8fa2e6e9faf2
Modified Files
--------------
.cirrus.tasks.yml | 12 ++++++++++++
1 file changed, 12 insertions(+)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Thomas Munro | 2025-11-06 01:28:48 | pgsql: ci: Add missing "set -e" to scripts run by su. |
| Previous Message | Thomas Munro | 2025-11-06 01:28:19 | pgsql: ci: Add missing "set -e" to scripts run by su. |