[Rpm-maint] [rpm-software-management/rpm] Ensure tests/setup.sh is only ever run inside the test-image (PR #3552)

Michal Domonkos notifications at github.com
Fri Jan 31 16:01:44 UTC 2025


@dmnks requested changes on this pull request.



>  
-mkdir -p $DESTDIR/build
-ln -sf ../data/SOURCES $DESTDIR/build/
+# Make sure nobody runs this accidentally outside the environment
+if [ ! /.rpmtestsuite ]; then

Oops, this is missing an `-f`. Without it, the conditional is always false (i.e. the file isn't actually required to exist) :sweat_smile:

It's funny because this is the kind of thing that you just don't want to test locally :smile: 

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3552#pullrequestreview-2587163374
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3552/review/2587163374 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20250131/9cd9a1e9/attachment.htm>


More information about the Rpm-maint mailing list