Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 337512 - dev-lang/php provide mans to build IBM DB2 PDO support
Summary: dev-lang/php provide mans to build IBM DB2 PDO support
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: PHP Bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-15 17:14 UTC by Maciej Mrozowski
Modified: 2010-09-19 15:16 UTC (History)
0 users

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


Attachments
src_configure patch (php-db2.diff,1.71 KB, patch)
2010-09-15 17:17 UTC, Maciej Mrozowski
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Maciej Mrozowski gentoo-dev 2010-09-15 17:14:16 UTC
It's not possible to build IBM DB2 PDO support because of hardcoded paths to DB2 SDK.
Comment 1 Maciej Mrozowski gentoo-dev 2010-09-15 17:17:00 UTC
Created attachment 247482 [details, diff]
src_configure patch

Makes src_configure aware of DB2_HOME environment variable (modeled after ORACLE_HOME).
Resulting php cgi/cli/apache2-mod objects have proper RPATH appendend sp no further care required. LD_LIBRARY_PATH (see patch) is only needed for src_configure purpose.
Comment 2 Matti Bickel (RETIRED) gentoo-dev 2010-09-19 13:14:00 UTC
Thanks for reporting this!
Just one question about the patch: shouldn't it be LD_LIBRARY_PATH+=...?
Comment 3 Maciej Mrozowski gentoo-dev 2010-09-19 13:26:20 UTC
Yeah, why not. If there's LD_LIBRARY_PATH defined already (it actually shouldn't be the case, and it's not supposed to be needed) let's preserve it.
Comment 4 Matti Bickel (RETIRED) gentoo-dev 2010-09-19 15:16:57 UTC
I see. After reading up on LD_LIBRARY_PATH seperators, it doesn't seem to be a good idea to use +=":${DB2_HOME}/`get_libdir`" here. Empty entries are "." by default and something tells me this is going to come back and bite us security-wise.

So I committed the patch as initially thought. Thanks :)