stefan-langenmaier / jami-overlay

Jami (formerly Ring (formerly SFLphone)) is a SIP compatible softphone for GNU/Linux.
GNU General Public License v3.0
22 stars 9 forks source link

net-voip/ring-daemon-20170803.2 fails to compile : missing Argon2 #38

Closed njuneau closed 7 years ago

njuneau commented 7 years ago

Hello Stefan,

I hope you are doing well! I tried to update to Ring 1.0 this morning and the ebuild failed at the configure phase with the following error:

configure: error: Missing Argon2

Here's the econf line:

'--without-opensl' '--with-alsa' '--with-dbus' '--with-gsm' '--without-jack' '--with-libilbc' '--with-natpmp' '--with-opus' '--without-portaudio' '--with-puls' '--without-restcpp' '--with-sdes' '--with-speex' '--with-speexdsp' '--with-upnp' '--enable-doxygen' '--enable-dot' '--enable-accel' '--enable-ipv6' '--enable-ringns' '--enable-vaapi' '--enable-video'

Do you have a similar issue on your end?

Thanks!

stefan-langenmaier commented 7 years ago

Salut Nicolas,

at the moment the bundled version of argon2 should be used again. Someone had this problem recently (#34) and I already created a reminder to fix it (#35). So no, I didn't have the problem when I built the package locally. Maybe you still have the argon2 package installed and this is confusing the build system.

I will have a closer look for what's the current state with the argon2 dep this week.

Thanks for the information et donne le bonjour à tout le monde!

YogSottot commented 7 years ago

Someone had this problem recently (#34)

This fix has been removed from this overlay, so the problem reappeared. The fix still works for me in my local overlay. I don't have installed argon2

njuneau commented 7 years ago

Hello Stefan,

With the new ebuild revision (20170803.2-r1), the daemon compiles just fine with our without system-pjproject. In all cases, Argon2 is found.

In my case, there's no longer an issue here. @YogSottot do you still have the problem?

YogSottot commented 7 years ago

In all cases, Argon2 is found

It was fixed again in this commit You can close this issue.