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
Fix/bitwig #50652
Fix/bitwig #50652
Conversation
1668a3b
to
fe12922
Compare
@@ -11157,7 +11157,9 @@ with pkgs; | |||
|
|||
libxdg_basedir = callPackage ../development/libraries/libxdg-basedir { }; | |||
|
|||
libxkbcommon = callPackage ../development/libraries/libxkbcommon { }; | |||
libxkbcommon = libxkbcommon_8; | |||
libxkbcommon_8 = callPackage ../development/libraries/libxkbcommon { }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Introducing libxkbcommon_8
and aliasing it to libxkbcommon
shouldn't be done for something as trivial as this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Mic92 Seems unnecessary to me, you disagree?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is quite a common pattern in nixpkgs that we keep have aliases like that even if there are only two versions.
"-Dx-locale-root=${libX11.out}/share/X11/locale" | ||
]; | ||
|
||
doCheck = false; # fails, needs unicode locale |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't reproduce locally.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah this came in after the version bump, according to git history.
fe12922
to
9820c38
Compare
as @worldofpeace pointed out there are some build configuration which are not correct, this was due to the fact, I used the build script for version 8 to build version 7. I fixed it by putting the build script of the older version 7 in place (from the git-history). |
Because the Borg has added some higher rebuild labels(501+) this should probably be delivered to |
In principle for the purposes stated in the PR this shouldn't need to rebuild anything except |
@@ -1,7 +1,8 @@ | |||
{ stdenv, fetchurl, fetchpatch, meson, ninja, pkgconfig, yacc, xkeyboard_config, libxcb, libX11, doxygen }: | |||
|
|||
stdenv.mkDerivation rec { | |||
name = "libxkbcommon-0.8.0"; | |||
name = "libxkbcommon-${version}"; | |||
version = "0.8.0"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is causing the mass-rebuild.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you undo it, we can merge it into master.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DONE 👍
9820c38
to
3a94627
Compare
@GrahamcOfBorg build libxkbcommon_7 |
Success on aarch64-linux (full log) Attempted: libxkbcommon_7 Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: libxkbcommon_7 Partial log (click to expand)
|
Looks good, this needs a backport to 18.09 I presume. |
Motivation for this change
fixing #49530
I needed to split up libxkbcommon to version 8 and version 7.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)