Re: [389-devel] Please review: Add update code - make setup-ds.pl -u do updates

[Rich Megginson]

Noriko Hosoi wrote:
>  On 09/17/2009 04:12 PM, Rich Megginson wrote:
> >
> > ------------------------------------------------------------------------
> >
> > --
> > 389-devel mailing list
> > 389-devel(a)redhat.com
> > https://www.redhat.com/mailman/listinfo/fedora-directory-devel
> >   
>  Hi Rich,
>
>  I noticed these 2 minor things.  Other than these, your patch looks good. 
Thanks. 
Fixed and pushed to master.
To ssh://git.fedorahosted.org/git/389/ds.git
   2de80f5..cc89083  master -> master
commit cc89083f1177606d4cbbb52f8cdc5e34d0d16f70
Author: Rich Megginson <rmeggins(a)redhat.com&gt;
Date:   Wed Sep 9 17:01:49 2009 -0600

    Add update code - make setup-ds.pl -u do updates

>  diff --git a/ldap/admin/src/scripts/Setup.pm.in 
>  b/ldap/admin/src/scripts/Setup.pm.in
>  index 7ad57c1..f314d67 100644
>  --- a/ldap/admin/src/scripts/Setup.pm.in
>  +++ b/ldap/admin/src/scripts/Setup.pm.in
>  @@ -142,6 +145,7 @@ sub init {
>       $self->{keep} = $keep;
>       $self->{preonly} = $preonly;
>       $self->{update} = $update;
>  +    $self->{force} = *$update;* <== could this be "$force"?
>       $self->{logfile} = $logfile;
>       $self->{log} = new SetupLog($self->{logfile});
>       # if user supplied inf file, use that to initializ
>
>
>  diff --git a/ldap/admin/src/scripts/dsupdate.map.in 
>  b/ldap/admin/src/scripts/dsupdate.map.in
>  new file mode 100644
>  +# Copyright (C) *2007* Red Hat, Inc. <== 2009? :)
>  +# All rights reserved.
>  +# END COPYRIGHT BLOCK
>
>  --noriko
>