[Buildroot] [PATCH 2/7 v2] package/freerdp: do not forcibly depend on X.Org
Samuel Martin
s.martin49 at gmail.com
Wed Oct 8 19:54:48 UTC 2014
On Tue, Oct 7, 2014 at 12:35 AM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> It is possible to build the libfreerdp standalone, without X.Org.
> Having a libfreerdp will be usefull for the weston RDP compositor.
>
> Re-order the menuconfig, as freerdp is no longer an X-only application.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
Reviewed-by: Samuel Martin <s.martin49 at gmail.com>
> ---
> package/Config.in | 2 +-
> package/freerdp/Config.in | 17 ++++++++++++-----
> package/freerdp/freerdp.mk | 10 ++++++++--
> 3 files changed, 21 insertions(+), 8 deletions(-)
>
> diff --git a/package/Config.in b/package/Config.in
> index 2dbe6f8..c2bd239 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -200,6 +200,7 @@ endif
> source "package/fb-test-app/Config.in"
> source "package/fbterm/Config.in"
> source "package/fbv/Config.in"
> + source "package/freerdp/Config.in"
> source "package/imagemagick/Config.in"
> source "package/linux-fusion/Config.in"
> source "package/lite/Config.in"
> @@ -238,7 +239,6 @@ comment "X applications"
> source "package/dillo/Config.in"
> source "package/docker/Config.in"
> source "package/feh/Config.in"
> - source "package/freerdp/Config.in"
> source "package/gmpc/Config.in"
> source "package/gob2/Config.in"
> source "package/gqview/Config.in"
> diff --git a/package/freerdp/Config.in b/package/freerdp/Config.in
> index c5ff769..81a67ea 100644
> --- a/package/freerdp/Config.in
> +++ b/package/freerdp/Config.in
> @@ -1,14 +1,21 @@
> config BR2_PACKAGE_FREERDP
> bool "freerdp"
> select BR2_PACKAGE_OPENSSL
> - select BR2_PACKAGE_XLIB_LIBX11
> - select BR2_PACKAGE_XLIB_LIBXT
> - select BR2_PACKAGE_XLIB_LIBXEXT
> - select BR2_PACKAGE_XLIB_LIBXCURSOR
> select BR2_PACKAGE_ZLIB
> - depends on BR2_PACKAGE_XORG7
> help
> FreeRDP is a free implementation of the Remote Desktop
> Protocol (RDP), released under the Apache license
>
> http://www.freerdp.com/
> +
> +if BR2_PACKAGE_FREERDP
> +
> +config BR2_PACKAGE_FREERDP_NEEDS_XLIBS
> + def_bool y
> + depends on BR2_PACKAGE_XORG7
> + select BR2_PACKAGE_XLIB_LIBX11
> + select BR2_PACKAGE_XLIB_LIBXT
> + select BR2_PACKAGE_XLIB_LIBXEXT
> + select BR2_PACKAGE_XLIB_LIBXCURSOR
> +
> +endif
> diff --git a/package/freerdp/freerdp.mk b/package/freerdp/freerdp.mk
> index e272165..49347fa 100644
> --- a/package/freerdp/freerdp.mk
> +++ b/package/freerdp/freerdp.mk
> @@ -7,13 +7,19 @@
> # Changeset on the stable-1.1 branch
> FREERDP_VERSION = 440916eae2e07463912d5fe507677e67096eb083
> FREERDP_SITE = $(call github,FreeRDP,FreeRDP,$(FREERDP_VERSION))
> -FREERDP_DEPENDENCIES = openssl zlib \
> - xlib_libX11 xlib_libXt xlib_libXext xlib_libXcursor
> +FREERDP_DEPENDENCIES = openssl zlib
> FREERDP_LICENSE = Apache-2.0
> FREERDP_LICENSE_FILES = LICENSE
>
> FREERDP_CONF_OPTS = -DWITH_MANPAGES=OFF
>
> +ifeq ($(BR2_PACKAGE_FREERDP_NEEDS_XLIBS),y)
> +FREERDP_DEPENDENCIES += xlib_libX11 xlib_libXt xlib_libXext xlib_libXcursor
> +FREERDP_CONF_OPTS += -DWITH_X11=ON
> +else
> +FREERDP_CONF_OPTS += -DWITH_X11=OFF
> +endif
> +
> ifeq ($(BR2_PACKAGE_CUPS),y)
> FREERDP_CONF_OPTS += -DWITH_CUPS=ON
> FREERDP_DEPENDENCIES += cups
> --
> 1.9.1
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
Samuel
More information about the buildroot
mailing list