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

Remove spellcheck.py and use distro supplied pygtkspellcheck. #734

Open
kathenas opened this issue Nov 23, 2023 · 3 comments
Open

Remove spellcheck.py and use distro supplied pygtkspellcheck. #734

kathenas opened this issue Nov 23, 2023 · 3 comments
Labels

Comments

@kathenas
Copy link
Contributor

Version: git

Remove spellcheck.py and use distro supplied pygtkspellcheck.

On Debian, there is no supported distro version that does not have python3-gtkspellcheck >= 4.0.5.

Note: I have tested Debian sid with spellcheck.py removed and using the distro package with no issues.

@jendrikseipp
Copy link
Owner

The file is bundled because it contains 2-3 patches that probably only makes sense for RedNotebook (see git log -p rednotebook/external/spellcheck.py). Testing whether they're still needed and/or upstreaming them doesn't seem worth the effort to me. And making sure that all Linux distros, macOS and Windows have a version of pygtkspellcheck that works would take a lot of time.

@kathenas
Copy link
Contributor Author

I can understand where you are coming from here. While having a patched version it is preferably to have it in the project code base. Over time I would hope time in the longer term can be made for a number of things.

  • Move to 5.0.x from upstream. Adds GTK 4 support amongst other things.
  • Checking if patches are still needed.
  • Test and approach upstream with possible bug fixes or applicable additions.

If the file could be removed one day with no headache, that would be good, but not something that is a must.

@jendrikseipp
Copy link
Owner

I agree that it would be nice to find time for this.

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

No branches or pull requests

2 participants