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 nix environment and update script #171

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Conversation

jtojnar
Copy link
Contributor

@jtojnar jtojnar commented Feb 11, 2024

Nix package manager makes it easy to obtain all the typelibs without having to install them globally. Just run nix-shell (or enter the project directory if you have direnv) and then you will have the libraries available in the environment.

Then you can run tools/update.sh to re-generate all stubs.

@jtojnar jtojnar closed this Feb 11, 2024
@jtojnar
Copy link
Contributor Author

jtojnar commented Feb 11, 2024

Just using this for local development.

@jtojnar
Copy link
Contributor Author

jtojnar commented Feb 11, 2024

Tried regenerating all stubs that worked (GstPbutils generation segfaults) before and after #167 for comparison.

Only two stubs are invalid code (Gst #172 and Gtk3 #169) but that was a pre-existing condition.

@lovetox
Copy link
Collaborator

lovetox commented Feb 11, 2024

you need to use the -o argument, not output to a file.

The current stubs need to be parsed to find all override comments.

@lovetox
Copy link
Collaborator

lovetox commented Feb 11, 2024

Can you contain the nix stuff to some subfolder, maybe with a README how to use?

Further please separate this MR from updating the files.

@jtojnar
Copy link
Contributor Author

jtojnar commented Feb 11, 2024

I mostly just opened this to share the diff for #167

It probably does not make sense to merge the Nix environment unless someone else is interested in using it.

@lovetox
Copy link
Collaborator

lovetox commented Feb 11, 2024

I would keep the update.sh though, it makes it easier for people to execute all steps, they can comment out the libs they dont want to update.

@matperc
Copy link
Collaborator

matperc commented Feb 11, 2024

I love the idea of update.sh.

IMHO we should automate this to run every X days, to keep every stub in sync with upstream

@jtojnar
Copy link
Contributor Author

jtojnar commented Feb 11, 2024

Opened #174 with the update script.

@jtojnar jtojnar force-pushed the nix branch 2 times, most recently from 26cce52 to e586229 Compare February 25, 2024 22:06
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.

3 participants