Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Python: 'Py_SetPythonHome' is deprecated since 3.11 #3214

Open
XVilka opened this issue Aug 2, 2023 · 3 comments · May be fixed by #3322
Open

Python: 'Py_SetPythonHome' is deprecated since 3.11 #3214

XVilka opened this issue Aug 2, 2023 · 3 comments · May be fixed by #3322
Labels

Comments

@XVilka
Copy link
Member

XVilka commented Aug 2, 2023

/Users/user/Rizin/cutter/src/common/PythonManager.cpp:61:9: warning: 'Py_SetPythonHome' is deprecated [-Wdeprecated-declarations]
        Py_SetPythonHome(pythonHome);
        ^
/opt/homebrew/Frameworks/Python.framework/Versions/3.11/include/python3.11/pylifecycle.h:40:1: note: 'Py_SetPythonHome' has been explicitly marked deprecated here
Py_DEPRECATED(3.11) PyAPI_FUNC(void) Py_SetPythonHome(const wchar_t *);
^
/opt/homebrew/Frameworks/Python.framework/Versions/3.11/include/python3.11/pyport.h:336:54: note: expanded from macro 'Py_DEPRECATED'
#define Py_DEPRECATED(VERSION_UNUSED) __attribute__((__deprecated__))
                                                ^

https://docs.python.org/3/c-api/init.html#c.Py_SetPythonHome
https://docs.python.org/3/c-api/init_config.html#init-config

Originally posted by @XVilka in #3210 (comment)

@XVilka XVilka added the Python label Aug 2, 2023
@r3yc0n1c
Copy link

wanna give this one a try, can you assign it to me please?

@karliss
Copy link
Member

karliss commented Mar 19, 2024

You can try, but take into account that you need to keep the compatibility with older versions. Also it is marked as part of STABLE python ABI, so it probably will remain available until Python 4 at which point who knows what else will they change.

@r3yc0n1c
Copy link

Sure thing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants