Created attachment 458610 [details] pecl-redis-3.1.0.ebuild with USE_PHP="php7-0 php7-1", EAPI=6 support and fixed DEPEND for igbinary serialization Hi, I wrote this ebuild to fix problems with the currently available ebuild in portage tree: * added support for USE_PHP="php7-0 php7-1" * corrected the DEPEND for dev-php/igbinary package * moved to EAPI=6 regards j0inty
Created attachment 458620 [details] pecl-redis-3.1.0.ebuildadded missing CREDITS and COPYING in DOCS and tried to get rid of the src_test
Created attachment 458622 [details] metadata.xml with description for both USE="igbinary redis-session" flags
Oh wonderful. This is now a small mess since they combined the two branches together. I am going to skip 3.1.0 as there are issues in the integration as seen on the github source page, but 3.1.1 should be soon released to fix it.
(In reply to Steffen 'j0inty' Stollfuß from comment #1) > Created attachment 458620 [details] > pecl-redis-3.1.0.ebuildadded missing CREDITS and COPYING in DOCS and tried > to get rid of the src_test We never install the COPYING or any other license file in Gentoo
I totally added this last night. Once I removed the versions with the 5.6 <-> 7.x hack, the dependencies became simple again. I didn't notice the new --enable-redis-session option though. Maybe we should wait and do that in 3.1.1 if it's "coming soon" and fixes other things, though.
(In reply to Brian Evans from comment #3) > Oh wonderful. This is now a small mess since they combined the two branches > together. > > I am going to skip 3.1.0 as there are issues in the integration as seen on > the github source page, but 3.1.1 should be soon released to fix it. Hi Brian, to test the current release candidate I'm using another ebuild for this. This ebuild does everything as expected, but todo this I was forced to use a assignment which should not be done in an ebuild file. But have a look by your own. regards j0inty
Created attachment 458704 [details] pecl-redis-3.1.1_rc1.ebuild Caution: UseItOwnRisk: ebuild use an assignment which should not be done in an ebuild
(In reply to Steffen 'j0inty' Stollfuß from comment #7) > Created attachment 458704 [details] > pecl-redis-3.1.1_rc1.ebuild Caution: UseItOwnRisk: ebuild use an assignment > which should not be done in an ebuild > PV="${PV//_rc1/_RC1}" > MY_PV="${PV//_/}" should be just MY_PV="${PV//_rc1/RC1}" for this particular ebuild... never ever try to change PV
(In reply to Brian Evans from comment #8) > (In reply to Steffen 'j0inty' Stollfuß from comment #7) > > Created attachment 458704 [details] > > pecl-redis-3.1.1_rc1.ebuild Caution: UseItOwnRisk: ebuild use an assignment > > which should not be done in an ebuild > > > PV="${PV//_rc1/_RC1}" > > MY_PV="${PV//_/}" > > should be just MY_PV="${PV//_rc1/RC1}" for this particular ebuild... never > ever try to change PV Hi, that was the reason I added the warning, too. But your fix alone doesn't work for my ebuild. I know about the MY_VAR's and the original ${P}, ${PV}, ... My notice above the PV statement pointed you out why I'm at time was only to get the php-ext-source src_unpack working which called by php-ext-pecl-r3 The Step: copy working dir to the php_target dirs. # php-ext-pecl-r3 set ${S} as # S="${WORKDIR}/${PHP_EXT_PECL_PKG}-${PV/_/}" # this results in phpredis-3.1.1rc1 # but we need phpredis-3.1.1RC1 when in php-ext-source-r3.eclass # the source copied for the diffrent php targets PV="${PV//_rc1/_RC1}" MY_PV="${PV//_/}" regards
(In reply to Steffen 'j0inty' Stollfuß from comment #9) > (In reply to Brian Evans from comment #8) > > (In reply to Steffen 'j0inty' Stollfuß from comment #7) > > > Created attachment 458704 [details] > > > pecl-redis-3.1.1_rc1.ebuild Caution: UseItOwnRisk: ebuild use an assignment > > > which should not be done in an ebuild > > > > > PV="${PV//_rc1/_RC1}" > > > MY_PV="${PV//_/}" > > > > should be just MY_PV="${PV//_rc1/RC1}" for this particular ebuild... never > > ever try to change PV > > Hi, > > that was the reason I added the warning, too. But your fix alone doesn't > work for my ebuild. I know about the MY_VAR's and the original ${P}, ${PV}, > ... > > My notice above the PV statement pointed you out why I'm at time was only to > get the php-ext-source src_unpack working which called by php-ext-pecl-r3 > The Step: copy working dir to the php_target dirs. > > # php-ext-pecl-r3 set ${S} as > # S="${WORKDIR}/${PHP_EXT_PECL_PKG}-${PV/_/}" > # this results in phpredis-3.1.1rc1 > # but we need phpredis-3.1.1RC1 when in php-ext-source-r3.eclass > # the source copied for the diffrent php targets > PV="${PV//_rc1/_RC1}" > > MY_PV="${PV//_/}" > > regards So reset S after the inherit
(In reply to Brian Evans from comment #10) > (In reply to Steffen 'j0inty' Stollfuß from comment #9) > > (In reply to Brian Evans from comment #8) > > > (In reply to Steffen 'j0inty' Stollfuß from comment #7) > > > > Created attachment 458704 [details] > > > > pecl-redis-3.1.1_rc1.ebuild Caution: UseItOwnRisk: ebuild use an assignment > > > > which should not be done in an ebuild > > > > > > > PV="${PV//_rc1/_RC1}" > > > > MY_PV="${PV//_/}" > > > > > > should be just MY_PV="${PV//_rc1/RC1}" for this particular ebuild... never > > > ever try to change PV > > > > Hi, > > > > that was the reason I added the warning, too. But your fix alone doesn't > > work for my ebuild. I know about the MY_VAR's and the original ${P}, ${PV}, > > ... > > > > My notice above the PV statement pointed you out why I'm at time was only to > > get the php-ext-source src_unpack working which called by php-ext-pecl-r3 > > The Step: copy working dir to the php_target dirs. > > > > # php-ext-pecl-r3 set ${S} as > > # S="${WORKDIR}/${PHP_EXT_PECL_PKG}-${PV/_/}" > > # this results in phpredis-3.1.1rc1 > > # but we need phpredis-3.1.1RC1 when in php-ext-source-r3.eclass > > # the source copied for the diffrent php targets > > PV="${PV//_rc1/_RC1}" > > > > MY_PV="${PV//_/}" > > > > regards > > So reset S after the inherit Ohje ... Sometimes it can be so simple and you don't see it ... :D Thanks j0inty
(In reply to Brian Evans from comment #10) > (In reply to Steffen 'j0inty' Stollfuß from comment #9) > > (In reply to Brian Evans from comment #8) > > > (In reply to Steffen 'j0inty' Stollfuß from comment #7) > > > > Created attachment 458704 [details] > > > > pecl-redis-3.1.1_rc1.ebuild Caution: UseItOwnRisk: ebuild use an assignment > > > > which should not be done in an ebuild > > > > > > > PV="${PV//_rc1/_RC1}" > > > > MY_PV="${PV//_/}" > > > > > > should be just MY_PV="${PV//_rc1/RC1}" for this particular ebuild... never > > > ever try to change PV > > > > Hi, > > > > that was the reason I added the warning, too. But your fix alone doesn't > > work for my ebuild. I know about the MY_VAR's and the original ${P}, ${PV}, > > ... > > > > My notice above the PV statement pointed you out why I'm at time was only to > > get the php-ext-source src_unpack working which called by php-ext-pecl-r3 > > The Step: copy working dir to the php_target dirs. > > > > # php-ext-pecl-r3 set ${S} as > > # S="${WORKDIR}/${PHP_EXT_PECL_PKG}-${PV/_/}" > > # this results in phpredis-3.1.1rc1 > > # but we need phpredis-3.1.1RC1 when in php-ext-source-r3.eclass > > # the source copied for the diffrent php targets > > PV="${PV//_rc1/_RC1}" > > > > MY_PV="${PV//_/}" > > > > regards > > So reset S after the inherit Sry but I must revert last comment because the problem exists while the "inherit" is running. build.log https://gist.github.com/j0inty/82031d7eeeab81c78f972cfc25f307ce The error occured in php-ext-source-r3_src_unpack as I said before when this function create the "source" folder for the diffrent php_targets (php5-6 php7-0 ...). A php_target source folder is nothing more that the ${S} copied to ${WORKDIR}/${php_target}. Here the name of the extracted folder from archive is not the same as in ${S}. regards
3.1.1_rc2 is now in the tree
