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 Makefile for rebuilding bindings #126

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

arnetheduck
Copy link
Contributor

Having a makefile is a nice little trick to re-run the binding generation code without having to copy-paste and/or remember the right docker incantations :)

Having a makefile is a nice little trick to re-run the binding
generation code without having to copy-paste and/or remember the right
docker incantations :)
@arnetheduck
Copy link
Contributor Author

Oops - forgot that docker mounts are owned by root by default - push a version that sets the user instead

@@ -46,3 +46,6 @@ examples/libraries/restricted-extras-qscintilla/restricted-extras-qscintilla
# android temporary build files
android-build
deployment-settings.json

*.docker-buildstamp
compile_flags.txt
Copy link
Owner

Choose a reason for hiding this comment

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

What is compile_flags.txt?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah, that's something clangd uses to provide (more) accurate intellisense - can remove or leave a comment, whichever you prefer.

@mappu
Copy link
Owner

mappu commented Jan 11, 2025

Thanks, this looks great. I have a similar helper script that sets up local disk paths for my dev environment, but having a good one in the repo already will be nice.

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.

2 participants