Friday, April 29, 2016

[389-commits] admserv/newinst

admserv/newinst/src/ | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit e5b5830d74879aa1cc2cea0fe6d1bb1531f1185c
Author: Noriko Hosoi <>
Date: Wed Apr 27 14:26:32 2016 -0700

Ticket #48429 - running multiple times will make it so you cannot install DS

Bug Description: resets the files in /etc/dirsrv/admin-
serv and put it to the original state just after "rpm -i 389-admin" is

To recover the original state, restores the config
files backed up by That is, if is
executed without or is repeatedly
run, the config files are failed to restore. It makes the following fail.

Fix Description: This patch changes the behaviour so
that the config files are not touched regardless of the existence of the
back up. If the back up exists, the files are overridden by the back up.
But even if it does not exist, the current config files remain in the
directory, which leads the next successfully finish.

Reviewed by (Thank you, Ludwig!!)

diff --git a/admserv/newinst/src/ b/admserv/newinst/src/
index 3e31e70..480fb3a 100644
--- a/admserv/newinst/src/
+++ b/admserv/newinst/src/
@@ -826,9 +826,9 @@ sub removeAdminServer {

# remove config files
- my @savefiles = @savesecfiles; # save security files by default
+ my @savefiles = (@savesecfiles, @saveconffiles); # save security and conf files by default
if ($all) {
- @savefiles = (); # $all means remove everything, save nothing
+ @savefiles = @saveconffiles; # $all means remove everything, except the files in rpm.
if (opendir(CONFDIR, $configdir)) {
while ($file = readdir(CONFDIR)) {

389-commits mailing list

No comments:

Post a Comment