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

gst-plugins-base: enable graphene and re-enable gstgl on aarch64-darwin #223248

Merged

Conversation

lilyinstarlight
Copy link
Member

@lilyinstarlight lilyinstarlight commented Mar 26, 2023

Description of changes

Graphene is packaged now and gstreamer-gl is no longer broken on darwin. This is needed for some plugins in gst-plugins-rs. I also re-enabled media-gstreamer in gtk4 on darwin since theoretically it should work now and I can confirm it builds fine on aarch64-darwin

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.05 Release Notes (or backporting 22.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

@lilyinstarlight lilyinstarlight force-pushed the fix/gst-plugins-base-darwin-gstgl branch from b93b985 to 8bbb9ca Compare March 26, 2023 23:38
@lilyinstarlight
Copy link
Member Author

lilyinstarlight commented Mar 26, 2023

Well it looks like although the original issue that the comment mentions is no longer present, it doesn't actually work without the darwin SDK bump and trying to build with a newer SDK than default for stdenv seems non-viable

So until the x86_64-darwin SDK gets bumped, I guess we'll just leave gstreamer-gl disabled on x86_64-darwin for now (but it seems fine enabled for aarch64-darwin)

When I make the gst-plugins-rs PR, I'll just have it disable the specific plugins that depend on gstreamer-gl (and have it hacky-patch it out of meson.build) and make a TODO note to revisit after the darwin SDK bump

@lilyinstarlight lilyinstarlight marked this pull request as ready for review March 26, 2023 23:42
@lilyinstarlight lilyinstarlight changed the title gst-plugins-base: enable graphene and re-enable gstgl on darwin gst-plugins-base: enable graphene and re-enable gstgl on aarch64-darwin Mar 27, 2023
@lovesegfault lovesegfault self-requested a review March 28, 2023 11:24
@lovesegfault
Copy link
Member

Is there a PR/issue for the SDK bump you can link to? Is it currently in staging?

@lilyinstarlight
Copy link
Member Author

Is there a PR/issue for the SDK bump you can link to? Is it currently in staging?

It'll still be a while, it's been in progress for a long time. See #101229 and it seems there was a recently opened one #222717

@vcunat vcunat merged commit 883780b into NixOS:staging Apr 16, 2023
@lilyinstarlight lilyinstarlight deleted the fix/gst-plugins-base-darwin-gstgl branch April 16, 2023 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants