Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit eba57b6

Browse files
committed
Add comments about Windows in Modules/getpath.c
1 parent 7e817d0 commit eba57b6

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

Modules/getpath.c

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@
4848
* argv0_path. For prefix, the landmark's path is derived from the VPATH
4949
* preprocessor variable (taking into account that its value is almost, but
5050
* not quite, what we need). For exec_prefix, the landmark is
51-
* Modules/Setup. If the landmark is found, we're done.
51+
* pybuilddir.txt. If the landmark is found, we're done.
5252
*
5353
* For the remaining steps, the prefix landmark will always be
5454
* lib/python$VERSION/os.py and the exec_prefix will always be
@@ -89,6 +89,8 @@
8989
* directory). This seems to make more sense given that currently the only
9090
* known use of sys.prefix and sys.exec_prefix is for the ILU installation
9191
* process to find the installed Python tree.
92+
*
93+
* NOTE: Windows MSVC builds use PC/getpathp.c instead!
9294
*/
9395

9496
#ifdef __cplusplus
@@ -134,7 +136,10 @@ static wchar_t *lib_python = L"lib/python" VERSION;
134136
/* In principle, this should use HAVE__WSTAT, and _wstat
135137
should be detected by autoconf. However, no current
136138
POSIX system provides that function, so testing for
137-
it is pointless. */
139+
it is pointless.
140+
Not sure whether the MS_WINDOWS guards are necessary:
141+
perhaps for cygwin/mingw builds?
142+
*/
138143
#ifndef MS_WINDOWS
139144
static int
140145
_wstat(const wchar_t* path, struct stat *buf)

0 commit comments

Comments
 (0)