[389-devel] Commit (3rd try): Plugin Default Config Entry

[Noriko Hosoi]

Thanks for the review, Rich!

Pushed to master.

$ git merge usn
Updating 177f772..6b08e5b
Fast forward
  ldap/servers/plugins/replication/repl5_agmt.c |  136 +++++++++++++++++----
  ldap/servers/plugins/usn/usn.c                |   40 +++++-
  ldap/servers/plugins/usn/usn.h                |    2 +-
  ldap/servers/slapd/dse.c                      |    7 +-
  ldap/servers/slapd/plugin.c                   |  156 
+++++++++++++++++++++++++
  ldap/servers/slapd/proto-slap.h               |    2 +-
  ldap/servers/slapd/slapi-plugin.h             |   23 ++++
  ldap/servers/slapd/slapi-private.h            |    2 +
  8 files changed, 329 insertions(+), 39 deletions(-)
$ git push
Counting objects: 31, done.
Delta compression using 4 threads.
Compressing objects: 100% (16/16), done.
Writing objects: 100% (16/16), 5.66 KiB, done.
Total 16 (delta 13), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
    177f772..6b08e5b  master -> master

On 08/31/2009 07:22 AM, Rich Megginson wrote:
> Noriko Hosoi wrote:
>> Thanks to Endi and Rich for their comments to improve the 
>> functionality.  I changed the slapi APIs to take and return the 
>> Slapi_Value(Set) to support any data type other than string.
>>
>> New slapi APIs in libslapd:
>>  int slapi_set_plugin_default_config(const char *type, Slapi_Value 
>> *value);
>>  Description: Add given "type: value" to the plugin default config entry
>>               (cn=plugin default config,cn=config) unless the same 
>> "type:
>>               value" pair already exists in the entry.
>>  Parameters: type - Attribute type to add to the default config entry
>>              value - Attribute value to add to the default config entry
>>  Return Value: 0 if the operation was successful
>>                non-0 if the operation was not successful
>>
>>  int slapi_get_plugin_default_config(char *type, Slapi_ValueSet 
>> **valueset);
>>  Description: Get attribute values of given type from the plugin default
>>               config entry (cn=plugin default config,cn=config).
>>  Parameters: type - Attribute type to get from the default config entry
>>              valueset Valueset holding the attribute values
>>  Return Value: 0 if the operation was successful
>>              non-0 if the operation was not successful
>>  warning: Caller is responsible to free attrs by slapi_ch_array_free
>>
>> Thanks,
>> --noriko
> Looks good.  ack
>>
>>
>> ------------------------------------------------------------------------
>>
>> -- 
>> 389-devel mailing list
>> 389-devel at redhat.com
>> https://www.redhat.com/mailman/listinfo/fedora-directory-devel
>
> ------------------------------------------------------------------------
>
> --
> 389-devel mailing list
> 389-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/fedora-directory-devel
>    

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/fedora-directory-devel/attachments/20090831/aa3c8cd5/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3250 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://listman.redhat.com/archives/fedora-directory-devel/attachments/20090831/aa3c8cd5/attachment.p7s>