Incorrect condition for lrelease/lrelease-qt5

DarkHobbit / doublecontact

Offline DE-independent Qt-based contact manager primarily for phonebooks editing/merging. Beta.

GNU General Public License v3.0

52 stars 11 forks source link

Incorrect condition for lrelease/lrelease-qt5 #45

Open bodqhrohro opened 4 years ago

bodqhrohro commented 4 years ago

Currently, the .pro file assumes that the binary is named lrelease-qt5 on non-macOS Unix-like systems, and lrelease elsewhere. Though I have lrelease binary in the qttools5-dev-tools package from Debian Bullseye. Is the condition not clarified enough, or maybe outdated then?

Actually, I guess that the best strategy would be to check for both variants, and use whatever is present in the system, rather than try to deduce it from unrelated parameters like the platform.

cppmm commented 1 year ago

Also on my system lrelease is placed in /usr/lib64/qt5/bin/lrelease It don't work with both lrelease and lrelease-qt5. I was able to compile the programm only with full path.