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

lmms: Add support for Carla #223103

Merged
merged 3 commits into from Apr 2, 2023
Merged

lmms: Add support for Carla #223103

merged 3 commits into from Apr 2, 2023

Conversation

Rampoina
Copy link
Contributor

@Rampoina Rampoina commented Mar 25, 2023

Add Carla as an optional dependency and the needed patch.
This adds the Carla rack and Carla instrument LMMS plugins.

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.

Add Carla as an optional dependency and the needed patch
@orivej
Copy link
Contributor

orivej commented Mar 28, 2023

Here is the upstream issue: LMMS/lmms#6395
And here is the corresponding patch: LMMS/lmms#6403
Could you apply it with fetchfatch? (https://github.com/LMMS/lmms/commit/3518d307f47c79db06bd0e9430f39b91488673dc.patch)
Fixes with sed are difficult to maintain, it is hard to know when they become obsolete.

@Rampoina
Copy link
Contributor Author

@orivej thanks for the suggestion, I'll work on that this friday

@orivej
Copy link
Contributor

orivej commented Mar 29, 2023

Thanks! Your help is welcome!

@Rampoina
Copy link
Contributor Author

Here is the upstream issue: LMMS/lmms#6395 And here is the corresponding patch: LMMS/lmms#6403 Could you apply it with fetchfatch? (https://github.com/LMMS/lmms/commit/3518d307f47c79db06bd0e9430f39b91488673dc.patch) Fixes with sed are difficult to maintain, it is hard to know when they become obsolete.

@orivej It looks like the files plugins/carlabase/carla.h were renamed to camelcase at some point so the patch doesn't apply.

Even if we change the names it still doesn't apply because of this line (and others) which comes from this feature

This patch from Arch Linux would apply or we could also include it ourselves.
How do you suggest to proceed?

@orivej
Copy link
Contributor

orivej commented Apr 1, 2023

Good idea! However, the link https://raw.githubusercontent.com/archlinux/svntogit-community/packages/lmms/trunk/lmms-carla-export.patch will expire as soon as ArchLinux deletes that patch (probably when they update lmms). To keep the link working we refer to patches on github by their tags or hashes rather than by their branch names (in this case the branch name is packages/lmms). To obtain a stable link you can open https://github.com/archlinux/svntogit-community/blob/packages/lmms/trunk/lmms-carla-export.patch , press y on the keyboard (or click … near "Go to file", choose "Copy permalink", and visit the copied link), and then click "Raw" and copy the link from the browser url bar. In this case the result is https://raw.githubusercontent.com/archlinux/svntogit-community/cf64acc45e3264c6923885867e2dbf8b7586a36b/trunk/lmms-carla-export.patch

@orivej-nixos orivej-nixos merged commit a24fa7a into NixOS:master Apr 2, 2023
8 checks passed
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

3 participants