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

Add some missing sqlite3 symbols #4966

Closed
wants to merge 540 commits into from

Conversation

devdanzin
Copy link
Contributor

This PR adds:

  • sqlite3.SQLITE_CREATE_VTABLE
  • sqlite3.SQLITE_DROP_VTABLE
  • sqlite3.SQLITE_FUNCTION
  • sqlite3.SQLITE_RECURSIVE
  • sqlite3.SQLITE_SAVEPOINT

Addresses part of #4962.

@mattip
Copy link
Member

mattip commented May 24, 2024

Looks good. This should target py3.9

mattip and others added 29 commits May 24, 2024 04:56
cfbolz and others added 24 commits May 24, 2024 04:58
…returns a code

object that:
- raises an AssertionError when called
- produces the correct line number when doing so
…ame, PyErr_SetExcFromWindowsErrWithFilenameObject, PyErr_SetExcFromWindowsErrWithFilenameObjects (issue pypy#4035)
@devdanzin devdanzin changed the base branch from branches/py3.10 to branches/py3.9 May 24, 2024 08:18
@devdanzin
Copy link
Contributor Author

While I'm sure there's a way to re-target a PR to a new branch and rebase things, I'm failing at it. Will create a new one instead.

@devdanzin devdanzin closed this May 24, 2024
@mattip
Copy link
Member

mattip commented May 24, 2024

I do this kind of thing with git cherry-pick <commit> accross branches. You need to know which commits you did, then you can

git checkout py3.9
git pull
git checkout -b sqlite_symbols2
git cherry-pick <commit1>
git cherry-pick <commit2>
...
git push devdanzin

Then when you create the new PR you will only have the commits you want

@devdanzin devdanzin deleted the sqlite_symbols branch May 27, 2024 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants