Bug 52823 - php should explicitly require curl
Summary: php should explicitly require curl
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Red Hat Linux
Classification: Retired
Component: php
Version: 7.3
Hardware: i386
OS: Linux
low
low
Target Milestone: ---
Assignee: Phil Copeland
QA Contact: David Lawrence
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2001-08-29 18:20 UTC by Chris Runge
Modified: 2005-10-31 22:00 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2002-04-16 01:46:02 UTC
Embargoed:


Attachments (Terms of Use)

Description Chris Runge 2001-08-29 18:20:50 UTC
Description of Problem:

trying to install php gives the error:

error: failed dependencies:
	libcurl.so.1   is needed by php-4.0.6-7

the php RPM should explicitly require curl (e.g., "Requires: curl" in .spec
file) so that people aren't left to wonder where libcurl.so.1 is located or
what it is (in this case it is fairly clear, but nonetheless...)

Version-Release number of selected component (if applicable):

php-4.0.6-7.i386.rpm

How Reproducible:


Steps to Reproduce:
1. 
2. 
3. 

Actual Results:


Expected Results:


Additional Information:

Comment 1 Phil Copeland 2002-01-08 19:34:51 UTC
In rawhide
ftp://rawhide.redhat.com/pub/redhat/linux/rawhide/SRPMS/SRPMS/php-4.0.6-13.src.rpm

...
BuildPrereq: unixODBC-devel, pspell-devel, curl-devel >= 7.8, bzip2
...

the cur-devel cannot be installed without curl I believe this should be
sufficient unless the curl maintbot has done something odd

Try that and tell me if I can close this bug please

Phil
=--=


Comment 2 Brent Nordquist 2002-03-19 21:04:02 UTC
Those are two different things:

BuildPrereq: curl-devel >= 7.8

means that curl-devel needs to be installed on the box where the RPMs are built.
 This bug is asking that:

Requires: curl

be added, which would require that the curl RPM be installed on the target box
before the php RPM can be installed.  The libcurl.so.1 dependency is created
automatically by the RPM build process.

I don't know if Red Hat has standards for this, but the libcurl.so.1 requirement
is sufficient, if the library is all that's required.  Lots of other RPMs have
the same scenario.  I think an explicit "Requires: curl" should only be there if
the curl RPM provides something the php RPM needs that the auto-dependency
generator doesn't find.  My $0.02

Comment 3 Peter Bowen 2002-04-16 01:45:57 UTC
I agree.  In 99.9% of cases if you have a Build(Pre)?Req of foo-devel, you
should  Require foo.

Comment 4 Phil Copeland 2002-06-04 21:26:39 UTC
Incoming errata candidate in rawhide real soon now,.. check your local listings,..

Phil
=--=


Note You need to log in before you can comment on or make changes to this bug.