-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 09/25/2009 06:53 AM, Jakub Hrozek wrote: > On 09/25/2009 12:22 PM, Stephen Gallagher wrote: >> Actually, I would prefer that we use this tool for potential future >> upgrades (that I hope never to have...). It would be better for it to >> detect which version is currently in place and upgrade it from 1 to 2, >> then 2 to 3, etc. internally and then write out the current version. > >> This way we don't need to change the spec file again for that, as well. > > > OK > >>>> >>>>>> - when using '-f /somewhere/my.conf' the backup is created in /somewhere >>>>>> but the new config is written to /etc/sss/sssd.conf. My expectation was >>>>>> that the conversion would be done in-place. If you prefer it this way >>>>>> '--help' should say that the default output file is >>>>>> /etc/sss/sssd.conf >>>> >>>> >>>> Fixed, thanks >> Uh, this doesn't look right to me: >> + if options.filename and not options.outfile: >> + options.outfile = options.filename >> + if not options.outfile: >> + options.outfile = '/etc/sssd/sssd.conf' > >> I think that second if needs to be elif, otherwise you're always going >> to override setting options.outfile = options.filename > > > D'oh.. > > the logic is fixed (note the default options are /etc/sssd/sssd.conf and > None respectively) > > > Jakub
Ack from me. - ------------------------------------------------------------------------ _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel - -- Stephen Gallagher RHCE 804006346421761 Looking to carve out IT costs? www.redhat.com/carveoutcosts/ -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/ iEYEARECAAYFAkq8oloACgkQeiVVYja6o6OtxQCgmm3BbrXyFwxg1mS8Ospsg1PE rcgAoKniV6DaTOXqmYWEh6vutRqSZGCL =VCpY -----END PGP SIGNATURE----- _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel