Michael Ströder wrote:
hyc@symas.com wrote:
michael@stroeder.com wrote:
Could we have this overlay in contrib/ shipped with 2.4.24? That would be really nice.
I've tested it with 2.4.23 and it still seems to work.
Committed with changes in HEAD.
Also in RE24. Thanks a lot!
Any objections adding a slightly more complete Makefile (see attachment)?
Yes, several objections. Post diffs that can be applied by the "patch" command, not the complete file. If you're going to add more rules, make them conform to the standard Makefiles. E.g. the correct macro is "prefix" not "PREFIX". Post diffs that actually work, your "OPT" macro is never referenced anywhere else in the Makefile so it's useless. The default prefix is always /usr/local. You can use whatever prefix you like in your personal copy, but stick with the defaults on anything you submit to us.
Ciao, Michael.
Makefile.lastbind
# Copyright 2009 Jonathan Clarkejonathan@phillipoux.net. # All rights reserved. # # Redistribution and use in source and binary forms, with or without # modification, are permitted only as authorized by the OpenLDAP # Public License. # # A copy of this license is available in the file LICENSE in the # top-level directory of the distribution or, alternatively, at #http://www.OpenLDAP.org/license.html.
PREFIX=/opt/openldap-RE24
CPPFLAGS+=-I../../../include -I../../../servers/slapd CPPFLAGS+=-DSLAPD_OVER_LASTBIND=SLAPD_MOD_DYNAMIC #LIBTOOL=libtool LIBTOOL=../../../libtool OPT=-g -O2 CC=gcc
all: lastbind.la
lastbind.lo: lastbind.c $(LIBTOOL) --mode=compile $(CC) $(CPPFLAGS) -Wall -c $?
lastbind.la: lastbind.lo $(LIBTOOL) --mode=link $(CC) -version-info 0:0:0 \ -rpath $(PREFIX)/lib -module -o $@ $?
clean: rm -rf lastbind.lo lastbind.la lastbind.o .libs/
install: lastbind.la mkdir -p $(PREFIX)/libexec/openldap $(LIBTOOL) --mode=install cp lastbind.la $(PREFIX)/libexec/openldap $(LIBTOOL) --finish $(PREFIX)/libexec/openldap