Bug 107479 – virtualx.eclass - SANDBOX_DISABLED is going away

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Brian Harring (RETIRED) 2005-09-28 01:29:15 UTC]

See dev ml regarding details; either use SANDBOX_ON within the specific phases
you need to disable sandbox (kindly flipping it back to it's previous state), or
set a RESTRICT="sandbox"
SANDBOX_DISABLED at the global level is effectively the restrict.

Ebuilds affected-
app-emacs/erc-cvs-4.1
app-emacs/gnu-cvs-5.11
app-editors/emacs-*
app-editors/emacs-cvs-*
app-editors/xemacs-gtk-*
app-editors/xemacs-*

So... kindly fix 'em up, cause I'd like to nuke SANDBOX_DISABLED sooner rather
then later. :)
mattam, assigning this to you since you seem to be active (usata I think is
active also), if I completely missed the boat on who's active or not, pardon. :)

[Comment 1 Brian Harring (RETIRED) 2005-12-26 01:13:39 UTC]

*bump*, anything on this?

[Comment 2 Christian Faulhammer (RETIRED) 2006-10-30 22:03:22 UTC]

(In reply to comment #1)
> *bump*, anything on this?
> 

 Sure, all packages are either nonexistant anymore or fixed.  So you can close the bug, I remove emacs from cc.

[Comment 3 Bryan Østergaard (RETIRED) 2007-04-07 11:59:10 UTC]

virtualx.eclass still needs to be fixed.

[Comment 4 Mark Loeser (RETIRED) 2008-01-01 01:06:59 UTC]

(In reply to comment #3)
> virtualx.eclass still needs to be fixed.
> 

So...I'm thinking about ripping out the global turning off of sandbox and testing all of the ebuilds that use the eclass quickly to make sure nothing breaks.  The few I looked at just seem to use Xmake for src_test.

Sound sane or am I missing something?

[Comment 5 Jakub Moc (RETIRED) 2008-02-28 10:11:34 UTC]

This clearly won't move an inch further with m-needed as assignee. And yeah, loosing bunch of apparently sucky tests is a whole lot better than disabling sandbox in global scope.

[Comment 6 Tomáš Chvátal (RETIRED) 2009-11-22 23:31:27 UTC]

Why noone asked the maintainers...

[Comment 7 Larry the Git Cow 2022-08-02 04:04:10 UTC]

The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=aa7fdc965621b413d85624ea30e9452a4130aced

commit aa7fdc965621b413d85624ea30e9452a4130aced
Author:     Michał Górny <[e-mail]>
AuthorDate: 2022-07-27 07:10:41 +0000
Commit:     Michał Górny <[e-mail]>
CommitDate: 2022-08-02 04:03:42 +0000

    virtualx.eclass: Do not disable sandbox
    
    The code responsible for disabling sandbox dates back to 2002 and does
    not include any explanation.  Let's try if we still need it.
    
    Bug: https://bugs.gentoo.org/107479
    Signed-off-by: Michał Górny <[e-mail]>

 eclass/virtualx.eclass | 7 -------
 1 file changed, 7 deletions(-)