Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

phase-functions.sh: do not "addpredict /" for src_test #823

Closed
wants to merge 1 commit into from

Conversation

floppym
Copy link
Contributor

@floppym floppym commented Apr 20, 2022

Silencing the sandbox by default makes it more difficult to track down
file access problems that may effect tests.

Removing this will likely cause an increased number of test failures
until affected ebuilds can be adjusted.

Bug: https://bugs.gentoo.org/836671

@floppym
Copy link
Contributor Author

floppym commented Apr 21, 2022

Before this gets merged, we should ensure that most of the bugs blocking bug 839747 have been addressed.

Silencing the sandbox by default makes it more difficult to track down
file access problems that may effect tests.

Removing this will likely cause an increased number of test failures
until affected ebuilds can be adjusted.

Bug: https://bugs.gentoo.org/836671
Signed-off-by: Mike Gilbert <floppym@gentoo.org>
@floppym
Copy link
Contributor Author

floppym commented Apr 24, 2022

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.

Closing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants