Re: commit: ldap/servers/slapd/tools slapcommon.c

["Kurt D. Zeilenga" <Kurt@OpenLDAP.org>]

I think this fix (and possibly the prior code) is actually
bad.

slapcat -n N should exports the Nth database, period.
slapadd -n N should import the Nth database, period.

If the user selects a database which doesn't (currently)
support exporting/importing, an error should be returned.

To do otherwise will lead to user confusion as its
possible that a database could support exporting
but not importing via slapadd (meaning slapadd -n N
may not refer to the same database as slapcat -n N)
or vice versa.

And support for importing/exporting may change over time.
Today only a few backends support these functions,
however nothing stops them from being more widely
implemented.

Let's keep the numbering simple.

Kurt


At 01:25 PM 3/3/2003, ando@OpenLDAP.org wrote:
>Update of /repo/OpenLDAP/pkg/ldap/servers/slapd/tools
>
>Modified Files:
>        slapcommon.c  1.33 -> 1.34
>
>Log Message:
>fix ITS#2343
>
>CVS Web URLs:
>  http://www.openldap.org/devel/cvsweb.cgi/servers/slapd/tools/
>    http://www.openldap.org/devel/cvsweb.cgi/servers/slapd/tools/slapcommon.c
>
>Changes are generally available on cvs.openldap.org (and CVSweb)
>within 30 minutes of being committed.