Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve building the LDAP and OCI8 extensions statically. #19

Closed
wants to merge 1 commit into from

Conversation

cjbj
Copy link
Contributor

@cjbj cjbj commented Mar 24, 2012

Allows the LDAP extension to link with the Oracle Instant Client ZIP
file library LDAP implementation, e.g. using
--with-ldap=/path/to/instantclient. (Does not support building with
Instant Client RPMS.)

Allows the LDAP and OCI8 extensions to be statically built when not
passing a directory to --with-ldap. However this uses Oracle ldap.h
headers but doesn't use Oracle's LDAP implementation. A preferred
user build scenario is to build PHP without OCI8, and then build OCI8
as a shared extension.

Allows the LDAP extension to link with the Oracle Instant Client ZIP
file library LDAP implementation, e.g. using
--with-ldap=/path/to/instantclient.  (Does not support building with
Instant Client RPMS.)

Allows the LDAP and OCI8 extensions to be statically built when not
passing a directory to --with-ldap.  However this uses Oracle ldap.h
headers but doesn't use Oracle's LDAP implementation.  A preferred
user build scenario is to build PHP without OCI8, and then build OCI8
as a shared extension.
@smalyshev
Copy link
Contributor

@cjbj So what's up with this patch - is it finished? Does it need changes?

@cjbj
Copy link
Contributor Author

cjbj commented Aug 26, 2013

If you're OK with the patch in principle, I'll update the patch to include Oracle 12 and will merge it directly to git.php

@vladsf
Copy link

vladsf commented Sep 13, 2013

Once HAVE_ORALDAP_11 is introduced then ext/ldap/ldap.c must be fixed as well, otherwise many ldap_* functions becomes unavailable. I will post a patch here later.

@cjbj
Copy link
Contributor Author

cjbj commented Oct 4, 2013

I merged a variation of the patch directly to git.php.net

@cjbj cjbj closed this Oct 4, 2013
@kaplanlior
Copy link
Contributor

For future reference, the commit SHA1 is 5bc3774

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants