Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 391965 - dev-db/postgresql-base: sys-apps/sandbox in ebuild dependencies
Summary: dev-db/postgresql-base: sys-apps/sandbox in ebuild dependencies
Status: RESOLVED FIXED
Alias: None
Product: Gentoo/Alt
Classification: Unclassified
Component: FreeBSD (show other bugs)
Hardware: x86 FreeBSD
: Normal normal (vote)
Assignee: PgSQL Bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-26 13:42 UTC by Denis I. Polukarov
Modified: 2011-12-27 13:59 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Denis I. Polukarov 2011-11-26 13:42:59 UTC
Cannot emerge dev-db/postgresql-server, ebuild's dependency is sys-apps/sandbox.

Reproducible: Always

Steps to Reproduce:
1. emerge dev-db/postgresql-server
2.
3.
Actual Results:  
Cannot emerge dev-db/postgresql-server.

Expected Results:  
Emerge dev-db/postgresql-server without sys-apps/sandbox.
Comment 1 Aaron W. Swenson gentoo-dev 2011-11-26 17:08:01 UTC
The dependency on sys-apps/sandbox was added to resolve bug 322597. That dependency will not be removed for a couple more years.
Comment 2 Naohiro Aota gentoo-dev 2011-12-02 05:08:08 UTC
(In reply to comment #1)
> The dependency on sys-apps/sandbox was added to resolve bug 322597. That
> dependency will not be removed for a couple more years.

Could you consider adding "kernel_linux?" (or elibc_glibc? ?) to the depend? >=sys-apps/sandbox-2.0 doesn't work on FreeBSD and at least USE=threads emerge -1 postgresql-base pass the thread safety test and also built well with sandbox enabled on FreeBSD, so it seems there's no reason to depend on sandbox.

# FEATURES=sandbox USE=threads emerge -1 postgresql-base
...
checking thread safety of required library functions... yes
...

# emerge -pv sandbox
[ebuild   R    ] sys-apps/sandbox-1.6-r2  0 kB

Also are there any way I can do to test if thread safety is working or not?
Comment 3 Aaron W. Swenson gentoo-dev 2011-12-10 15:20:12 UTC
(In reply to comment #2)
> (In reply to comment #1)
> > The dependency on sys-apps/sandbox was added to resolve bug 322597. That
> > dependency will not be removed for a couple more years.
> 
> Could you consider adding "kernel_linux?" (or elibc_glibc? ?) to the depend?
> >=sys-apps/sandbox-2.0 doesn't work on FreeBSD and at least USE=threads emerge
> -1 postgresql-base pass the thread safety test and also built well with sandbox
> enabled on FreeBSD, so it seems there's no reason to depend on sandbox.
> 
> # FEATURES=sandbox USE=threads emerge -1 postgresql-base
> ...
> checking thread safety of required library functions... yes
> ...
> 
> # emerge -pv sandbox
> [ebuild   R    ] sys-apps/sandbox-1.6-r2  0 kB
> 
> Also are there any way I can do to test if thread safety is working or not?

Bug 322597 is a tricky bug. Machines that look otherwise identical will not have the same experience. So, while 1.6-r2 works for you, someone else with an apparent same setup will experience a failure.

I'm not opposed to adding a conditional upon the dependency. But, I cannot say whether the failure is due to either the it's the kernel or the library that's the root cause. I don't want to use the wrong conditional and have that bug get unsquished.

You can grab the C source code attachment on bug 322597 to test the thread safety of libpq.
Comment 4 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2011-12-16 20:49:07 UTC
sandbox isn't supported in Gentoo Prefix and definately not on non-linux.

A nice solution to "fix" this for Gentoo Prefix would be "!<sys-apps/sandbox-2" - Please consider, Aaron. Thanks.
Comment 5 Aaron W. Swenson gentoo-dev 2011-12-27 13:59:06 UTC
  27 Dec 2011; Aaron W. Swenson <titanofold@gentoo.org>
  postgresql-base-8.2.23.ebuild, postgresql-base-8.3.17.ebuild,
  postgresql-base-8.4.10.ebuild, postgresql-base-9.0.6.ebuild,
  postgresql-base-9.1.2.ebuild:
  Changed sandbox dependency to block less than 2.0 rather than pull in greater
  than 2.0. Fixes bug 391965.