[Rpm-maint] [rpm-software-management/rpm] Minor test-suite tweaks/fixes, volume III (PR #2568)

[Panu Matilainen]

@pmatilai commented on this pull request.



>      fi
 }
 
 function sandbox()
 {
-    (unset RPM_CONFIGDIR RPM_POPTEXEC_PATH
-     cd ${RPMTEST} && \
-     FAKECHROOT_BASE="${RPMTEST}" fakechroot "$@"
-    )
+    local cmd=$1
+    shift
+    case $cmd in
+        exec)
+            (unset RPM_CONFIGDIR RPM_POPTEXEC_PATH
+             cd ${RPMTEST} && \

Here's another case on unquoted RPMTEST. For whatever reason, it seems to be semi-consistently RPMTEST that is being unquoted. Amusingly, it's quoted on the next line though :smile:

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

Message ID: <rpm-software-management/rpm/pull/2568/review/1517995447 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20230706/3f2cffcc/attachment.html>