On Thu, Apr 16, 2009 at 17:52, Corinna Vinschen wrote: > > [ "${opt_force}" = "yes" ] && opt_f=-f > [ -n "${user_account}" ] && opt_u="-u ""${user_account}""" > csih_select_privileged_username ${opt_f} ${opt_u} > > Shouldn't the last line read > > csih_select_privileged_username ${opt_f} ${opt_u} sshd >
Yes, I think it is yet another improvement. That would make the script try to reuse the service account if it already is installed... besides the display improvement (explicit reference to the sshd service). I just didn't add that because it was not being used in the original code, and I though that there would be a design reason behind that - like, for example, always force the privileged account selection algorithm to run, even if there is a service installed? That's your call. For me it's fine either way. ___________ Julio Costa -- Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple Problem reports: http://cygwin.com/problems.html Documentation: http://cygwin.com/docs.html FAQ: http://cygwin.com/faq/