@dmnks commented on this pull request.
> snapshot exec "$@" \
- --define "_buildhost testhost" \
- --define "_topdir /build"
+ --define "_tmppath /var/tmp"
So using `/var/tmp` would work for the "native" (originally `run`) case, too,
however it would violate the rule that no test can influence other tests (since
now they would all share a common `/var/tmp`). I guess let's leave it as is for
now, then. It could be managed somehow, pretty sure, just not something to
block this PR on.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3447#discussion_r1846352083
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/3447/review/2442242...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
https://lists.rpm.org/mailman/listinfo/rpm-maint
