Page MenuHomePhabricator

Switch to using explicit dependency func instead of using find_library
ClosedPublic

Authored by JoshStrobl on Sep 15 2021, 3:26 PM.

Details

Summary

This fixes intl detection (under Solus, x86_64-solus-linux-gcc (Solus) 11.2.0).

Test Plan
  1. Compiled terminology without patch, failed to find intl and would not include gettext po files.
  2. Compiled with patch, successfully found intl.

Tested against efl 1.25.1 with terminology 1.9.0. Meson release is 1.59.1.

Diff Detail

Repository
rTRM apps/terminology
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
JoshStrobl requested review of this revision.Sep 15 2021, 3:26 PM
JoshStrobl created this revision.
billiob accepted this revision.Sep 19 2021, 12:37 PM
This revision is now accepted and ready to land.Sep 19 2021, 12:37 PM
This revision was automatically updated to reflect the committed changes.

I've done some changes to make it still work on some distro I use. Could you please test latest branch?

I've done some changes to make it still work on some distro I use. Could you please test latest branch?

Latest changes break intl detection again.

@JoshStrobl is it better now?

Can confirm that rTRMf11b69882808251577938fe8d520de4ad41a896c fixed it. Thanks!