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

vlc: compile against qt 5.6 #23688

Closed
wants to merge 1 commit into from
Closed

vlc: compile against qt 5.6 #23688

wants to merge 1 commit into from

Conversation

peterhoeg
Copy link
Member

Motivation for this change

vlc 2.x doesn't work with qt 5.7.

@ttuegel, I just went ahead and merged a few other fixes for packages not supporting qt 5.7. Are you OK with that or do we need to handle such cases in a certain way?

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@ttuegel
Copy link
Member

ttuegel commented Mar 10, 2017

We need to mark packages broken when they don't compile with a particular version. Although this fixes the top-level vlc attribute, Hydra will hopelessly churn trying to build libsForQt57.vlc. The trick is to set meta.broken = builtins.compareVersion qtbase.version "5.7.0" >= 0.

@peterhoeg peterhoeg closed this Mar 13, 2017
@peterhoeg peterhoeg deleted the f/vlc branch March 13, 2017 03:56
@peterhoeg peterhoeg restored the f/vlc branch March 14, 2017 00:53
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.

None yet

2 participants