ldb: support for modrdn LDIF

simo idra at samba.org
Sat Apr 9 06:25:20 MDT 2011


On Sat, 2011-04-09 at 11:43 +0200, Stefan (metze) Metzmacher wrote:
> Hi,
> 
> I've added support for modrdn in LDIF, this looks like this:
> 
> dn: cn=Ursula Hampster,ou=Alumni Association,ou=People,o=University of
> Michiga
>  n,c=TEST
> changetype: moddn
> newrdn: cn=Hampster Ursula
> deleteoldrdn: 1
> newsuperior: ou=Alumni Association,ou=People,o=University of Michigan,c=TEST
> 
> dn: cn=Hampster Ursula,ou=Alumni Association,ou=People,o=University of
> Michiga
>  n,c=TEST
> changetype: modrdn
> newrdn: cn=Ursula Hampster
> deleteoldrdn: 1
> 
> The code can be found here:
> http://gitweb.samba.org/?p=metze/samba/wip.git;a=shortlog;h=refs/heads/master4-ldb
> 
> I've added a ldb_ldif_parse_modrdn() helper function which parses the modrdn
> related stuff out of a ldb_message. I think this is better than changing
> struct ldb_ldif, as that would be an ABI breakage and we would need to
> change the version
> to 2.0.0.
> 
> As ldb_ldif_parse_modrdn() is a new public function I changed the
> version to 1.1.0 instead of 1.0.3.
> 
> Is it ok to push this?

Ok to push but no need for a new Y version, 1.0.4 is enough as a new
function doesn't cause any incompatibility with previous library
versions.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Principal Software Engineer at Red Hat, Inc. <simo at redhat.com>



More information about the samba-technical mailing list