Bug 371985 – dev-php5/pecl-imagick: add prefix support

• Status: RESOLVED FIXED
• Product: Gentoo/Alt

[Description yegle 2011-06-17 02:55:13 UTC]

This package is in Prefix portage tree but cannot be compiled.

It hardcoded imagick prefix path to /usr and uses /usr/lib/php5.3/bin/phpize.

[Comment 1 Jeremy Olexa (darkside) (RETIRED) 2011-11-16 17:16:41 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?

[Comment 2 Christoph Junghans (RETIRED) 2012-08-24 13:47:08 UTC]

pecl-imagick-3.1.0_rc2 is EAPI=4, I guess we just need to tweak conf to be "--with-imagick=${EPREFIX}/usr"

[Comment 3 Christoph Junghans (RETIRED) 2012-10-18 08:27:51 UTC]

and we will need to add prefix support to php-ext-pecl-r2.eclass.

[Comment 4 Michael Orlitzky 2015-08-24 17:07:38 UTC]

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?

[Comment 5 Michael Orlitzky 2015-08-24 17:08:30 UTC]

Created attachment 410152 [details, diff]
php-ext-source-r2.eclass-prefix.diff

[Comment 6 Fabian Groffen 2015-08-24 17:12:45 UTC]

It looks good to me.

[Comment 7 Michael Orlitzky 2015-08-30 14:29:07 UTC]

Fixed in 852a595a2.