We need to be able to pass in all configurable paths to ds_newinst, such as sysconfdir, localstatedir, db_dir, config_dir, etc.
Created attachment 140740 [details] diffs for fix
Just add all of the paths that are set-able in create_instance.c to ds_newinst.pl. The paths will be parsed from the input .inf file e.g. config_dir= /path/to/config sysconfdir= /path/to/sysconf etc. in the [slapd] section.
Looks good. Thank you, Rich! I've just noticed that sasl_path is not on the list that can be specified in the inf file. Do we want to do it? Or there's no such need? > cf->sasl_path = PR_smprintf("%s%csasl2", cf->sroot, FILE_PATHSEP);
Reviewed by: nhosoi (Thanks!) Fix Description: In addition to https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214733#c2 I added sasl_path: + $cgiargs{"sasl_path"} = $table{"slapd"}->{"sasl_path"}; Files: ds_newinst.pl Branch: HEAD Tested on: RHEL4 Flag day: no Checking in ds_newinst.pl; /cvs/dirsec/ldapserver/ldap/admin/src/ds_newinst.pl,v <-- ds_newinst.pl new revision: 1.10; previous revision: 1.9 done
There were a couple of other minor problems with ds_newinst.pl 1) Should only set cgi parameters for those items which have a value 2) Should not setenv NETSITE_ROOT - this overrides prefix settings 3) Should not set prefix here - it is set elsewhere Checking in ds_newinst.pl; /cvs/dirsec/ldapserver/ldap/admin/src/ds_newinst.pl,v <-- ds_newinst.pl new revision: 1.11; previous revision: 1.10 done