[Freeipa-devel] [PATCH] 029 Enforce uniqueness on (key, info) pairs in automount keys

[Jakub Hrozek]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 01/10/2011 05:15 PM, Jakub Hrozek wrote:
> On 12/20/2010 03:33 PM, Jakub Hrozek wrote:
>> On 12/20/2010 02:49 PM, Jakub Hrozek wrote:
>>> Attached is a patch that changes the uniqueness constraint of automount
>>> keys from (key) to (key,info) pairs. The patch is not really standard
>>> baseldap style. The reason is that during development, I found that
>>> baseldap is really dependent on having a single primary key and also
>>> during many operations accessing it as keys[-1].
> 
>>> Please note that the ipa automountkey-* commands used to have three
>>> args, now its two args and two required options (that compose the tuple
>>> that is primary key). I know next to nothing about UI, but I assume this
>>> has consequences as the JSON marshalled call needs to be different now.
>>> Can someone point me to the place in code that I need to fix now?
> 
>>> Fixes:
>>> https://fedorahosted.org/freeipa/ticket/293
> 
>> Sorry, I left some debugging statements in. Attached is a new patch.
> 
> Attached is a patch that applies cleanly on top of origin/master.

Another rebase on top of the recent API changes.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAk03/ykACgkQHsardTLnvCX8YACggTayFsJWvZX7mueh1D4jEnSQ
fzgAoMv+zmQmY1bF9QeI3J+mmmWqoR0d
=70CM
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jhrozek-freeipa-029-04-automount-keys-uniqueness.patch
Type: text/x-patch
Size: 27586 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20110120/fbc009fe/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jhrozek-freeipa-029-04-automount-keys-uniqueness.patch.sig
Type: application/pgp-signature
Size: 72 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20110120/fbc009fe/attachment.sig>