-
Notifications
You must be signed in to change notification settings - Fork 171
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
Does 1.128 beta support Python 3.13 on Linux? #5722
Comments
I will try to test Py 3.13 support in fedora 41 (as it is available there, it is the default) later this week. |
It should work. I tested it on mate's pc under win10. But next year we will have 3.14 and its not supported yet, its api has hard changes. |
does it work? |
Been so busy with work that I've not gotten back to this. So I've not tried yet. |
Without cuda LSP
|
But then after reinstalling cuda LSP
|
But that might be a different issue:
|
|
|
@ildarkhasanshin can you try to update Pydantic module in LSP from v1.10.x to v2 ? get v2 in PyPI.org. |
@bogen85 I got advice that you must upgrade to the RELEASE of 3.13 (you have beta version). |
@Alexey-T Thanks. Yeah, distrobox is broken on Opensuse so could not quickly try it with Fedora 41, so I added one of the OpenSuse language devel repos to get python 3.13. If anyone here tries this with distrobox before I get to it, use create with the |
My bad, I did not have access to my proper setup to test this. I will try to test this with the 3.13 release when I get a chance. |
Alright, got fedora 41 working with rootlless distrobox on opensuse. So I ran 1.128 beta in that.
I'll reinstall the cuda LSP plugin. $ python --version
Python 3.13.0
|
Without Cuda LSP
And with Cuda LSP reinstalled.
|
I was not sure what to do about the missing sudo dnf install python3-legacy-cgi |
I did not remove
Not sure why |
cgi module was removed from 3.13. maybe we can include it to folder 'lsp_modules'. but that will conflict with module from 3.12 distro. maybe we can include folder 'lsp_modules313' with 'cgi.py'. language.py has ver = sys.version_info
if (ver.major, ver.minor) < (3, 7):
modules36_dir = os.path.join(os.path.dirname(os.path.realpath(__file__)), 'lsp_modules36')
#sys.path.append(modules36_dir)
# instead of append use insert
sys.path.insert(0, modules36_dir) |
made this change. updated LSP plugin. pls try with removed |
|
Yes, those are fixed. See above output, warnings are not there. I should have noted that. |
|
Is module cgi found now (remove it from OS first)? |
I removed it from the OS, restarted Cuda, got the no cgi module found error.
|
Ok. Issue solved. |
I've not touched project_man code yet. I'll try to look at that later. I get that in 3.12 as well.
|
Ok. In python we can add some small code to print the stacktrace. I suggest to do it near the place which prints 'Saving project'. |
Sure. I don't have time now, I will look at it later. |
We will discuss it in #5733. closing this issue. |
beta files:
https://github.com/Alexey-T/CudaText/discussions/5720
Pls check somebody the support of Py 3.13 (which is released this week). I don't know how to test it with Ubuntu which has 3.10. maybe ArchLinux is good to have here.
The text was updated successfully, but these errors were encountered: