-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Migrate to new LV2 header paths #6990
Conversation
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.
LGTM
@@ -29,7 +29,7 @@ | |||
|
|||
#ifdef LMMS_HAVE_LV2 | |||
|
|||
#include <lv2/lv2plug.in/ns/ext/urid/urid.h> | |||
#include <lv2/urid/urid.h> |
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.
Looks like our automated Linux builds can't find this. This is probably another reason why our Linux builds need to be upgraded to ubuntu-20.04 or ubuntu-22.04, since I believe we're still using 18.04.
@wujekbrezniew Our Linux CI has been upgraded, so if you rebase on the latest master, this PR should build successfully on Linux now. |
Merge changes from base repo
Done c: |
Fix lv2 headers to comply to latest versions of lv2. It was neccesary for me to build LMMS on Ubuntu over KXStudio packages.
lv2-dev will change to 1.18.10 in next stable ubuntu release as launchpad says:
https://packages.ubuntu.com/search?keywords=lv2-dev