Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-22 Thread Corinna Vinschen
On Aug 21 23:11, Charles Wilson wrote: Corinna Vinschen wrote: There are actually two questions: (a) should csih_check_access be checking that the Administrators group has the desired access?, and (b) are extended ACLs *actually* masked by the other bits? a) Actually, since all file

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-22 Thread Charles Wilson
Corinna Vinschen wrote: See Eric's comments. Especially the $var -eg foo is probably not a good idea. OK...I'm sure I /did/ try it that way, but got an error of some kind (related to fun_call, not the -eq test). But I'll try it again. Maybe I fat-fingered something the first time. BTW, foo

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-21 Thread Charles Wilson
Corinna Vinschen wrote: I'm about to create a new openssh package for 1.7 with fixes along the lines of what you sent in your OP. While I'm at it, I see that there's a bit of dual work in the csih-0.1.7 script and the ssh-host-config script: So I was also trying to update my iu-config

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-21 Thread Corinna Vinschen
On Aug 21 13:14, Charles Wilson wrote: Corinna Vinschen wrote: I'm about to create a new openssh package for 1.7 with fixes along the lines of what you sent in your OP. While I'm at it, I see that there's a bit of dual work in the csih-0.1.7 script and the ssh-host-config script: So

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-21 Thread Charles Wilson
Corinna Vinschen wrote: There are actually two questions: (a) should csih_check_access be checking that the Administrators group has the desired access?, and (b) are extended ACLs *actually* masked by the other bits? a) Actually, since all file access is using backup privileges,

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-21 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Charles Wilson on 8/21/2008 9:11 PM: if [ shell_fun_call -a $var -eq foo ] doesn't work. It should be if ( shell_fun_call [ $var -eq foo ]) You don't necessarily need the extra subshells. if shell_fun_call [ $var -eq foo ]

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-21 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Eric Blake on 8/21/2008 10:41 PM: You don't necessarily need the extra subshells. if shell_fun_call [ $var -eq foo ] is valid shell, provided $var is guaranteed to be non-empty, doesn't start with any character that might confuse

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-20 Thread Corinna Vinschen
Hi Chuck, On Aug 16 22:26, Charles Wilson wrote: Corinna -- Attached see (1) bugfix for ssh-user-config -- self-explanatory. (2) behavior enhancement for ssh-host-config Takes advantage of new functions and cleans up behavior under 1.7 -- esp. 'mount' doesn't have a -t option

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-20 Thread Charles Wilson
Corinna Vinschen wrote: I would like to propose to do all the work in the csih script, like this: csih_make_dir ${LOCALSTATEDIR}/run chmod 1777 ${LOCALSTATEDIR}/run csih_make_dir ${LOCALSTATEDIR}/log chmod 1777 ${LOCALSTATEDIR}/log csih_make_dir ${LOCALSTATEDIR}/empty chmod

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-20 Thread Corinna Vinschen
On Aug 20 12:35, Charles Wilson wrote: Corinna Vinschen wrote: I would like to propose to do all the work in the csih script, like this: [...] csih_make_dir ${SYSCONFDIR} chmod 755 ${LOCALSTATEDIR}/etc ^ I assume you mean ${SYSCONFDIR} here.

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-20 Thread Charles Wilson
Corinna Vinschen wrote: On Aug 20 12:35, Charles Wilson wrote: csih_make_dir ${SYSCONFDIR} - +chmod 755 ${LOCALSTATEDIR}/etc || /bin/true + I guess that should have been chmod 755 ${SYSCONFDIR} || /bin/true Err, yeah... I got two other problems: *** Query:

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-20 Thread Corinna Vinschen
On Aug 20 14:59, Charles Wilson wrote: Corinna Vinschen wrote: In the long run, it would be cool if the script would be domain aware, of course, but there might be some tweaking of tools necessary before we can do that. Yes, but I'm afraid I will be completely at the mercy of PTC for

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-18 Thread Corinna Vinschen
On Aug 16 22:26, Charles Wilson wrote: Corinna -- Attached see (1) bugfix for ssh-user-config -- self-explanatory. (2) behavior enhancement for ssh-host-config Takes advantage of new functions and cleans up behavior under 1.7 -- esp. 'mount' doesn't have a -t option anymore, so we

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-18 Thread Charles Wilson
Corinna Vinschen wrote: If this is really a cygcheck bug, we should fix the bug, rather than to workaround it here. Consider that Cygwin 1.7 hasn't been released yet. Well, yeah, but I wanted the script to work /now/, not whenever we can track down and fix the bug. Of course, nobody else is

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-18 Thread Corinna Vinschen
On Aug 18 08:10, Charles Wilson wrote: Corinna Vinschen wrote: If this is really a cygcheck bug, we should fix the bug, rather than to workaround it here. Consider that Cygwin 1.7 hasn't been released yet. Well, yeah, but I wanted the script to work /now/, not whenever we can track

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-18 Thread Christopher Faylor
On Mon, Aug 18, 2008 at 02:37:23PM +0200, Corinna Vinschen wrote: -o text exists since 2003. /me bows It's even documented: http://cygwin.com/cygwin-ug-net/using-utils.html#mount . cgf -- Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple Problem reports:

[ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-16 Thread cygwin
csih (cygwin-service-installation-helper) provides a library of shell functions that can be used by other cygwin packages that provide servers and daemons. It can assist in various service installation tasks, such as: * identifying the underlying Windows OS * detecting whether a privileged user

Re: [ANNOUNCEMENT] Updated: csih-0.1.7-1

2008-08-16 Thread Charles Wilson
Corinna -- Attached see (1) bugfix for ssh-user-config -- self-explanatory. (2) behavior enhancement for ssh-host-config Takes advantage of new functions and cleans up behavior under 1.7 -- esp. 'mount' doesn't have a -t option anymore, so we need another mechanism to deal with the