Bug 490246 – sys-apps/sandbox: prefix support

• Status: RESOLVED FIXED
• Product: Portage Development

[Description Benda Xu 2013-11-03 10:08:49 UTC]

Hi, please review the patches of Prefix support for sandbox.

As long as sandbox is a gentoo project, why are we maintaining a long list of patches in FILESDIR?

Reproducible: Always

[Comment 1 Benda Xu 2013-11-03 10:10:36 UTC]

Created attachment 362474 [details, diff]
sandbox-ebuild.patch

EAPI bump.

[Comment 2 Benda Xu 2013-11-03 10:12:14 UTC]

Created attachment 362476 [details, diff]
sandbox-2.6-prefix.patch

substitute /etc, /var and /bin/bash with those of Prefix.

[Comment 3 Benda Xu 2013-11-03 10:16:56 UTC]

configure and Makefile.in are generated from configure.ac and Makefile.am respectively. They are included in the patch only for reference.

[Comment 4 SpanKY 2013-11-27 21:43:41 UTC]

Comment on attachment 362474 [details, diff]
sandbox-ebuild.patch

>+EAPI=5

no.  this cannot exceed 2 as long as portage does not.

[Comment 5 SpanKY 2013-11-27 21:44:30 UTC]

Comment on attachment 362476 [details, diff]
sandbox-2.6-prefix.patch

create proper patches using the sandbox.git repo and `git send-email` them

[Comment 6 Benda Xu 2013-12-16 04:12:42 UTC]

(In reply to SpanKY from comment #5)
> Comment on attachment 362476 [details, diff] [details, diff]
> sandbox-2.6-prefix.patch
> 
> create proper patches using the sandbox.git repo and `git send-email` them

Thanks Mike. Patch sent.

Message-Id: <[e-mail]>

[Comment 7 Benda Xu 2014-01-18 03:15:14 UTC]

(In reply to Benda Xu from comment #6)
> (In reply to SpanKY from comment #5)
> > Comment on attachment 362476 [details, diff] [details, diff] [details, diff]
> > sandbox-2.6-prefix.patch
> > 
> > create proper patches using the sandbox.git repo and `git send-email` them
> 
> Thanks Mike. Patch sent.
> 
> Message-Id: <[e-mail]>

Hi, Mike. In case you missed the last email. I've sent a new one with

Message-Id: <[e-mail]>

[Comment 8 Rick Farina (Zero_Chaos) 2014-02-01 17:21:01 UTC]

The profiles will require EAPI 5 in 30 day, so clinging to old EAPI is pointless now.

This has been an issue for a much longer time than this bug, and there is a patch available.  Where are we on this?  Sandbox is a good thing, I think we want this support in prefix.

[Comment 9 Benda Xu 2014-12-04 04:49:31 UTC]

Hello, Mike. Any update in the reviewing?

[Comment 10 Sergey Popov (RETIRED) 2016-01-29 11:55:52 UTC]

Yet another friendly ping.

@vapier: please, tell us your thoughts about this

[Comment 11 SpanKY 2016-01-29 22:32:00 UTC]

i posted an update to Benda's latest e-mail

[Comment 12 Michał Górny 2017-09-26 10:27:13 UTC]

What's the status here? If there's still some patch not applied, could you resend it since there are new people on the alias?

[Comment 13 Fabian Groffen 2021-01-06 10:19:53 UTC]

I see sandbox in use for Linux RAP (Prefix), so I guess this is sorted these days.

[Comment 14 Nathan Shearer 2021-03-17 06:42:48 UTC]

The make.conf file generated in Prefix still contains a reference to this bug and FEATURES="-sandbox"

Is it save to remove that? Should it also be removed from new make.conf files that are generated by Prefix?