[Xfce-bugs] [Bug 7420] New: compile time dependency on `iceauth` should be runtime dependency/configurable

[bugzilla-daemon at xfce.org]

http://bugzilla.xfce.org/show_bug.cgi?id=7420

           Summary: compile time dependency on `iceauth` should be runtime
                    dependency/configurable
    Classification: Xfce
           Product: Xfce4-session
           Version: 4.8.0
          Platform: Other
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: Medium
         Component: General
        AssignedTo: xfce-bugs at xfce.org
        ReportedBy: paulepanter at users.sourceforge.net
         QAContact: bjt23 at cornell.edu
                CC: benny at xfce.org


Since its creation `configure.in.in` contains a check for `iceauth`.

    dnl Check for iceauth
    AC_PATH_PROG([ICEAUTH], [iceauth])
    if test x"$ICEAUTH" != x""; then
        AC_DEFINE_UNQUOTED([ICEAUTH_CMD], ["$ICEAUTH"], [path to iceauth])
    else
        AC_MSG_ERROR([iceauth missing, please check your X11 installation])
    fi

This check does not work if you are for example cross compiling [1]. For
example you cannot execute a binary compiled for the ARM architecture on the a
x86 build host. Additionally the assumption that `iceauth` should be present
when compiling the program is not valid.

I do not know if there is a better configure check [2] or if this should be
converted to an option, where the default patch can be changed.

A default path should be set up

    dnl Check for iceauth
    AC_PATH_PROG([ICEAUTH], [iceauth], [/usr/bin/iceauth])

and `xfce4-session` should fail gracefully when it is run and `iceauth` could
not be found. Some check in `xfce4-session/ice-layer.c`.

[1]
http://lists.linuxtogo.org/pipermail/openembedded-devel/2011-March/030721.html
[2]
http://www.gnu.org/software/autoconf/manual/autoconf-2.60/html_node/Generic-Programs.html

-- 
Configure bugmail: http://bugzilla.xfce.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.