Summary: | net-analyzer/acid depends on =dev-php/mod-php-4* instead of virtual/httpd-php | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Jakub Moc (RETIRED) <jakub> |
Component: | New packages | Assignee: | PHP Bugs <php-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | 106514 | ||
Bug Blocks: | 60440 | ||
Attachments: |
acid-0.9.6_beta23.ebuild.diff
patch for acid-0.9.6_beta23-r1 ebuild acid-0.9.6_beta23-r1.ebuild.diff patch - final version :) |
Description
Jakub Moc (RETIRED)
2005-08-17 05:57:07 UTC
OK, the new PHP ebuilds are in portage and will be removed from package.mask in two days (Thursday, Sept. 8). This needs an urgent fix so that we don't break the dependency tree for ~arch users. Mass ping. ;) Hi, I am not too sure how to proceed here. For example, pkg_setup() includes a "built_with_use dev-php/mod_php gd" condition, is replacing dev-php/mod_php with virtual/httpd-php a plausible option? Better yet, would you mind attaching a patch? :-) Thanks, Marcelo Created attachment 67964 [details, diff]
acid-0.9.6_beta23.ebuild.diff
OK, here's the patch. I don't understand though, why the checks for mod_php-5*
w/ session use flags were added in the original ebuild, because the ebuild did
specifically depend on mod_php-4*.
If this actually works w/ php-5, then drop "need_php4" from the patch and
adjust the RDEPEND accordingly. Check for session use flag was added now
anyway, b/c dev-lang/php-4* has this use flag as well.
Thanks.
To be frank, neither do I. Thanks for the patch and the php magic. I can't really test it with php-5 right now, but at least your patch is in so that this ebuild won't burst into flames. OK, this needs a bit more lovin'. ;) dev-php/jpgraph will be "splitted" into dev-php{4,5}/jpgraph, as there are different versions incompatible with the other php version. (In fact, this has already been done in PHP overlay). If this just works w/ php4, it'll be pretty simple, otherwise some eclass magic will be needed. I'll keep you posted. :) Created attachment 68802 [details, diff]
patch for acid-0.9.6_beta23-r1 ebuild
Fixed need_php4, inherits depend.apache, adapted for new dev-php4/jpgraph
ebuild, fixed dependency on adodb to version that is compatible w/
dev-lang/php.
Please, revbump this, the changes are really substantial now.
OK, please don't keyword ~ppc and ~amd64 until dev-php4/jpgraph and =dev-php/adodb-4.65-r1 are keyworded. Will file a bug for them. Created attachment 68820 [details, diff]
acid-0.9.6_beta23-r1.ebuild.diff
I'm actually having a problem with apache (I unmerged it along with mod_php and
php to work around a blocker) and I can't emerge it. If anyone can test this,
feel free to commit.
Finally got to this... Please, replace the pkg_setup with the following and all will be set: pkg_setup() { webapp_pkg_setup require_gd require_php_with_use session } require_gd allows for install with dev-lang/php compiled with either USE=gd or USE=gd-external, both have necessary gd support. Thanks. Created attachment 76171 [details, diff]
patch - final version :)
added - thanks Jakub. @ppc: Please, re-keyword 0.9.6_beta23-r1 now that dev-php4/jpgraph is keyworded... Thanks. Marked ~ppc, transfering the bug back to php-bugs, I'm not sure if they want to close it yet. Closing, this has never been keyworded stable. Thanks. |