Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 635826 - dev-libs/xapian-bindings: Support for multiple slots of PHP and Ruby, other fixes
Summary: dev-libs/xapian-bindings: Support for multiple slots of PHP and Ruby, other f...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: Normal normal (vote)
Assignee: Anthony Basile
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-10-29 18:49 UTC by Arfrever Frehtes Taifersar Arahesis
Modified: 2017-11-20 10:37 UTC (History)
2 users (show)

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


Attachments
xapian-bindings.patch (xapian-bindings.patch,6.92 KB, patch)
2017-10-29 18:50 UTC, Arfrever Frehtes Taifersar Arahesis
Details | Diff
xapian-bindings-1.4.5.patch (xapian-bindings-1.4.5.patch,7.68 KB, patch)
2017-11-06 15:07 UTC, Brian Evans (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Arfrever Frehtes Taifersar Arahesis 2017-10-29 18:49:41 UTC
This patch adds proper support for multiple slots of PHP and Ruby.
It is done similarly to the way Python bindings are handled in this ebuild.

Detection of Lua headers fails. Setting LUA_INC fixes this problem.

Previously tests of Python bindings were not run. This patch fixes it.
Comment 1 Arfrever Frehtes Taifersar Arahesis 2017-10-29 18:50:41 UTC
Created attachment 500934 [details, diff]
xapian-bindings.patch
Comment 2 Anthony Basile gentoo-dev 2017-10-30 14:01:33 UTC
(In reply to Arfrever Frehtes Taifersar Arahesis from comment #1)
> Created attachment 500934 [details, diff] [details, diff]
> xapian-bindings.patch

With this patch

USE="php" ebuild xapian-bindings-1.4.5.ebuild clean install

with only php7 install leads to

checking for PHP5 extension directory... ./configure: line 17539: /usr/lib64/php5.6/bin/php-config: No such file or directory

However, it works fine without the patch.
Comment 3 Arfrever Frehtes Taifersar Arahesis 2017-10-30 16:34:02 UTC
(In reply to Anthony Basile from comment #2)

You used `ebuild` instead of `emerge`, so dependencies were ignored.
Maybe your PHP_TARGETS includes php5-6.
Check: emerge -ptv1 dev-libs/xapian-bindings

If problem is still present after potential correction of configuration, please attach build log and config.log
Comment 4 Anthony Basile gentoo-dev 2017-11-06 13:07:47 UTC
(In reply to Arfrever Frehtes Taifersar Arahesis from comment #3)
> (In reply to Anthony Basile from comment #2)
> 
> You used `ebuild` instead of `emerge`, so dependencies were ignored.
> Maybe your PHP_TARGETS includes php5-6.
> Check: emerge -ptv1 dev-libs/xapian-bindings
> 
> If problem is still present after potential correction of configuration,
> please attach build log and config.log

The value of PHP_TARGETS is set to php5-6 in base profiles.  That needs to be fixed before I push this otherwise lots of people will get errors.
Comment 5 Michael Orlitzky gentoo-dev 2017-11-06 13:37:08 UTC
(In reply to Anthony Basile from comment #2)
> 
> With this patch
> 
> USE="php" ebuild xapian-bindings-1.4.5.ebuild clean install
> 
> with only php7 install...

If you have "php5-6" in PHP_TARGETS, then the php-ext-source-r3 eclass should be adding php:5.6 to (R)DEPEND. Arfrever may have guessed right: is it because you used "ebuild" instead of "emerge"? Can you figure out why portage didn't pull in dev-lang/php:5.6 if you have PHP_TARGETS="php5-6"?

(The eclass should work regardless of what users have in PHP_TARGETS.)
Comment 6 Brian Evans (RETIRED) gentoo-dev 2017-11-06 14:30:53 UTC
Patch needs to add
	addpredict /usr/share/snmp/mibs/.index
	addpredict /var/lib/net-snmp/mib_indexes
to just before "php_foreach_impl run_in_build_dir php_configure" in case php is built with snmp USE (stupid net-analyzer/net-snmp library)
Comment 7 Brian Evans (RETIRED) gentoo-dev 2017-11-06 15:07:02 UTC
Created attachment 502812 [details, diff]
xapian-bindings-1.4.5.patch

This builds on Arfrever's patch adding addpredicts for SNMP with PHP and skips single bindings if only php, python or ruby are enabled
Comment 8 Anthony Basile gentoo-dev 2017-11-20 10:37:14 UTC
(In reply to Brian Evans from comment #7)
> Created attachment 502812 [details, diff] [details, diff]
> xapian-bindings-1.4.5.patch
> 
> This builds on Arfrever's patch adding addpredicts for SNMP with PHP and
> skips single bindings if only php, python or ruby are enabled

okay i finally had a chance to properly test.  looks good.  thanks everyone!  this is a significant improvement to the package.