<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html; charset=ISO-8859-1"
 http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
Thanks for the review, Rich!<br>
<br>
Pushed to master.<br>
<br>
$ git merge usn<br>
Updating 177f772..6b08e5b<br>
Fast forward<br>
&nbsp;ldap/servers/plugins/replication/repl5_agmt.c |&nbsp; 136
+++++++++++++++++----<br>
&nbsp;ldap/servers/plugins/usn/usn.c&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; |&nbsp;&nbsp; 40 +++++-<br>
&nbsp;ldap/servers/plugins/usn/usn.h&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; |&nbsp;&nbsp;&nbsp; 2 +-<br>
&nbsp;ldap/servers/slapd/dse.c&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; |&nbsp;&nbsp;&nbsp; 7 +-<br>
&nbsp;ldap/servers/slapd/plugin.c&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; |&nbsp; 156
+++++++++++++++++++++++++<br>
&nbsp;ldap/servers/slapd/proto-slap.h&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; |&nbsp;&nbsp;&nbsp; 2 +-<br>
&nbsp;ldap/servers/slapd/slapi-plugin.h&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; |&nbsp;&nbsp; 23 ++++<br>
&nbsp;ldap/servers/slapd/slapi-private.h&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; |&nbsp;&nbsp;&nbsp; 2 +<br>
&nbsp;8 files changed, 329 insertions(+), 39 deletions(-)<br>
$ git push<br>
Counting objects: 31, done.<br>
Delta compression using 4 threads.<br>
Compressing objects: 100% (16/16), done.<br>
Writing objects: 100% (16/16), 5.66 KiB, done.<br>
Total 16 (delta 13), reused 0 (delta 0)<br>
To ssh://git.fedorahosted.org/git/389/ds.git<br>
&nbsp;&nbsp; 177f772..6b08e5b&nbsp; master -&gt; master<br>
<br>
On 08/31/2009 07:22 AM, Rich Megginson wrote:
<blockquote cite="mid:4A9BDC88.9030903@redhat.com" type="cite">Noriko
Hosoi wrote:
  <br>
  <blockquote type="cite">Thanks to Endi and Rich for their comments to
improve the functionality.&nbsp; I changed the slapi APIs to take and return
the Slapi_Value(Set) to support any data type other than string.
    <br>
    <br>
New slapi APIs in libslapd:
    <br>
&nbsp;int slapi_set_plugin_default_config(const char *type, Slapi_Value
*value);
    <br>
&nbsp;Description: Add given "type: value" to the plugin default config
entry
    <br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; (cn=plugin default config,cn=config) unless the same
"type:
    <br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; value" pair already exists in the entry.
    <br>
&nbsp;Parameters: type - Attribute type to add to the default config entry
    <br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; value - Attribute value to add to the default config entry
    <br>
&nbsp;Return Value: 0 if the operation was successful
    <br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; non-0 if the operation was not successful
    <br>
    <br>
&nbsp;int slapi_get_plugin_default_config(char *type, Slapi_ValueSet
**valueset);
    <br>
&nbsp;Description: Get attribute values of given type from the plugin
default
    <br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; config entry (cn=plugin default config,cn=config).
    <br>
&nbsp;Parameters: type - Attribute type to get from the default config entry
    <br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; valueset Valueset holding the attribute values
    <br>
&nbsp;Return Value: 0 if the operation was successful
    <br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; non-0 if the operation was not successful
    <br>
&nbsp;warning: Caller is responsible to free attrs by slapi_ch_array_free
    <br>
    <br>
Thanks,
    <br>
--noriko
    <br>
  </blockquote>
Looks good.&nbsp; ack
  <br>
  <blockquote type="cite"><br>
    <br>
------------------------------------------------------------------------
    <br>
    <br>
--
    <br>
389-devel mailing list
    <br>
<a class="moz-txt-link-abbreviated" href="mailto:389-devel@redhat.com">389-devel@redhat.com</a>
    <br>
<a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/fedora-directory-devel">https://www.redhat.com/mailman/listinfo/fedora-directory-devel</a>
    <br>
  </blockquote>
  <br>
  <pre wrap="">
<hr size="4" width="90%">
--
389-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:389-devel@redhat.com">389-devel@redhat.com</a>
<a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/fedora-directory-devel">https://www.redhat.com/mailman/listinfo/fedora-directory-devel</a>
  </pre>
</blockquote>
<br>
</body>
</html>