2008/4/29 Sakesun Roykiattisak <[EMAIL PROTECTED]>:
>  are you sure the patch is right ?
>
>  @@ -146,7 +146,7 @@
>         self.exclude = None
>         self.getTestCaseNamesCompat = False
>         self.includeExe = env.get('NOSE_INCLUDE_EXE',
>  -                                sys.platform == 'win32')
>  +                                sys.platform == ('win32', 'cli'))
>
>
>
>  Isn't it should be "sys.platform in ('win32', 'cli'))" ?

Oops, you are absolutely right. (Die in shame.)

For my defence, this is a documentation, not a code. (It is a
documentation including the copy of code to explain how exactly config
is set...) I did run full tests before submitting the patch, so
executable part is correct...

/me dies in shame anyway.

-- 
Seo Sanghyeon
_______________________________________________
Users mailing list
[email protected]
http://lists.ironpython.com/listinfo.cgi/users-ironpython.com

Reply via email to