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

Switch from unmaintained lash library to the more active ladish one #210

Closed
GvMariani opened this issue Jun 23, 2022 · 2 comments
Closed

Comments

@GvMariani
Copy link

The lash library, used for session management with jack sound server, is very old (last release in 2009) and unmaintained (no commit in the upstream git since July 2009).
Luckily there is a viable replacement: the Nedko Arnaudov's LADI Session Handler (https://github.com/LADI/ladish). It's a complete rewrite of lash and is much more active (last git commit was in February 2021); it's already used by Carla and Cadence packages from Kxstudio project. It also sports an handy compatibility layer (liblash): all it should be needed to use it is changing the references to lash and its headers in those for the ladish compatibility layer and its headers.

The test patch attached simply does this, without bothering to change "lash" in "ladish" in the messages: it builds and runs OK for simple use on my system...
amsynth-1.12.4-use-ladish-for-lash.txt

Please consider the switch...

Information

  • Which version of amsynth are you using? 1.12.4
  • Which build of amsynth are you using? (stand-alone application, LV2, DSSI, VST) standalone application
  • If using amsynth as a plugin, which host application are you using?
  • Which operating system & version are you using? Rosa Linux (a Mandriva derivative) Fresh 2021.1 on x86_64
@nickdowell
Copy link
Member

Sounds like a sensible change, and a good one for a new contributor!

@nickdowell
Copy link
Member

I think there is no need to change anything here.

If ladish is configured with ./waf configure --enable-liblash then amsynth will build against it.

The supplied patch did not work for me... lash (ladish) was not detected because there is no liblash.pc installed by ladish; it installs /usr/local/lib/pkgconfig/lash-1.0.pc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants