[PR #1984] [MERGED] Re-re-re-fix propagating the return code #2282

New issue

Closed

opened 2026-03-04 02:04:43 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 02:04:43 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/1984
Author: @gaul
Created: 7/16/2022
Status: ✅ Merged
Merged: 7/16/2022
Merged by: @ggtakec

Base: master ← Head: test/errexit

📝 Commits (1)

d60571d Re-re-re-fix propagating the return code

📊 Changes

1 file changed (+13 additions, -2 deletions)

View changed files

📝 test/test-utils.sh (+13 -2)

📄 Description

This shell incantation is necessary to have errexit take effect for
test bodies. Previous only the last command's exit code was
propagated to the test runner, masking real failures. Reverts
ea3c21f270.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/s3fs-fuse/s3fs-fuse/pull/1984 **Author:** [@gaul](https://github.com/gaul) **Created:** 7/16/2022 **Status:** ✅ Merged **Merged:** 7/16/2022 **Merged by:** [@ggtakec](https://github.com/ggtakec) **Base:** `master` ← **Head:** `test/errexit` --- ### 📝 Commits (1) - [`d60571d`](https://github.com/s3fs-fuse/s3fs-fuse/commit/d60571d6d037e18c470500cc51e518bbe22af201) Re-re-re-fix propagating the return code ### 📊 Changes **1 file changed** (+13 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `test/test-utils.sh` (+13 -2) </details> ### 📄 Description This shell incantation is necessary to have errexit take effect for test bodies. Previous only the _last_ command's exit code was propagated to the test runner, masking real failures. Reverts ea3c21f270db3c3298ba9b7d04e61146c712e9bb. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>