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
yabridge,yabridgectl: init at 3.0.0 #108835
Conversation
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 1 package built:
|
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 2 packages built:
|
db3752a
to
49b47b9
Compare
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 2 packages built:
|
FWIW, yabridge 3.0 is out, with support for VST3, might be worth updating the PR to directly go to that version, since this hasn't landed yet. |
@alpmestan Yep, I already have an update that I'm testing right now, and I'll submit soon. |
49b47b9
to
b773751
Compare
b773751
to
6a238b2
Compare
Something that may also be good to be aware of is that yabridge (or any application compiled with wineg++ that directly or indirectly uses |
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.
small nit. Trying to build it later. Otherwise LGTM.
6a238b2
to
a119043
Compare
a119043
to
ebbb927
Compare
Result of 2 packages built:
|
I just added a tiny workaround for the compile errors caused by that regression in Wine 6.2 here: robbert-vdh/yabridge@c29bc59 It might be a good idea to compile from that commit instead of from the 3.0.0 tag! There have been no other changes since version 3.0.0. |
@robbert-vdh Since we're using the stable version of wine (6.0) to compile yabridge, I think it should be ok to leave it as it is. |
Motivation for this change
Closes #108791
@thornycrackers I tried using Upright Piano through Reaper and everything seems to be working as expected:
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)