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, "f

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,

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 migh

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 && [ $va

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 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 scr

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 s

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

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:

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 "${SYSCO

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}/

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 op

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: http://cygwin.com

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 >

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

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 anymor

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 dos-f

[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"