On Tue, Apr 10, 2018 at 11:06:46AM +0200, Michael Tretter wrote:
> Split the version into major.minor.pl and use variable instead of hard
> coded version for installing libweston.
> 
> Signed-off-by: Michael Tretter <m.tret...@pengutronix.de>
> ---
>  rules/weston.in   |  1 -
>  rules/weston.make | 26 ++++++++++++++------------
>  2 files changed, 14 insertions(+), 13 deletions(-)
> 
> diff --git a/rules/weston.in b/rules/weston.in
> index 7db255202..a1755f510 100644
> --- a/rules/weston.in
> +++ b/rules/weston.in
> @@ -13,7 +13,6 @@ menuconfig WESTON
>       select CAIRO
>       select CAIRO_PNG
>       select LIBDRM                   if WESTON_DRM_COMPOSITOR
> -     select MTDEV                    if WESTON_DRM_COMPOSITOR || 
> WESTON_FBDEV_COMPOSITOR
>       select UDEV                     if WESTON_DRM_COMPOSITOR || 
> WESTON_FBDEV_COMPOSITOR
>       select UDEV_LIBUDEV             if WESTON_DRM_COMPOSITOR || 
> WESTON_FBDEV_COMPOSITOR
>       select MESALIB                  if WESTON_DRM_COMPOSITOR
> diff --git a/rules/weston.make b/rules/weston.make
> index 6e7065982..06175a304 100644
> --- a/rules/weston.make
> +++ b/rules/weston.make
> @@ -17,8 +17,11 @@ PACKAGES-$(PTXCONF_WESTON) += weston
>  #
>  # Paths and names
>  #
> -WESTON_VERSION       := 3.0.0
> -WESTON_MD5   := 9c42a4c51a1b9f35d040fa9d45ada36d
> +WESTON_MAJOR := 4
> +WESTON_MINOR := 0
> +WESTON_PL    := 0
> +WESTON_VERSION       := $(WESTON_MAJOR).$(WESTON_MINOR).$(WESTON_PL)

I prefer the version in one line. And use this for the major version:

WESTON_MAJOR    := $(basename $(basename $(WESTON_VERSION)))

> +WESTON_MD5   := 33709aa4d5916f89643fca0fc0064b39
>  WESTON               := weston-$(WESTON_VERSION)
>  WESTON_SUFFIX        := tar.xz
>  WESTON_URL   := 
> http://wayland.freedesktop.org/releases/$(WESTON).$(WESTON_SUFFIX)
> @@ -66,7 +69,6 @@ WESTON_CONF_OPT             := \
>       --disable-junit-xml \
>       --disable-ivi-shell \
>       --$(call ptx/endis, PTXCONF_WESTON_WCAP_TOOLS)-wcap-tools \
> -     --disable-libunwind \
>       --disable-demo-clients-install \
>       --disable-lcms \
>       --$(call ptx/endis, PTXCONF_WESTON_SYSTEMD)-systemd-notify \
> @@ -96,23 +98,23 @@ ifdef PTXCONF_WESTON_WCAP_TOOLS
>       @$(call install_copy, weston, 0, 0, 0755, -, /usr/bin/wcap-decode)
>  endif
>  
> -     @$(call install_lib, weston, 0, 0, 0644, libweston-3)
> -     @$(call install_lib, weston, 0, 0, 0644, libweston-desktop-3)
> +     @$(call install_lib, weston, 0, 0, 0644, libweston-$(WESTON_MAJOR))
> +     @$(call install_lib, weston, 0, 0, 0644, 
> libweston-desktop-$(WESTON_MAJOR))
>  ifdef PTXCONF_WESTON_XWAYLAND
> -     @$(call install_lib, weston, 0, 0, 0644, libweston-3/xwayland)
> +     @$(call install_lib, weston, 0, 0, 0644, 
> libweston-$(WESTON_MAJOR)/xwayland)
>  endif
>  ifdef PTXCONF_WESTON_DRM_COMPOSITOR
> -     @$(call install_lib, weston, 0, 0, 0644, libweston-3/drm-backend)
> +     @$(call install_lib, weston, 0, 0, 0644, 
> libweston-$(WESTON_MAJOR)/drm-backend)
>  endif
>  ifdef PTXCONF_WESTON_HEADLESS_COMPOSITOR
> -     @$(call install_lib, weston, 0, 0, 0644, libweston-3/headless-backend)
> +     @$(call install_lib, weston, 0, 0, 0644, 
> libweston-$(WESTON_MAJOR)/headless-backend)
>  endif
>  ifdef PTXCONF_WESTON_FBDEV_COMPOSITOR
> -     @$(call install_lib, weston, 0, 0, 0644, libweston-3/fbdev-backend)
> +     @$(call install_lib, weston, 0, 0, 0644, 
> libweston-$(WESTON_MAJOR)/fbdev-backend)
>  endif
>  ifdef PTXCONF_WESTON_GL
> -     @$(call install_lib, weston, 0, 0, 0644, libweston-3/wayland-backend)
> -     @$(call install_lib, weston, 0, 0, 0644, libweston-3/gl-renderer)
> +     @$(call install_lib, weston, 0, 0, 0644, 
> libweston-$(WESTON_MAJOR)/wayland-backend)
> +     @$(call install_lib, weston, 0, 0, 0644, 
> libweston-$(WESTON_MAJOR)/gl-renderer)
>  endif
>       @$(call install_lib, weston, 0, 0, 0644, weston/desktop-shell)
>       @$(call install_lib, weston, 0, 0, 0644, weston/fullscreen-shell)
> @@ -135,7 +137,7 @@ endif
>               sign_maximize.png \
>               icon_window.png \
>               sign_close.png \
> -             sign_maximize.png, \
> +             sign_minimize.png, \

Please sort these while at it.

>               $(call install_copy, weston, 0, 0, 0644, -, 
> /usr/share/weston/$(image));)

use $(ptx/nl) instead of ';' here.

Michael

>  
>  
> -- 
> 2.16.1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

Reply via email to