[issue9752] MSVC Compiler warning in Python\import.c

2010-09-09 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc amaur...@gmail.com added the comment: The patch is good, and fixes an incorrect call to the mkdir function. -- nosy: +amaury.forgeotdarc resolution: - accepted ___ Python tracker rep...@bugs.python.org

[issue9752] MSVC Compiler warning in Python\import.c

2010-09-09 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc amaur...@gmail.com added the comment: BTW: should I be using the .patch extension or .diff extension? Both extensions are fine and handled the same way -- ___ Python tracker rep...@bugs.python.org

[issue9752] MSVC Compiler warning in Python\import.c

2010-09-09 Thread Daniel Stutzbach
Daniel Stutzbach dan...@stutzbachenterprises.com added the comment: Committed as r84659. Thanks for the patch! -- stage: patch review - committed/rejected status: open - closed type: - compile error ___ Python tracker rep...@bugs.python.org

[issue9752] MSVC Compiler warning in Python\import.c

2010-09-08 Thread Jon Anglin
Jon Anglin jang...@fortresgrand.com added the comment: Visual Studio ships with the source code for the CRT (\Program Files\Microsoft Visual Studio 9.0\VC\crt\src). I looked up _mkdir. It does just call CreateDirectory(path, NULL). If no error occurs it returns zero. If an error occurs, it

[issue9752] MSVC Compiler warning in Python\import.c

2010-09-08 Thread Brian Curtin
Brian Curtin cur...@acm.org added the comment: Thanks for looking into that. Since we now know that there is no use for the mode parameter on Windows, let's just remove the mode related stuff (of course leaving it for other OSes). We could then remove the macro and do the #ifdef dance around

[issue9752] MSVC Compiler warning in Python\import.c

2010-09-08 Thread Jon Anglin
Jon Anglin jang...@fortresgrand.com added the comment: How about this: see no-macro.diff BTW: should I be using the .patch extension or .diff extension? -- Added file: http://bugs.python.org/file18798/no-macro.diff ___ Python tracker

[issue9752] MSVC Compiler warning in Python\import.c

2010-09-07 Thread Jon Anglin
Jon Anglin jang...@fortresgrand.com added the comment: Windows provides two versions of mkdir in direct.h: int mkdir(const char* dirname) int _mkdir(const char* dirname) The latter is the preferred function because it is conformant to the ISO C++ standard. As you can see, neither

[issue9752] MSVC Compiler warning in Python\import.c

2010-09-07 Thread Brian Curtin
Brian Curtin cur...@acm.org added the comment: What about using CreateDirectory? -- components: +Build nosy: +brian.curtin ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue9752 ___

[issue9752] MSVC Compiler warning in Python\import.c

2010-09-07 Thread Daniel Stutzbach
Daniel Stutzbach dan...@stutzbachenterprises.com added the comment: As far as I can tell, _mkdir(name) is equivalent to CreateDirectoryA(name, NULL), except one uses errno and the other uses GetLastErrno. It is definitely possible that there's something I don't know, though, and the

[issue9752] MSVC Compiler warning in Python\import.c

2010-09-03 Thread Daniel Stutzbach
New submission from Daniel Stutzbach dan...@stutzbachenterprises.com: 7..\Python\import.c(1158) : warning C4013: 'mkdir' undefined; assuming extern returning int -- assignee: stutzbach components: Interpreter Core, Windows keywords: easy messages: 115428 nosy: stutzbach priority: low