Bug 214733 - be able to pass in all configurable paths to ds_newinst
Summary: be able to pass in all configurable paths to ds_newinst
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: 389
Classification: Retired
Component: Install/Uninstall
Version: 1.0.2
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Rich Megginson
QA Contact: Viktor Ashirov
URL:
Whiteboard:
Depends On:
Blocks: 152373 240316 FDS1.1.0
TreeView+ depends on / blocked
 
Reported: 2006-11-09 02:48 UTC by Rich Megginson
Modified: 2015-12-07 17:05 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-12-07 17:05:53 UTC
Embargoed:


Attachments (Terms of Use)
diffs for fix (1.89 KB, patch)
2006-11-09 02:48 UTC, Rich Megginson
no flags Details | Diff

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



Note You need to log in before you can comment on or make changes to this bug.