Opened 12 years ago

Last modified 22 months ago

#20492 assigned defect

php4, php5, php5-devel: php links with libldap if present

Reported by: ryandesign (Ryan Schmidt) Owned by: ryandesign (Ryan Schmidt)
Priority: Normal Milestone:
Component: ports Version: 1.7.1
Keywords: Cc: ingo@…, chrstphrchvz (Christopher Chavez)
Port: php4, php5, php5-devel

Description

php links with libldap if the openldap port is installed, though no dependency is declared. This causes an error if openldap is subsequently uninstalled:

$ php
dyld: Library not loaded: /mp/lib/libldap-2.3.0.dylib
  Referenced from: /mp/bin/php
  Reason: image not found
Trace/BPT trap

php should be changed to not link with unrequested libraries. See if --disable-all fixes this. Check whether php4 is affected as well.

Change History (8)

comment:1 Changed 12 years ago by nerdling (Jeremy Lavergne)

We cannot specify --with-ldap and leave it blank either: in the case the user does not specify a path, configure will look for ldap.h and include in /usr and /usr/local. The ./configure --help output did not reflect this properly. This is changed in CVS now.

comment:2 Changed 12 years ago by nerdling (Jeremy Lavergne)

Keep in mind that's a dinosaur ticket – really.

comment:3 in reply to:  description Changed 12 years ago by ryandesign (Ryan Schmidt)

Status: newassigned

Replying to ryandesign@…:

php should be changed to not link with unrequested libraries. See if --disable-all fixes this.

php5 @5.3.0_2 uses --disable-all but it has not fixed this problem.

comment:4 Changed 11 years ago by ryandesign (Ryan Schmidt)

We are actually requesting ldap, but wanting it to come from /usr:

platform macosx {
    configure.args-append \
        --with-ldap=/usr
}

I don't know why we do that. Presumably the fix is to not request ldap at all in the php5 port, and instead create a php5-ldap port that depends on the openldap port, thereby furthering #19091.

comment:5 Changed 10 years ago by ryandesign (Ryan Schmidt)

Has duplicate #26492.

comment:6 Changed 10 years ago by ryandesign (Ryan Schmidt)

Cc: ingo@… added
Port: php4 added
Summary: php5, php5-devel: php links with libldap if presentphp4, php5, php5-devel: php links with libldap if present
  • r71587: php5-ldap port added
  • r71588: built-in ldap removed from php5 and php5-devel ports

php4 is still affected

comment:7 Changed 22 months ago by chrstphrchvz (Christopher Chavez)

php5-devel was removed: [e1b3c7504a/macports-ports]

php5 was obsoleted: [c7b8ec711f/macports-ports]

comment:8 Changed 22 months ago by chrstphrchvz (Christopher Chavez)

Cc: chrstphrchvz added
Note: See TracTickets for help on using tickets.