[Rpm-maint] [rpm-software-management/rpm] Private /tmp directory for scriptlets (PR #2617)

Panu Matilainen notifications at github.com
Fri Aug 25 08:03:45 UTC 2023


I'm not at all opposed to adding Linux-specific enhancements where they make sense, but we can't rely on them unconditionally. So at the very least this will need conditionalizing on the availability of the relevant unshare() etc facilities, but maybe also a way to disable at runtime even when built in. The need for one may not be obvious now, but pretty much every time I've added I've added  such a feature without an explicit disabler, I've regretted it later :sweat_smile: 

Another, stylistic nit is that the rpm codebase doesn't use `bool` and doing so in this one place only looks just odd, just use a plain old `int` instead. 

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

Message ID: <rpm-software-management/rpm/pull/2617/c1692941379 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20230825/3029c6ba/attachment.html>


More information about the Rpm-maint mailing list