Skip to content

nix: use ffmpeg-headless#1530

Open
nyakase wants to merge 1 commit into
9001:hovudstraumfrom
nyakase:ffmpeg-headless
Open

nix: use ffmpeg-headless#1530
nyakase wants to merge 1 commit into
9001:hovudstraumfrom
nyakase:ffmpeg-headless

Conversation

@nyakase

@nyakase nyakase commented Jun 16, 2026

Copy link
Copy Markdown

This PR complies with the DCO; https://developercertificate.org


I noticed the default build has a gigantic closure size of ~1.6GiB and wanted to reduce it, since the majority came from graphical libraries.

Swapping ffmpeg-full for ffmpeg-headless brings the closure size down to ~600MiB. Since late 2024 ffmpeg-headless has used withWebp and withOpenmpt (see NixOS/nixpkgs@193b980 and NixOS/nixpkgs@2fa42d7 respectively) so the reasons in favor of ffmpeg-full in the README do not apply anymore.

@9001

9001 commented Jun 16, 2026

Copy link
Copy Markdown
Owner

looks good, just pinging @toast003 and/or @chinponya for a quick opinion since they're the nix gurus (I don't really know anything about this stuff)

also, good time to ask -- now that copyparty is in the main nixos repo (iiuc), can some of the nix stuff in the copyparty repo be deleted, or does it still serve a purpose?

@JustTNE

JustTNE commented Jun 21, 2026

Copy link
Copy Markdown

also, good time to ask -- now that copyparty is in the main nixos repo (iiuc), can some of the nix stuff in the copyparty repo be deleted, or does it still serve a purpose?

The binary of copyparty is in the nixos repos now, but the "module" - basically what you use to configure copyparty on nixos - is not available there, which is a huge and important part of the nixos stuff in your repo. (I personally rely on the copyparty module for both the package itself as well as the module)

Although it would be possible to remove the package code from your repo, I would suggest keeping it as long as someone is here to maintain it for the simple reason that it is always up to date with the very latest changes to this repo and always in sync with the module version since both are from the same commit.

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