search

Heather / gentoo-gnome

Unofficial Gnome Overlay (Also contains elementary stuff)
50 stars 40 forks source link

mail-client/evolution fails to build with USE=map #33

Closed uzytkownik closed 10 years ago

uzytkownik commented 10 years ago 
checking if source code highlighting support is enabled... no
checking for GWEATHER... yes
checking for CHAMPLAIN... yes
checking for GEO... no
configure: error: 

        geoclue and/or geocode-glib not found.

        If you want to disable the contact maps feature,
        please append --disable-contact-maps to configure.

!!! Please attach the following file when seeking support:
!!! /var/tmp/portage/mail-client/evolution-3.10.1/work/evolution-3.10.1/config.log
cnd commented 10 years ago

It looks strange because of : https://github.com/Heather/gentoo-gnome/blob/master/mail-client/evolution/evolution-3.10.1.ebuild#L61

        map? (
                >=app-misc/geoclue-0.12.0
                >=sci-geosciences/geocode-glib-0.99.0

can you check if they are installed? maybe versions should be bumped to geoclue-2.0.0 ...

uzytkownik commented 10 years ago

On Sun, 2013-10-27 at 11:10 -0700, Heather wrote:

It looks strange because of : https://github.com/Heather/gentoo-gnome/blob/master/mail-client/evolution/evolution-3.10.1.ebuild#L61

    map? (
            >=app-misc/geoclue-0.12.0
            >=sci-geosciences/geocode-glib-0.99.0

can you check if they are installed?

— Reply to this email directly or view it on GitHub.

Yes. It looks like geocode-glib bumped pc (does it qualify for slotting?) to -1.0. Sorry I don't have more time to look into this but they are installed.

canek-pelaez commented 10 years ago

The code in configure.ac looks weird:

    PKG_CHECK_MODULES(
            [GEO],
            [geoclue >= geoclue_minimum_version
             geocode-glib = 0.99.0],,
            [AC_MSG_ERROR([

    geoclue and/or geocode-glib not found.

    If you want to disable the contact maps feature,
    please append --disable-contact-maps to configure.

    ])])

Looks like they are asking for geocode-glib exactly equals to 0.99.0. I'm trying to change the = to >= and see if it fixes the issue.

cnd commented 10 years ago

@canek-pelaez good find, thanks.

canek-pelaez commented 10 years ago

Anyway, that didn't fixed the issue. Changing geocode-glib = 0.99.0 to geocode-glib-1.0 >= 0.99.0 did it, thought. I'm emerging evolution right now; if it finishes without issues, I will commit the fix.

canek-pelaez commented 10 years ago

No, it didn't; build fails with:

libtool: compile: x86_64-pc-linux-gnu-gcc -DHAVE_CONFIG_H -I. -I../../.. -fno-strict-aliasing -DGLIB_VERSION_MAX_ALLOWED=GLIB_VERSION_2_34 -DGDK_VERSION_MAX_ALLOWED=GDK_VERSION_3_4 -DSOUP_VERSION_MAX_ALLOWED=SOUP_VERSION_2_40 -DGLIB_VERSION_MIN_REQUIRED=GLIB_VERSION_2_34 -DGDK_VERSION_MIN_REQUIRED=GDK_VERSION_3_2 -DSOUP_VERSION_MIN_REQUIRED=SOUP_VERSION_2_40 -DG_LOG_DOMAIN=\"eab-widgets\" -DEVOLUTION_ETSPECDIR=\"/usr/share/evolution/3.10/etspec\" -DEVOLUTION_GALVIEWSDIR=\"/usr/share/evolution/3.10/views\" -DEVOLUTION_RULEDIR=\"/usr/share/evolution/3.10\" -DEVOLUTION_IMAGESDIR=\"/usr/share/evolution/3.10/images\" -DEVOLUTION_PRIVDATADIR=\"/usr/share/evolution/3.10\" -I../../.. -I../../../addressbook -I../../../addressbook/gui/merging -I../../../addressbook/util -I../../../shell -pthread -I/usr/include/evolution-data-server -I/usr/include/nss -I/usr/include/nspr -I/usr/include/libsecret-1 -I/usr/include/libsoup-2.4 -I/usr/include/libxml2 -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -pthread -I/usr/include/gail-3.0 -I/usr/include/gnome-desktop-3.0 -I/usr/include/webkitgtk-3.0 -I/usr/include/gtk-3.0 -I/usr/include/at-spi2-atk/2.0 -I/usr/include/gtk-3.0 -I/usr/include/gio-unix-2.0/ -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/pango-1.0 -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libdrm -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng16 -I/usr/include/libsoup-2.4 -I/usr/include/libxml2 -I/usr/include/webkitgtk-3.0 -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/gsettings-desktop-schemas -pthread -I/usr/include/libgtkhtml-4.0/editor -I/usr/include/libgtkhtml-4.0 -I/usr/include/gtk-3.0 -I/usr/include/at-spi2-atk/2.0 -I/usr/include/gtk-3.0 -I/usr/include/gio-unix-2.0/ -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/pango-1.0 -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/enchant -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libdrm -I/usr/include/libpng16 -I/usr/include/gsettings-desktop-schemas -pthread -I/usr/include/libchamplain-gtk-0.12 -I/usr/include/clutter-gtk-1.0 -I/usr/include/libchamplain-0.12 -I/usr/include/libsoup-gnome-2.4 -I/usr/include/libsoup-2.4 -I/usr/include/libxml2 -I/usr/include/clutter-1.0 -I/usr/include/cogl -I/usr/include/json-glib-1.0 -I/usr/include/gtk-3.0 -I/usr/include/at-spi2-atk/2.0 -I/usr/include/gtk-3.0 -I/usr/include/gio-unix-2.0/ -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/pango-1.0 -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libdrm -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng16 -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -pthread -I/usr/include/dbus-1.0 -I/usr/lib64/dbus-1.0/include -I/usr/include/libxml2 -I/usr/include/geocode-glib-1.0 -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -march=core2 -O2 -pipe -fno-strict-aliasing -c e-minicard.c -fPIC -DPIC -o .libs/libeabwidgets_la-e-minicard.o e-contact-map.c:32:26: fatal error: geocode-glib.h: No such file or directory

include 

                      ^

compilation terminated.

I think geocode-glib-3* requires its own SLOT, so we can install it along with version 0.99.*, unless I'm missing something. I will take a look in the code of evolution.

canek-pelaez commented 10 years ago

I tried to solve it by changing #include to #include <geocode-glib/geocode-glib.h> in addressbook/gui/widgets/e-contact-map.c; however the build still fails. I think evo really needs geocode-glib 0.99.*.

In my system, only gnome-base/gnome-settings-daemon, gnome-extra/gnome-clocks and mail-client/evolution depends on sci-geosciences/geocode-glib. I will locally create a new slot for sci-geosciences/geocode-glib-3.10.0, and I will try to re emerge evo.

I will post the results.

canek-pelaez commented 10 years ago

OK, I wasn't able to parallel install geocode-glib-3.10.0 with geocode-glib-0.99.0 (nor 0.99.1, nor 0.99.4); they both use the same soname.

I think we should just disable the contact-maps option until it gets ported to geocode-glib-3.

cnd commented 10 years ago

@canek-pelaez agreed.

canek-pelaez commented 10 years ago

OK, I think I got it.

Using https://bugzilla.gnome.org/show_bug.cgi?id=689055 as basis, I wrote a patch porting Evolution to >=geocode-glib-0.99.1. I compiled and tested the patch; I'm able to see little cute maps for my contacts, although a label "Missing Plug-in" appears, and the map apparently doesn't save the location of my contacts.

I posted the patch in the bug (https://bugzilla.gnome.org/show_bug.cgi?id=689055#c5). I could commit the necessary changes to the overlay, but I think is better if we wait for upstream to review it and test it.

cnd commented 10 years ago

@canek-pelaez ok, good work.

canek-pelaez commented 10 years ago

OK, so my patch (reworked and improved) went in https://git.gnome.org/browse/evolution/commit/?id=8eb21abc046be972a82afedb3a0ab844b00ec7e3. I backported the patch; I think is a nice idea to have it as a "technological preview". Went in 8dac54bf0a0db6dd4962dae5539ffefaae28b718.

cnd commented 10 years ago

@canek-pelaez really great, thanks!