Re: ports/125400: devel/apr APR_UTIL_WITH_LDAP=yes building is broken

[Ruediger Pluem]

On 07/08/2008 09:23 PM, Philip M. Gollucci wrote:

Ruediger Pluem wrote:

 if test "$apu_has_ldap_openldap" = "1"; then
+    CPPFLAGS=$save_cppflags
+    LDFLAGS=$save_ldflags
+    LIBS=$save_libs

Shouldn't this be the other way round such that you *safe* the old values
of CPPFLAGS, LDFLAGS and LIBS for later restore in the safe* variables?

You are absolutely right. I can't believe I cut/pasted that...

*sigh*....

worth another diff ?

Yes please. For ease of appliance and track of record.

Regards

Rüdiger