Bug 368340 - Build fails with mediastreamer 2.14
Summary: Build fails with mediastreamer 2.14
Status: ASSIGNED
Alias: None
Product: kopete
Classification: Applications
Component: Jabber Plugin (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Kopete Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-09-06 14:28 UTC by Antonio Rojas
Modified: 2023-07-27 13:28 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
patch (3.21 KB, patch)
2016-09-07 21:24 UTC, Antonio Rojas
Details
Improved patch (4.58 KB, patch)
2016-09-27 13:05 UTC, Fabian Vogt
Details
Non-working skeleton patch against Debian version 4:17.08.3-1 (7.31 KB, patch)
2018-08-14 13:03 UTC, Felix Lechner
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Antonio Rojas 2016-09-06 14:28:16 UTC
kopete fails to build against mediastreamer 2.14

/build/kdenetwork-kopete/src/kopete-16.08.1/protocols/jabber/libjingle/talk/session/phone/linphonemediaengine.cc: In constructor 'cricket::LinphoneVoiceChannel::LinphoneVoiceChannel(cricket::LinphoneMediaEngine*)':
/build/kdenetwork-kopete/src/kopete-16.08.1/protocols/jabber/libjingle/talk/session/phone/linphonemediaengine.cc:156:37: error: cannot convert 'OrtpLogLevel' to 'const char*' for argument '1' to 'void ortp_set_log_level_mask(const char*, int)'
   ortp_set_log_level_mask(ORTP_FATAL);
                                     ^
/build/kdenetwork-kopete/src/kopete-16.08.1/protocols/jabber/libjingle/talk/session/phone/linphonemediaengine.cc:162:45: error: invalid conversion from 'int' to 'MSFactory* {aka _MSFactory*}' [-fpermissive]
   audio_stream_ = audio_stream_new(-1, -1, 0); /* -1 means that function will choose some free port */
                                             ^
/build/kdenetwork-kopete/src/kopete-16.08.1/protocols/jabber/libjingle/talk/session/phone/linphonemediaengine.cc:162:45: error: too few arguments to function 'AudioStream* audio_stream_new(MSFactory*, int, int, bool_t)'


Reproducible: Always
Comment 1 Antonio Rojas 2016-09-07 21:24:17 UTC
Created attachment 100980 [details]
patch

This patch fixes the build for me. I can't test that it works though, as I don't have a jabber account supporting jingle.
Comment 2 Fabian Vogt 2016-09-27 13:05:48 UTC
Created attachment 101315 [details]
Improved patch

The attached patch works also for building with old versions.
Comment 3 Pali Rohár 2016-10-23 15:05:23 UTC
Hi Fabian Vogt! Can you cleanup your patch and post it to git.reviewboard.kde.org to kopete group?
Comment 4 Fabian Vogt 2016-11-03 17:28:28 UTC
(In reply to Pali Rohár from comment #3)
> Hi Fabian Vogt! Can you cleanup your patch and post it to
> git.reviewboard.kde.org to kopete group?

I can do that, but I never tried to use it with linphone. If anyone can confirm that it still works, I'll submit it for review.
Comment 5 Pali Rohár 2016-11-03 17:40:13 UTC
Those are probably only compile errors... I can review changes.

Real testing will be harder as I have older libphone version which do not produce compile errors and it is working fine (testing kopete vs kopete). Also my automated builds of kopete on all supported and future ubuntu versions passe, see:
https://launchpad.net/~pali/+archive/ubuntu/kopete/+packages

So verification if compilation is really fixed must be done by somebody already affected by this bug. But I have no problem of merging this patch to kopete master as this I believe will be straightforward.
Comment 6 Fabian Vogt 2016-11-03 18:01:54 UTC
Ok, I made a review request on https://git.reviewboard.kde.org/r/129325/
Comment 7 David Geiger 2016-12-02 21:25:59 UTC
Hi,

We have now the same issue on Mageia6.
Comment 8 Pali Rohár 2016-12-05 16:32:51 UTC
(In reply to David Geiger from comment #7)
> We have now the same issue on Mageia6.

Problem is because external mediastreamer library changed its API.

I have not find any information about it yet, nor how to port old applications.

Proposed patch pass NULL pointer to some functions, but again I did not find anything about it, that this is really correct solution.

Passing NULL on "random" place without understood 'WHY?' is wrong solution.

I would suggest to report bug to mediastreamer project, that they broke API and existing application... And without providing any information what to do...
Comment 9 Felix Lechner 2018-08-14 13:03:19 UTC
Created attachment 114432 [details]
Non-working skeleton patch against Debian version 4:17.08.3-1

Hi, I am part of the Linphone team at Debian and tried to patch kopete. This bug is keeping us from upgrading Linphone. Unfortunately, I have unrelated build problems with kopete, and cannot get the build process to stop at an error. The patch is against Debian version 4:17.08.3-1.

I am posting my untested, non-working skeleton patch for now. I am happy to cooperate if upstream has additional pointers. Thank you!
Comment 10 Bernhard Schmidt 2018-10-11 19:51:32 UTC
Felix has created a patch that builds fine both against old and new mediastreamer2 libraries and submitted it to https://phabricator.kde.org/D15956
Comment 11 Bug Janitor Service 2023-07-27 13:28:49 UTC
A possibly relevant merge request was started @ https://invent.kde.org/network/kopete/-/merge_requests/10