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

Mixxx: 2.0.0 -> 2.1.0 #39106

Closed
wants to merge 3 commits into from
Closed

Mixxx: 2.0.0 -> 2.1.0 #39106

wants to merge 3 commits into from

Conversation

magnetophon
Copy link
Member

@magnetophon magnetophon commented Apr 18, 2018

Motivation for this change

Version 2.0.0 doesn't build on nixos-unstable, this does.
Needs #39105

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
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • 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.

@peterhoeg
Copy link
Member

Since qt4 is deprecated, please incorporate the mixxx changes from #33239 that move things to qt5.

@magnetophon
Copy link
Member Author

The mixxx documentation says that building with qt5 is experimental.
Not sure if that's desirable for a program meant for live performance.

@peterhoeg
Copy link
Member

2.1 is the last version with support for qt4 ref mixxxdj/mixxx#1526 (comment) so things are moving towards qt5. That being said, there seems to be an issue with qt 5.10.

I think you make a valid point - I'll update my PR to make qt5 optional so people can at least start trying out.

@@ -1,16 +1,19 @@
# set VAMP_PATH ?
# plugins availible on sourceforge and http://www.vamp-plugins.org/download.html (various licenses)

{ stdenv, fetchurl, pkgconfig, libsndfile }:
{ stdenv, fetchFromGitHub, pkgconfig, libsndfile }:

rec {

vampSDK = stdenv.mkDerivation {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you make this vampSDK = stdenv.mkDerivation rec { you can use version properly below.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm sorry, I don't get what you are saying here.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have just pushed a commit to your branch.

vampSDK = stdenv.mkDerivation {
name = "vamp-sdk-2.5";
vampSDK = stdenv.mkDerivation rec {
name = "vamp-sdk-${version}";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eval fails because version is undefined here (see ofborg checks).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@magnetophon
Copy link
Member Author

No longer relevant.

@magnetophon magnetophon closed this Sep 4, 2018
@magnetophon magnetophon deleted the mixxx branch September 4, 2018 21:41
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.

None yet

4 participants