After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 672613 - smclient detection not working when gdk-3.0.pc supports multiple backends
smclient detection not working when gdk-3.0.pc supports multiple backends
Status: RESOLVED FIXED
Product: evince
Classification: Core
Component: general
3.3.x
Other Linux
: Normal normal
: ---
Assigned To: Evince Maintainers
Evince Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-03-22 10:19 UTC by Juan RP
Modified: 2012-03-26 11:00 UTC
See Also:
GNOME target: 3.4
GNOME version: ---


Attachments
Fix with_smclient detection (465 bytes, patch)
2012-03-22 10:19 UTC, Juan RP
committed Details | Review

Description Juan RP 2012-03-22 10:19:29 UTC
Created attachment 210319 [details] [review]
Fix with_smclient detection

I just noticed that building evince-3.3.92 did not detect xsmp as smclient backend. After looking at the configure.ac I have found the problem due to $GDK_TARGET not checking for multiple backends:

configure.ac:

...
GDK_TARGET="$($PKG_CONFIG --variable targets gdk-3.0)"
...

AC_ARG_WITH([smclient],
  [AS_HELP_STRING([--with-smclient-backend=no|xsmp|win32|quartz],
                  [Setting smclient backend (default:auto)])],
  [],
  [case "$GDK_TARGET" in
    x11*) case "$with_platform" in
           gnome) with_smclient=xsmp ;;
           *) with_smclient=no ;;
         esac ;;
    win32|quartz) with_smclient=$GDK_TARGET ;;
    *) with_smclient=no ;;
   esac])

On my system, $GDK_TARGET reports "x11" and "broadway". Therefore the switch statement will always evaluate to false.

The attached patch fixes the issue. Please apply it before releasing evince-3.4.0.

Thanks.
Comment 1 Carlos Garcia Campos 2012-03-26 10:41:20 UTC
Review of attachment 210319 [details] [review]:

Pushed, thank you very much
Comment 2 André Klapper 2012-03-26 10:54:24 UTC
I'll consider this a build fix that does not fall under hardcode freeze. ;-)
Comment 3 Carlos Garcia Campos 2012-03-26 11:00:11 UTC
Yes, I think so, that's why I didn't ask for freeze breakage.