Bug 214733

Summary: be able to pass in all configurable paths to ds_newinst
Product: [Retired] 389 Reporter: Rich Megginson <rmeggins>
Component: Install/UninstallAssignee: Rich Megginson <rmeggins>
Status: CLOSED CURRENTRELEASE QA Contact: Viktor Ashirov <vashirov>
Severity: medium Docs Contact:
Priority: medium    
Version: 1.0.2CC: herrold, nhosoi, nkinder
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-12-07 17:05:53 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 152373, 240316, 427409    
Attachments:
Description Flags
diffs for fix none

Description Rich Megginson 2006-11-09 02:48:10 UTC
We need to be able to pass in all configurable paths to ds_newinst, such as
sysconfdir, localstatedir, db_dir, config_dir, etc.

Comment 1 Rich Megginson 2006-11-09 02:48:11 UTC
Created attachment 140740 [details]
diffs for fix

Comment 2 Rich Megginson 2006-11-09 02:50:27 UTC
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.

Comment 3 Noriko Hosoi 2006-11-09 07:11:23 UTC
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);


Comment 4 Rich Megginson 2006-11-09 15:42:21 UTC
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


Comment 5 Rich Megginson 2006-11-10 04:24:51 UTC
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