Summary: | dev-php5/pecl-imagick: add prefix support | ||
---|---|---|---|
Product: | Gentoo/Alt | Reporter: | yegle <cnyegle> |
Component: | Prefix Support | Assignee: | Gentoo Prefix <prefix> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | php-bugs |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 315803 | ||
Attachments: |
php-ext-source-r2.eclass
php-ext-source-r2.eclass-prefix.diff |
Description
yegle
2011-06-17 02:55:13 UTC
(In reply to comment #0) > This package is in Prefix portage tree but cannot be compiled. Right. In tree does NOT mean that it was ever tested or that it works. Do you have a patch for review? pecl-imagick-3.1.0_rc2 is EAPI=4, I guess we just need to tweak conf to be "--with-imagick=${EPREFIX}/usr" and we will need to add prefix support to php-ext-pecl-r2.eclass. Created attachment 410150 [details]
php-ext-source-r2.eclass
I've just added dev-php/pecl-imagick-3.3.0_rc2 to the tree with the EPREFIX thing added.
The eclass that needs prefix support is php-ext-source-r2. I mangled my local copy adding $EPREFIX wherever a raw path was used (but not passed to a dobin, newins, etc. helper). The patched eclass and diff are attached.
This works in my prefix installation, but I don't know wtf I'm doing, so can someone else please take a look?
Created attachment 410152 [details, diff]
php-ext-source-r2.eclass-prefix.diff
It looks good to me. Fixed in 852a595a2. |