On Thu, Jan 11, 2024 at 09:20:22AM -0800, Darrick J. Wong wrote: > > + mountpoint $mountpoint >/dev/null || echo "$mountpoint is not mounted" > > The helper needs to return nonzero on failure, e.g. > > if ! mountpoint -q $mountpoint; then > echo "$mountpoint is not mounted" > return 1 > fi No, it doesn't.. I actually did exactly that first, but that causes the test to be _notrun instead of reporting the error and thus telling the author that they usage of this helper is wrong.
