Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 85082 - dev-php/phpdbg-* broken
Summary: dev-php/phpdbg-* broken
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: PHP Bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-03-13 06:40 UTC by Jürgen Hötzel
Modified: 2005-05-10 22:14 UTC (History)
0 users

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


Attachments
move php.ini modifications to src_install (phpdbg-2.11.23-r1.patch,645 bytes, patch)
2005-03-13 06:42 UTC, Jürgen Hötzel
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jürgen Hötzel 2005-03-13 06:40:18 UTC
The current ebuilds (2.11.23 and 2.11.32) call
php-ext-base_addtoinifiles "[Debugger]"
to add a section to the php.ini files. This fails because the eclass functions expects key value parameters:

...
* Added 'extension=/etc/php4/lib/dbg.so' to /etc/php/apache2-php4/php.ini
* Added 'extension=/etc/php4/lib/dbg.so' to /etc/php/cli-php4/php.ini
/usr/lib/portage/bin/ebuild.sh: line 125: $3: ambiguous redirect
 * Added '[Debugger]=etc/php/apache2-php4/php.ini' to /
dirname: too few arguments
Try `dirname --help' for more information.
/usr/lib/portage/bin/doins: at least one argument needed
...

Perhaps we need a "addsectiontoini" function. But the Extension runs fine without the [Debugger] line in the php.ini file.

Also, the php.ini modification stay in the sandbox, because they are done in pkg_postinst. We need to do this in src_install (like mmcache ebuild). 

The line
php-ext-base_addtoinifiles "extension" "/etc/php4/lib/dbg.so" 
is unnecessary  because the extension is already added in  php-ext-source_src_install.

Patches enclosed. 
Cheers, J
Comment 1 Jürgen Hötzel 2005-03-13 06:40:18 UTC
The current ebuilds (2.11.23 and 2.11.32) call
php-ext-base_addtoinifiles "[Debugger]"
to add a section to the php.ini files. This fails because the eclass functions expects key value parameters:

...
* Added 'extension=/etc/php4/lib/dbg.so' to /etc/php/apache2-php4/php.ini
* Added 'extension=/etc/php4/lib/dbg.so' to /etc/php/cli-php4/php.ini
/usr/lib/portage/bin/ebuild.sh: line 125: $3: ambiguous redirect
 * Added '[Debugger]=etc/php/apache2-php4/php.ini' to /
dirname: too few arguments
Try `dirname --help' for more information.
/usr/lib/portage/bin/doins: at least one argument needed
...

Perhaps we need a "addsectiontoini" function. But the Extension runs fine without the [Debugger] line in the php.ini file.

Also, the php.ini modification stay in the sandbox, because they are done in pkg_postinst. We need to do this in src_install (like mmcache ebuild). 

The line
php-ext-base_addtoinifiles "extension" "/etc/php4/lib/dbg.so" 
is unnecessary  because the extension is already added in  php-ext-source_src_install.

Patches enclosed. 
Cheers, Jürgen
Comment 2 Jürgen Hötzel 2005-03-13 06:42:37 UTC
Created attachment 53332 [details, diff]
move php.ini modifications to src_install
Comment 3 Sebastian Bergmann (RETIRED) gentoo-dev 2005-05-10 22:14:47 UTC
The proposed changes to the dev-php/phpdbg-* ebuilds have been made.