Bug 836671

Summary:	sys-apps/portage adds / to SANDBOX_PREDICT during test phase
Product:	Portage Development	Reporter:	Mike Gilbert <floppym>
Component:	Core - Ebuild Support	Assignee:	Portage team <dev-portage>
Status:	RESOLVED WONTFIX
Severity:	normal	CC:	gentoo
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
See Also:	https://bugs.gentoo.org/show_bug.cgi?id=836560 https://github.com/gentoo/portage/pull/823
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:	839747
Bug Blocks:

Description Mike Gilbert gentoo-dev

2022-04-03 01:34:16 UTC

Portage currently adds the root directory (/) to SANDBOX_PREDICT while executing the src_test function.

https://gitweb.gentoo.org/proj/portage.git/tree/bin/phase-functions.sh?h=portage-3.0.30#n505

This effectively silences any sandbox denials that occur while running tests.

I'm not sure this is really a good idea; it can end up masking file access issues during test execution. For example, see bug 836560.

Comment 1 Mike Gilbert gentoo-dev

2022-04-24 15:42:01 UTC

The fallout from this is too much; it's disturbingly common for test suites to try things like rmdir / with failure as the expected result.