Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.3.0

Request #6971 Create deps on php-[ext]
Submitted: 2006-03-01 11:00 UTC
From: timj at php dot net Assigned: timj
Status: Closed Package: PEAR_Command_Packaging (version CVS)
PHP Version: 4.4.2 OS:
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 48 - 21 = ?

 
 [2006-03-01 11:00 UTC] timj at php dot net (Tim Jackson)
Description: ------------ The current code does not create any RPM dependencies on PHP extensions or modules (e.g. xml, pcre etc.). Such deps would depend on downstream packaging (e.g. see https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=173804) so it would be good to be configurable in the same way as the --rpm-depname and --rpm-pkgname options. See also Problem 5 of: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=176725 Part of this patch does the trick: https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=122656 Test script: --------------- pear make-rpm-spec Crypt_CHAP-1.0.0.tgz grep ^Requires PEAR\:\:Crypt_CHAP-1.0.0.spec Expected result: ---------------- Requires: php-mhash, php-mcrypt Actual result: -------------- [nothing]

Comments

 [2006-03-02 16:37 UTC] glen at delfi dot ee (Elan Ruusamäe)
i've solved similiar problem in makerpm command. but i believe these can be integrated into this package too. http://cvs.pld-linux.org/cgi-bin/cvsweb/SOURCES/php-pear-PEAR-rpmvars.patch the patch is actually meta patch, ie it solves various issues i list the previous work regardin spec packaging also here (some of them already obsolete): http://cvs.pld-linux.org/cgi-bin/cvsweb/SOURCES/php-pear-PEAR-rpmpkgname.patch http://cvs.pld-linux.org/cgi-bin/cvsweb/SOURCES/php-pear-PEAR-specfile.patch
 [2006-04-15 01:41 UTC] timj at php dot net (Tim Jackson)
OK, I've fixed this bug in CVS. Elan, I've also added in one way or another most of the changes from your own patches, at least the ones that are appropriate to pull upstream. Please take a look at the cvs logs and have a play with this new version. You should find that your distro-specific patches are now much smaller and simpler. After that, if there's anything that's outstanding that can be pushed upstream (e.g. your "optional deps" stuff; I didn't do anything with that so far), please file new bugs here as appropriate. (Patches appreciated :-)