Bug 570776 – =net-im/pidgin-2.10.12[gstreamer] does not find net-libs/farstream-0.1.2-r2

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Sebastian Pipping 2016-01-03 19:09:57 UTC]

Created attachment 421802 [details]
pidgin 2.10.12 build.log

[..]
checking for FARSTREAM... no
configure: error: 
[..]

[Comment 1 Jan Psota 2016-01-03 19:21:56 UTC]

pidgin-2.10.12 should depend on net-libs/farstream:0.2
(>=0.2.7 but there is no lower in tree)
and media-libs/gstreamer:1.0 - both instead of :0.1

I removed media-plugins/gst-plugins-meta:0.10 and media-plugins/gst-plugins-gconf:0.10 (no :1.0 in tree) deps and it works for me :-)

>=dev-lang/perl-5.16:= can be replaced with dev-lang/perl too.

[Comment 2 Sebastian Pipping 2016-01-03 20:15:11 UTC]

The configure script reads like 0.1 for both should be supported and does auto-detection by default, which the ebuild does not yet disable.

So we may need

  --with-gstreamer=1.0

and dependencies on

  >=net-libs/farstream-0.2.7:0.2
  media-libs/gstreamer:1.0

I'm unsure if those are changes that I should do by myself.

[Comment 3 Lars Wendler (Polynomial-C) (RETIRED) 2016-01-03 21:17:40 UTC]

commit 3781323eb0cc2694486cde82e3146074b0bcaa40
Author: Lars Wendler <[e-mail]>
Date:   Sun Jan 3 22:16:56 2016

    net-im/pidgin: Fixed dependency on gstreamer to use 1.0 slot (bug #570776).

    Package-Manager: portage-2.2.26
    Signed-off-by: Lars Wendler <[e-mail]>

[Comment 4 Sebastian Pipping 2016-01-03 23:26:34 UTC]

Lars, thanks for the quick reaction!  Comparing

  # git show 3781323eb0cc2694486cde82e3146074b0bcaa40 | tail 
          tcl? ( dev-lang/tcl:0= )
  @@ -202,7 +203,7 @@ src_configure() {
                  $(use_enable debug) \
                  $(use_enable dbus) \
                  $(use_enable meanwhile) \
  -               $(use_enable gstreamer) \
  +               $(use_enable gstreamer gstreamer 1.0) \
                  $(use_enable gstreamer farstream) \
                  $(use_enable gstreamer vv) \
                  $(use_enable sasl cyrus-sasl ) \

with configure.ac, I am thinking that we may need

  $(use_enable gstreamer) \
  $(use_with gstreamer gstreamer 1.0) \

instead.  It is --with-gstreamer that is taking "1.0", not --enable-gstreamer.  Can you confirm?

[Comment 5 Lars Wendler (Polynomial-C) (RETIRED) 2016-01-04 10:28:20 UTC]

(In reply to Sebastian Pipping from comment #4)
> Lars, thanks for the quick reaction!  Comparing
> 
>   # git show 3781323eb0cc2694486cde82e3146074b0bcaa40 | tail 
>           tcl? ( dev-lang/tcl:0= )
>   @@ -202,7 +203,7 @@ src_configure() {
>                   $(use_enable debug) \
>                   $(use_enable dbus) \
>                   $(use_enable meanwhile) \
>   -               $(use_enable gstreamer) \
>   +               $(use_enable gstreamer gstreamer 1.0) \
>                   $(use_enable gstreamer farstream) \
>                   $(use_enable gstreamer vv) \
>                   $(use_enable sasl cyrus-sasl ) \
> 
> with configure.ac, I am thinking that we may need
> 
>   $(use_enable gstreamer) \
>   $(use_with gstreamer gstreamer 1.0) \
> 
> instead.  It is --with-gstreamer that is taking "1.0", not
> --enable-gstreamer.  Can you confirm?

commit 19714751749b7f07ecfcce527c3961532ee72437
Author: Lars Wendler <[e-mail]>
Date:   Mon Jan 4 11:23:43 2016

    net-im/pidgin: Fixed gstreamer support for real this time (bug #570776).

    Package-Manager: portage-2.2.26
    Signed-off-by: Lars Wendler <[e-mail]>


Oh man. Upstream is really funny in that part of configure.ac...
They use --enable-gstreamer and --with-gstreamer=$ver

I've hopefully changed that part now correctly. Please verify.

[Comment 6 Sebastian Pipping 2016-01-05 00:17:09 UTC]

Looks all good now.  Thank you!