Bug 326967 – dev-lang/php-5.3.2 "-threads" blocked by postgresql-base "+threads"

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Holger Hoffstätte 2010-07-05 09:49:59 UTC]

php-5.3.2 with USE="-threads" forces apache to be "-threads" as well (correct: enforce prefork model) but also does the same with postgresql-base. This is not necessary, as a threadsafe libpq works fine with a single-threaded php.


Reproducible: Always

Steps to Reproduce:
1. emerge postgresql-base with +threads (make libpq "threadsafe")
2. try to emerge php-5.3.2 with -threads (for use in prefork mode)

Actual Results:  
holger>emerge -vp php
 
These are the packages that would be merged, in order:
 
Calculating dependencies... done!
 
emerge: there are no ebuilds built with USE flags to satisfy ">=dev-db/postgresql-base-7.1[-threads]".
!!! One of the following packages is required to complete your request:
- dev-db/postgresql-base-9.0_beta2-r1 (Change USE: -threads)
(dependency required by "dev-lang/php-5.3.2" [ebuild])
(dependency required by "php" [argument])
 


Expected Results:  
emerged php-5.3.2

In this case statements like:

    >=dev-db/postgresql-base-7.1[threads=]

should be changed to enforce +threads when php is built with +threads (enforcing safety for in-process concurrency), but also allow a dependency with +threads when -threads is set (safety not required for prefork model).

[Comment 1 Holger Hoffstätte 2010-07-05 09:51:42 UTC]

Btw this is unrelated to bug#299964.

[Comment 2 Matti Bickel (RETIRED) 2010-07-05 10:33:42 UTC]

Sounds reasonable. I'll dig into that and commit a fix shortly if appropiate.

[Comment 3 Aaron W. Swenson 2010-07-07 13:29:23 UTC]

In the dev-lang/php-5.3.2 ebuild:

  postgres? (
    || (
      >=dev-db/postgresql-base-7.1[threads=]
        (
          || (
            <dev-db/libpq-8
            >=dev-db/libpq-8[threads=]
          )
        )
      )
    )

Should be this:

  postgres? ( dev-db/postgresql-base )

As version 6.5 is the minimum requirement -- 8.0 is the minimum to enable all of the features available -- and the dev-db/libpq packages are masked for removal.

The lowest version available in the tree is 7.4. Tacking on 7.1 to the dependency isn't necessary and doesn't make sense given the stated requirements.

7.4 will be removed from the tree in a couple months. 8.0 will be removed sometime after this November.

[Comment 4 Aaron W. Swenson 2010-07-08 00:25:49 UTC]

Sorry, I was mistaken. 7.4 and 8.0 will be removed soonish and 8.1 will be removed sometime after November.

[Comment 5 Matti Bickel (RETIRED) 2010-07-09 13:47:58 UTC]

I've uploaded the reduced deps to the php overlay and will migrate the fix to the main tree shortly.

[Comment 6 Matti Bickel (RETIRED) 2010-07-26 15:58:21 UTC]

The fixed deps are now in php-5.3.3 and php-5.3.2