<!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">
Your fix looks good. <br>
<br>
It'd be taken care in the ds_newinst script or setuputil (I mean, not
in create_instance.c, thus it won't affect your proposal), but
following the current info file format: e.g.,<br>
SecurityOn= Yes<br>
DisableSchemaChecking= No<br>
it'd be better supporting the same value set.<br>
install_full_schema= Yes / No<br>
<br>
And by default, the value should be Yes? Looking at this coding, if
InstallFullSchema does not exist in the info file, it's set NULL
(install_full_schema=NULL).<br>
<pre>@@ -4490,6 +4504,7 @@
return 1;
}
cf->start_server = ds_a_get_cgi_var("start_server", NULL, NULL);
+ cf->install_full_schema = ds_a_get_cgi_var("install_full_schema", NULL, NULL);
cf->secserv = ds_a_get_cgi_var("secserv", NULL, NULL);</pre>
As being done for start_server, we could force to set 1 in the install
script by default (as follows in ds_newinst.pl), but it'd be
straightforward for the front-end scripts and the back-end program
(create_instance.c) to agree on the default value, I think.<br>
<blockquote># if for some reason you do not want the server started
after instance creation<br>
# the following line can be commented out - NOTE that if you are
creating the<br>
# Configuration DS, it will be started anyway<br>
if (defined($table{"slapd"}->{"start_server"})) {<br>
$cgiargs{start_server} = $table{"slapd"}->{"start_server"};<br>
} else { # default is on<br>
$cgiargs{start_server} = 1;<br>
}<br>
</blockquote>
Thanks,<br>
--noriko<br>
<br>
Andrew Bartlett wrote:
<blockquote cite="mid:1179720974.12583.67.camel@localhost.localdomain"
type="cite">
<pre wrap=""><a class="moz-txt-link-freetext" href="https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239765">https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239765</a>
Resolves: bug #239765
Bug Description: Allow mimimum schema in ds_newinst.pl
Reviewed by: ???
Files:<a class="moz-txt-link-freetext" href="https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=155069">https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=155069</a>
Branch: HEAD
Fix Description: Patch to add an option for installing partial/full schema
This patch implements a new configuration option
[slapd]
install_full_schema= 1
Setting this to 0 will only install 00core.ldif
Platforms tested: Fedora Core 5 (with Samba4 as testing client)
Flag Day: no
Doc impact: yes - wiki on install inf settings needs update
Thanks,
Andrew Bartlett
</pre>
<pre wrap="">
<hr size="4" width="90%">
--
Fedora-directory-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Fedora-directory-devel@redhat.com">Fedora-directory-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>