steinbergmedia / vstgui

A user interface toolkit mainly for audio plug-ins
Other
857 stars 122 forks source link

Use execvpe to execute zenity and kdialog #234

Open paulfd opened 2 years ago

paulfd commented 2 years ago

This removes the hard-coding of zenity or kdialog paths. Zenity is preferred still.

scheffle commented 2 years ago

@paulfd: Is 'execvpe' available on BSD? We have to make sure it works there too.

paulfd commented 2 years ago

I'll check it out, are there any particular version/OS you're targeting?

paulfd commented 2 years ago

Apparently it's a problem on FreeBSD so you were right to be wary! If you're OK in principle for the patch I can look into it to find a suitable, portable solution.

scheffle commented 2 years ago

Sure, a portable solution would be nice! Thanks.

redtide commented 1 year ago

I rebased the code to a recent commit, left the original execve for BSD and execvpe for Linux, which helps NixOS like distributions to find the KDialog/Zenity paths.