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

flake.nix init #2

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

flake.nix init #2

wants to merge 1 commit into from

Conversation

BlaiZephyr
Copy link

this adds a simple flake.nix to use with nix develop or direnv.

nix run is borked - this issue seems to be nix's way of sandboxing, since we're patching rust-ffmpeg-sys it will fail.
related discussion:
https://discourse.nixos.org/t/rust-packaging-issue-unable-to-fetch-git-dependencies-correctly/46547

i dont really wanna create a PR just to update the .gitignore, so i'll keep it in mind if i actually do make a PR with more than 4 loc for that.

@BlaiZephyr
Copy link
Author

BlaiZephyr commented Jan 4, 2025

image

zwp_pointer_constraints_v1#21: error 1: the surface is already constrained
Protocol error 1 on object zwp_pointer_constraints_v1@21:
Protocol error 1 on object zwp_pointer_constraints_v1@21:

when using cargo watch and the example state proj.
unrelated to the actual PR but related to nix - just curious, maybe you have an idea

@Jupeyy
Copy link
Member

Jupeyy commented Jan 4, 2025

About the image... that is weird. seems like it seeks for a shared lib version of zstd, which makes the wasm mod expect an import from host side.
Not sure how to fix it, because there is no feature for the rust zstd crate that explicitly enables static builds.
According to gyscos/zstd-rs#159 (comment) we'd need to enable pkg-config feature to even have shared builds, and I don't see any code that could enable it.

So no idea sry :/ (Likely due to some weird pkg-config check)

zwp_pointer_constraints_v1#21: error 1: the surface is already constrained

Yeah that is the wayland thing we talked about

@BlaiZephyr
Copy link
Author

BlaiZephyr commented Jan 4, 2025

seems i'll need to dig a bit deeper since wasm is kinda a little important for development and modding

since we're in our early stages, are you fine with me putting this as draft? or would you rather have a new PR in the near future?

@Jupeyy
Copy link
Member

Jupeyy commented Jan 4, 2025

You can make it a draft until you feel it is ready ofc

@BlaiZephyr BlaiZephyr marked this pull request as draft January 4, 2025 15:21
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