-
Notifications
You must be signed in to change notification settings - Fork 12
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
tip doesn't build #77
Comments
On second glance, it looks like we're updating source and not updating the hash that represents the locked cargo deps. AKA, this is likely a me issue. Closing for now. |
Looks like it might have been a legit report: nix-community/nixpkgs-wayland#452 (comment) reopened. |
From <https://github.com/russelltg/wl-screenrec/blob/v0.1.3/README.md#from-source-using-cargo>: > Install ffmpeg 6 or later, which is a required dependency. ffmpeg > 5 may work, but is untested (open an issue or PR if you test with > ffmpeg 5 so I can update these docs on if it works or not) I guess it worked… 7.x will need a bump to the ffmpeg-sys-next crate, but see <russelltg/wl-screenrec#77>.
wl-screenrec supports ffmpeg 7 (just pushed a release, but tip has supported it for a while). If someone is still running into this issue, can I get the --verbose output of cargo build? |
Uh, sorry about the commit spam, those are from a work‐in‐progress branch ^^; I’m still running into this with the new release; here’s the log (sorry for all the weird Nix stuff):
I guess this might be something to do with our Rust packaging system and default features? |
It seems cargo-auditable is failing due to those database errors. This seems unrelated to my project. |
No, that message is just an irrelevant warning. I’ve verified, however, that b817acc fixes the error. Could you cut a release with that commit included? |
Includes support for FFmpeg 7. We can’t use the release tag because of a Cargo build issue; see <russelltg/wl-screenrec#77>. Hopefully there’ll be a new release tag soon. Diff: <russelltg/wl-screenrec@v0.1.3...b817acc>
Includes support for FFmpeg 7. We can’t use the release tag because of a Cargo build issue; see <russelltg/wl-screenrec#77>. Hopefully there’ll be a new release tag soon. Diff: <russelltg/wl-screenrec@v0.1.3...b817acc>
0.1.5 is released |
Thanks! |
anyway, you get the idea. no idea what's going on.
The text was updated successfully, but these errors were encountered: