! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
xfdesktop 4.11.3 (and git) with ./configure --disable-file-icons --disable-th...
Status:
RESOLVED: FIXED
Product:
Xfdesktop
Component:
General

Comments

Description Samuli Suominen 2014-03-12 19:44:48 CET
Run of `./configure --disable-file-icons --disable-thunarx` causes configure.ac.in because of

<snip>

if test "x$enable_desktop_icons" = "xyes" -a "x$enable_file_icons" = "xyes"; then
    dnl here i shall abuse the XDT_CHECK_PACKAGE macro
    XDT_CHECK_PACKAGE([DBUS], [dbus-glib-1], [dbus_minimum_version], [], [

</snip>

the compile to fail because at least common/xfdesktop-thumbnailer.c has #include <dbus/dbus-glib.h> without anykind of #ifdef later like:

<snip>
libtool: compile:  x86_64-pc-linux-gnu-gcc -DHAVE_CONFIG_H -I. -I.. -DNDEBUG -I../src -I/usr/include/xfce4 -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -pthread -I/usr/include/gtk-2.0 -I/usr/lib64/gtk-2.0/include -I/usr/include/pango-1.0 -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pixman-1 -I/usr/include/libdrm -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng16 -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/freetype2 -DDBUS_API_SUBJECT_TO_CHANGE -O2 -pipe -march=native -frecord-gcc-switches -Wimplicit-function-declaration -c xfdesktop-thumbnailer.c  -fPIC -DPIC -o .libs/libxfdesktop_la-xfdesktop-thumbnailer.o
xfdesktop-thumbnailer.c:39:28: fatal error: dbus/dbus-glib.h: No such file or directory
 #include <dbus/dbus-glib.h>
                            ^
compilation terminated.
Makefile:516: recipe for target 'libxfdesktop_la-xfdesktop-thumbnailer.lo' failed
</snip>

Looks like the "if test "x$enable_desktop_icons" = "xyes" -a "x$enable_file_icons" = "xyes";" type of code should be dropped from configure.ac so that dbus-glib is always checked if it's purposely made a new non-optional dependency
Comment 1 Samuli Suominen 2014-03-12 19:49:00 CET
Same problem in 'src/xfdesktop-file-utils.c' and 'src/xfdesktop-file-utils.h' because src/Makefile.am always compiles them, and they are without #ifdef in the .c and .h
Comment 2 Samuli Suominen 2014-03-12 19:57:15 CET
Created attachment 5383 
Always depend on dbus-glib

If the change was intention, then this should be applied too
Comment 3 Samuli Suominen 2014-03-12 19:57:36 CET
*intentional, sorry for bugspam
Comment 4 Eric Koegel editbugs 2014-03-14 18:53:04 CET
Good catch, thanks for the patch. Pushed to master in:
http://git.xfce.org/xfce/xfdesktop/commit/?id=f0e5154a5ff15f4d12d08cbc7f044641fa5b768e

Bug #10745

Reported by:
Samuli Suominen
Reported on: 2014-03-12
Last modified on: 2014-03-14

People

Assignee:
Eric Koegel
CC List:
1 user

Version

Version:
Unspecified

Attachments

Always depend on dbus-glib (1.17 KB, patch)
2014-03-12 19:57 CET , Samuli Suominen
no flags

Additional information