Re: Warp: RSS Package Manager

2008/5/31 Isak Savo <[email protected]>:

> On Sat, May 31, 2008 at 2:29 PM, Neil Munro <[email protected]> wrote:
> >
> > The code is used was this:
> > if ( Environment.OSVersion.ToString( ).ToLower( ).Contains( "unix" ) )
> > {
> >      string sArgs = "+x " + sFileName;
> >
> >      Process pSetPermissions = new Process( );
> >      pSetPermissions.StartInfo.FileName = "chmod";
> >      pSetPermissions.StartInfo.Arguments = sArgs;
> >
> >      pSetPermissions.Start( );
> >      pSetPermissions.WaitForExit( );
> > }
> >
>
> Looks ok. (Except perhaps the hungarian notation for variable names..
> *shrug*)


I adopted hugarian notation at Uni, they like it when you use recognizable
styles.

>
>
> I'd probably try to do a more reliable OS check. I would not count on
> all OS version strings where chmod is used contains the string "unix".


So far I have only observed windows or unix being returned as OS strings.

>
>
> -Isak
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: autopackage-dev-unsubscribe-OfajU3CKLf1/[email protected]
> For additional commands, e-mail: autopackage-dev-help-OfajU3CKLf1/[email protected]
>
>