Skip to content

Conversation

@serhiy-storchaka
Copy link
Member

@serhiy-storchaka serhiy-storchaka commented Dec 2, 2017

return SET_DECODE_ERROR("PYTHONEXECUTABLE environment "
"variable", len);
SET_DECODE_ERROR("PYTHONEXECUTABLE environment variable", len);
return -1;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why -1? Does the return matter?

@ned-deily
Copy link
Member

On macOS:

../../source/Modules/main.c:904:13: error: use of undeclared identifier 'pymain'
            SET_DECODE_ERROR("PYTHONEXECUTABLE environment variable", len);
            ^
../../source/Modules/main.c:48:13: note: expanded from macro 'SET_DECODE_ERROR'
            pymain->err = _Py_INIT_USER_ERR("cannot decode " #NAME); \
            ^
../../source/Modules/main.c:904:13: error: use of undeclared identifier 'pymain'
../../source/Modules/main.c:51:13: note: expanded from macro 'SET_DECODE_ERROR'
            pymain->err = _Py_INIT_NO_MEMORY(); \
            ^
../../source/Modules/main.c:905:20: error: returning 'int' from a function with incompatible result type '_PyInitError'
            return -1;
                   ^~
../../source/Modules/main.c:919:17: error: use of undeclared identifier 'pymain'
                SET_DECODE_ERROR("__PYVENV_LAUNCHER__ environment variable",
                ^
../../source/Modules/main.c:48:13: note: expanded from macro 'SET_DECODE_ERROR'
            pymain->err = _Py_INIT_USER_ERR("cannot decode " #NAME); \
            ^
../../source/Modules/main.c:919:17: error: use of undeclared identifier 'pymain'
../../source/Modules/main.c:51:13: note: expanded from macro 'SET_DECODE_ERROR'
            pymain->err = _Py_INIT_NO_MEMORY(); \
            ^
../../source/Modules/main.c:921:24: error: returning 'int' from a function with incompatible result type '_PyInitError'
                return -1;

@serhiy-storchaka serhiy-storchaka deleted the main-compile-error-osx branch December 2, 2017 18:40
@serhiy-storchaka
Copy link
Member Author

My attempt failed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge skip news type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants