Bug 266008 - net/linphone: 4.4.8_1: broken plugin dir path
Summary: net/linphone: 4.4.8_1: broken plugin dir path
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Felix Palmen
URL:
Keywords:
Depends on:
Blocks: 266006
  Show dependency treegraph
 
Reported: 2022-08-24 03:14 UTC by Ruslan Zalata
Modified: 2022-09-05 16:50 UTC (History)
0 users

See Also:
zirias: maintainer-feedback+


Attachments
Patch CMakeLists.txt to set MSPLUGINS_DIR correctly (277 bytes, patch)
2022-08-24 03:52 UTC, Ruslan Zalata
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ruslan Zalata 2022-08-24 03:14:05 UTC
After upgrading to net/linphone 4.4.8 I discovered that Linphone no longer loads H264 plugin (msopenh264). Logging with -V has shown that mediastreamers is provided with bogus path to find its plugins:

[06:25:03:075][Info]Core:linphone: Loading ms plugins from [/usr/local]

It should be /usr/local/lib/mediastreamer/plugins/ instead.

I tracked it down to the below code that coins incorrect MS plugin dir path:

linphone-app/src/app/paths/Paths.cpp:

static inline QString getAppPackageMsPluginsDirPath () {
        QDir dir = getAppPackageDir();
        dir.cd(MSPLUGINS_DIR);
        return dir.absolutePath();
}
Comment 1 Ruslan Zalata 2022-08-24 03:52:08 UTC
Created attachment 236084 [details]
Patch CMakeLists.txt to set MSPLUGINS_DIR correctly

Ok, the problem caused by wrongly set MSPLUGINS_DIR is CMakeLists.txt. Here's the patch.
Comment 2 Felix Palmen freebsd_committer freebsd_triage 2022-08-24 06:41:41 UTC
This looks like another fallout from upstream's recent decision to only support building some AppImage (with all dependencies bundled).

Thanks for the patch, will test it soon!
Comment 3 Felix Palmen freebsd_committer freebsd_triage 2022-08-25 09:53:42 UTC
Patch tested, works for me! I prepared a commit with you as the author adding that patch. If I understood it correctly, it's also necessary to really fix your other PR 266006, so adding it as dependent here.

Therefore, I will wait for approval on the updates and fixes of the other ports and then commit everything in one batch. Thanks again for your work!
Comment 4 Ruslan Zalata 2022-08-25 11:21:36 UTC
Yes, that's right. Thank you too.
Comment 5 commit-hook freebsd_committer freebsd_triage 2022-09-05 16:37:24 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=de2c238aa0a69cd2dcf828f3ae0d80108194f44f

commit de2c238aa0a69cd2dcf828f3ae0d80108194f44f
Author:     Ruslan Zalata <rz@fabmicro.ru>
AuthorDate: 2022-08-24 03:52:08 +0000
Commit:     Felix Palmen <zirias@FreeBSD.org>
CommitDate: 2022-09-05 16:36:20 +0000

    net/linphone: Fix mediastreamer plugins dir

    PR:                     266008
    Approved by:            tcberner (mentor)
    Differential Revision:  https://reviews.freebsd.org/D36458

 net/linphone/Makefile                         |  2 +-
 net/linphone/files/patch-CMakeLists.txt (new) | 11 +++++++++++
 2 files changed, 12 insertions(+), 1 deletion(-)
Comment 6 Felix Palmen freebsd_committer freebsd_triage 2022-09-05 16:50:06 UTC
Patch committed, thanks!