diff options
| author | Rich Megginson <rmeggins@redhat.com> | 2007-06-29 21:12:22 +0000 |
|---|---|---|
| committer | Rich Megginson <rmeggins@redhat.com> | 2007-06-29 21:12:22 +0000 |
| commit | ad85c482b235c603bd29da7d754415065400bdf1 (patch) | |
| tree | b300f61426dccbe71e8e83a6567d42026089cd6c /ldap/admin/src/create_instance.c | |
| parent | da39f866049f4416691e77c1aa71a1643fdccbf9 (diff) | |
| download | ds-ad85c482b235c603bd29da7d754415065400bdf1.tar.gz ds-ad85c482b235c603bd29da7d754415065400bdf1.tar.xz ds-ad85c482b235c603bd29da7d754415065400bdf1.zip | |
Resolves: bug 245815
Description: DS Admin Migration framework
Reviewed by: nhosoi (Thanks!)
Fix Description: Created a Migration class that is very similar to the Setup class - to act as a sort of global context for the migration process. Moved most of the guts of migrateTo11 into the new DSMigration class and the new migrate-ds.pl - we should deprecate migrateTo11 in favor of migrate-ds.pl. I had to enhance the check_and_add_entry function to handle pseudo-LDIF change records - pseudo because mozilla perldap LDIF has no real LDIF change record support.
Fixed a bug in create_instance.c - creating an instance without starting it was not working if the port number of an existing directory server was supplied.
Added a new method createDSInstance to Util - this just wraps ds_newinst.pl for now.
Platforms tested: RHEL4
Doc: Yes. We will need to document the migration procedures.
Flag day: Yes. Autotool file changes.
Diffstat (limited to 'ldap/admin/src/create_instance.c')
| -rw-r--r-- | ldap/admin/src/create_instance.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ldap/admin/src/create_instance.c b/ldap/admin/src/create_instance.c index f5421589..52641425 100644 --- a/ldap/admin/src/create_instance.c +++ b/ldap/admin/src/create_instance.c @@ -437,7 +437,7 @@ static char *sanity_check(server_config_s *cf, char *param_name) /* if we don't need to start the server right away, we can skip the port number checks */ - if (!needToStartServer(cf)) + if (needToStartServer(cf)) { if( (t = create_instance_checkports(cf))) { |