* [Buildroot] [PATCH v2 1/2] package/kodi-skin-confluence: new package
@ 2017-04-29 20:36 Bernd Kuhls
2017-04-29 20:36 ` [Buildroot] [PATCH v2 2/2] package/kodi: add Kconfig options to remove default skins Bernd Kuhls
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Bernd Kuhls @ 2017-04-29 20:36 UTC (permalink / raw)
To: buildroot
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: bumped to git HEAD
v1: Sending as v1 the unchanged patch from Kodi series v5.
package/kodi-skin-confluence/Config.in | 6 ++++++
.../kodi-skin-confluence/kodi-skin-confluence.hash | 2 ++
.../kodi-skin-confluence/kodi-skin-confluence.mk | 25 ++++++++++++++++++++++
package/kodi/Config.in | 4 ++++
4 files changed, 37 insertions(+)
create mode 100644 package/kodi-skin-confluence/Config.in
create mode 100644 package/kodi-skin-confluence/kodi-skin-confluence.hash
create mode 100644 package/kodi-skin-confluence/kodi-skin-confluence.mk
diff --git a/package/kodi-skin-confluence/Config.in b/package/kodi-skin-confluence/Config.in
new file mode 100644
index 000000000..08b656808
--- /dev/null
+++ b/package/kodi-skin-confluence/Config.in
@@ -0,0 +1,6 @@
+config BR2_PACKAGE_KODI_SKIN_CONFLUENCE
+ bool "kodi-skin-confluence"
+ help
+ Kodi confluence skin
+
+ https://github.com/xbmc/skin.confluence
diff --git a/package/kodi-skin-confluence/kodi-skin-confluence.hash b/package/kodi-skin-confluence/kodi-skin-confluence.hash
new file mode 100644
index 000000000..9aa2ea4a8
--- /dev/null
+++ b/package/kodi-skin-confluence/kodi-skin-confluence.hash
@@ -0,0 +1,2 @@
+# Locally computed
+sha256 47970d61b2f617921dfbc36188ed11de9af366cd97d9105e19cbe96598fe7b62 kodi-skin-confluence-7044dabb3a818d7ff47b7a34bfb3480f2a9441d9.tar.gz
diff --git a/package/kodi-skin-confluence/kodi-skin-confluence.mk b/package/kodi-skin-confluence/kodi-skin-confluence.mk
new file mode 100644
index 000000000..c679ec687
--- /dev/null
+++ b/package/kodi-skin-confluence/kodi-skin-confluence.mk
@@ -0,0 +1,25 @@
+################################################################################
+#
+# kodi-skin-confluence
+#
+################################################################################
+
+KODI_SKIN_CONFLUENCE_VERSION = 7044dabb3a818d7ff47b7a34bfb3480f2a9441d9
+KODI_SKIN_CONFLUENCE_SITE = $(call github,xbmc,skin.confluence,$(KODI_SKIN_CONFLUENCE_VERSION))
+KODI_SKIN_CONFLUENCE_LICENSE = GPLv2+
+KODI_SKIN_CONFLUENCE_LICENSE_FILES = LICENSE.txt
+KODI_SKIN_CONFLUENCE_DEPENDENCIES = kodi
+KODI_SKIN_CONFLUENCE_CONF_OPTS = -DOVERRIDE_PATHS=1
+
+define KODI_SKIN_CONFLUENCE_BUILD_CMDS
+ $(HOST_DIR)/usr/bin/TexturePacker -input $(@D)/media/ -output $(@D)/media/Textures.xbt -dupecheck -use_none
+endef
+
+define KODI_SKIN_CONFLUENCE_INSTALL_TARGET_CMDS
+ mkdir -p $(TARGET_DIR)/usr/share/kodi/addons/skin.confluence
+ cp -dpfr $(@D)/* $(TARGET_DIR)/usr/share/kodi/addons/skin.confluence
+ find $(TARGET_DIR)/usr/share/kodi/addons/skin.confluence/media -name *.jpg -delete
+ find $(TARGET_DIR)/usr/share/kodi/addons/skin.confluence/media -name *.png -delete
+endef
+
+$(eval $(generic-package))
diff --git a/package/kodi/Config.in b/package/kodi/Config.in
index 6bc81338a..cc747e816 100644
--- a/package/kodi/Config.in
+++ b/package/kodi/Config.in
@@ -326,6 +326,10 @@ menu "Screensavers"
source "package/kodi-screensaver-stars/Config.in"
endmenu
+menu "Skins"
+ source "package/kodi-skin-confluence/Config.in"
+endmenu
+
menu "Visualisations"
source "package/kodi-visualisation-fishbmc/Config.in"
source "package/kodi-visualisation-goom/Config.in"
--
2.11.0
^ permalink raw reply related [flat|nested] 7+ messages in thread* [Buildroot] [PATCH v2 2/2] package/kodi: add Kconfig options to remove default skins
2017-04-29 20:36 [Buildroot] [PATCH v2 1/2] package/kodi-skin-confluence: new package Bernd Kuhls
@ 2017-04-29 20:36 ` Bernd Kuhls
2017-05-01 8:09 ` Yann E. MORIN
2018-03-31 12:45 ` Arnout Vandecappelle
2017-05-01 7:46 ` [Buildroot] [PATCH v2 1/2] package/kodi-skin-confluence: new package Yann E. MORIN
2017-05-04 19:10 ` Thomas Petazzoni
2 siblings, 2 replies; 7+ messages in thread
From: Bernd Kuhls @ 2017-04-29 20:36 UTC (permalink / raw)
To: buildroot
The skins estuary and estouchy are new in Kodi 17-Krypton and replace
the skin confluence, used until Kodi 16-Jarvis.
This patch allows to disable one or both of the new skins, it also takes
care that at least one skin needs to be enabled. With this patch Kodi
can be used only with skin confluence.
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: no changes
v1: Sending as v1 the unchanged patch from Kodi series v5.
package/kodi-skin-confluence/Config.in | 1 +
package/kodi/Config.in | 35 ++++++++++++++++++++
package/kodi/kodi.mk | 58 ++++++++++++++++++++++++++++++++++
3 files changed, 94 insertions(+)
diff --git a/package/kodi-skin-confluence/Config.in b/package/kodi-skin-confluence/Config.in
index 08b656808..a623b03da 100644
--- a/package/kodi-skin-confluence/Config.in
+++ b/package/kodi-skin-confluence/Config.in
@@ -1,5 +1,6 @@
config BR2_PACKAGE_KODI_SKIN_CONFLUENCE
bool "kodi-skin-confluence"
+ select BR2_PACKAGE_KODI_SKIN_ENABLED
help
Kodi confluence skin
diff --git a/package/kodi/Config.in b/package/kodi/Config.in
index cc747e816..dfe92d2d1 100644
--- a/package/kodi/Config.in
+++ b/package/kodi/Config.in
@@ -55,6 +55,7 @@ menuconfig BR2_PACKAGE_KODI
select BR2_PACKAGE_FFMPEG_SWSCALE
select BR2_PACKAGE_FONTCONFIG
select BR2_PACKAGE_FREETYPE
+ select BR2_PACKAGE_KODI_SKIN_ESTUARY if !BR2_PACKAGE_KODI_SKIN_ENABLED
select BR2_PACKAGE_LIBASS
select BR2_PACKAGE_LIBCDIO
select BR2_PACKAGE_LIBCROSSGUID
@@ -327,6 +328,40 @@ menu "Screensavers"
endmenu
menu "Skins"
+
+config BR2_PACKAGE_KODI_SKIN_ENABLED
+ bool
+
+choice
+ prompt "Default skin"
+
+config BR2_PACKAGE_KODI_SKIN_DEFAULT_ESTUARY
+ bool "kodi-skin-estuary"
+ depends on BR2_PACKAGE_KODI_SKIN_ESTUARY
+
+config BR2_PACKAGE_KODI_SKIN_DEFAULT_ESTOUCHY
+ bool "kodi-skin-estouchy"
+ depends on BR2_PACKAGE_KODI_SKIN_ESTOUCHY
+
+config BR2_PACKAGE_KODI_SKIN_DEFAULT_CONFLUENCE
+ bool "kodi-skin-confluence"
+ depends on BR2_PACKAGE_KODI_SKIN_CONFLUENCE
+
+endchoice
+
+config BR2_PACKAGE_KODI_SKIN_ESTUARY
+ bool "kodi-skin-estuary"
+ default y
+ help
+ Default skin Estuary
+
+config BR2_PACKAGE_KODI_SKIN_ESTOUCHY
+ bool "kodi-skin-estouchy"
+ default y
+ select BR2_PACKAGE_KODI_SKIN_ENABLED
+ help
+ Default skin Estouchy
+
source "package/kodi-skin-confluence/Config.in"
endmenu
diff --git a/package/kodi/kodi.mk b/package/kodi/kodi.mk
index 0b89bdd86..73e44f861 100644
--- a/package/kodi/kodi.mk
+++ b/package/kodi/kodi.mk
@@ -355,6 +355,64 @@ define KODI_CLEAN_UNUSED_ADDONS
endef
KODI_POST_INSTALL_TARGET_HOOKS += KODI_CLEAN_UNUSED_ADDONS
+# Skins estuary and estouchy are installed by default and need to be
+# removed if they are disabled in buildroot
+ifeq ($(BR2_PACKAGE_KODI_SKIN_ESTUARY),y)
+define KODI_CLEAN_SKIN_ESTUARY
+ find $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary/media -name *.gif -delete
+ find $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary/media -name *.jpg -delete
+ find $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary/media -name *.png -delete
+endef
+KODI_POST_INSTALL_TARGET_HOOKS += KODI_CLEAN_SKIN_ESTUARY
+else
+define KODI_REMOVE_SKIN_ESTUARY
+ rm -Rf $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary
+ $(HOST_DIR)/usr/bin/xml ed -L \
+ -d "/addons/addon[text()='skin.estuary']" \
+ $(KODI_ADDON_MANIFEST)
+endef
+KODI_POST_INSTALL_TARGET_HOOKS += KODI_REMOVE_SKIN_ESTUARY
+endif
+
+ifeq ($(BR2_PACKAGE_KODI_SKIN_ESTOUCHY),y)
+define KODI_CLEAN_SKIN_ESTOUCHY
+ find $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy/media -name *.gif -delete
+ find $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy/media -name *.jpg -delete
+ find $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy/media -name *.png -delete
+endef
+KODI_POST_INSTALL_TARGET_HOOKS += KODI_CLEAN_SKIN_ESTOUCHY
+else
+define KODI_REMOVE_SKIN_ESTOUCHY
+ rm -Rf $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy
+ $(HOST_DIR)/usr/bin/xml ed -L \
+ -d "/addons/addon[text()='skin.estouchy']" \
+ $(KODI_ADDON_MANIFEST)
+endef
+KODI_POST_INSTALL_TARGET_HOOKS += KODI_REMOVE_SKIN_ESTOUCHY
+endif
+
+# The default value 'skin.estuary' is stored in
+# xbmc/system/settings/settings.xml.
+# If skin estuary is disabled this value needs to be changed to avoid
+# https://github.com/xbmc/xbmc/blob/32a6916059a0b14ab5fc65cedb17b2615c039918/xbmc/Application.cpp#L1124
+
+define KODI_SET_DEFAULT_SKIN_ESTOUCHY
+ $(SED) 's/skin.estuary/skin.estouchy/#g' $(TARGET_DIR)/usr/share/kodi/system/settings/settings.xml
+endef
+
+define KODI_SET_DEFAULT_SKIN_CONFLUENCE
+ $(SED) 's/skin.estuary/skin.confluence/#g' $(TARGET_DIR)/usr/share/kodi/system/settings/settings.xml
+ $(HOST_DIR)/usr/bin/xml ed -L -O --subnode "/addons" \
+ -t elem -n "addon" -v "skin.confluence" \
+ $(KODI_ADDON_MANIFEST)
+endef
+
+ifeq ($(BR2_PACKAGE_KODI_SKIN_DEFAULT_ESTOUCHY),y)
+KODI_POST_INSTALL_TARGET_HOOKS += KODI_SET_DEFAULT_SKIN_ESTOUCHY
+else ifeq ($(BR2_PACKAGE_KODI_SKIN_DEFAULT_CONFLUENCE),y)
+KODI_POST_INSTALL_TARGET_HOOKS += KODI_SET_DEFAULT_SKIN_CONFLUENCE
+endif
+
define KODI_INSTALL_BR_WRAPPER
$(INSTALL) -D -m 0755 package/kodi/br-kodi \
$(TARGET_DIR)/usr/bin/br-kodi
--
2.11.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 2/2] package/kodi: add Kconfig options to remove default skins
2017-04-29 20:36 ` [Buildroot] [PATCH v2 2/2] package/kodi: add Kconfig options to remove default skins Bernd Kuhls
@ 2017-05-01 8:09 ` Yann E. MORIN
2018-03-31 12:45 ` Arnout Vandecappelle
1 sibling, 0 replies; 7+ messages in thread
From: Yann E. MORIN @ 2017-05-01 8:09 UTC (permalink / raw)
To: buildroot
Bernd, All,
On 2017-04-29 22:36 +0200, Bernd Kuhls spake thusly:
> The skins estuary and estouchy are new in Kodi 17-Krypton and replace
> the skin confluence, used until Kodi 16-Jarvis.
>
> This patch allows to disable one or both of the new skins, it also takes
> care that at least one skin needs to be enabled. With this patch Kodi
> can be used only with skin confluence.
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
Tested-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Regards,
Yann E. MORIN.
> ---
> v2: no changes
> v1: Sending as v1 the unchanged patch from Kodi series v5.
>
> package/kodi-skin-confluence/Config.in | 1 +
> package/kodi/Config.in | 35 ++++++++++++++++++++
> package/kodi/kodi.mk | 58 ++++++++++++++++++++++++++++++++++
> 3 files changed, 94 insertions(+)
>
> diff --git a/package/kodi-skin-confluence/Config.in b/package/kodi-skin-confluence/Config.in
> index 08b656808..a623b03da 100644
> --- a/package/kodi-skin-confluence/Config.in
> +++ b/package/kodi-skin-confluence/Config.in
> @@ -1,5 +1,6 @@
> config BR2_PACKAGE_KODI_SKIN_CONFLUENCE
> bool "kodi-skin-confluence"
> + select BR2_PACKAGE_KODI_SKIN_ENABLED
> help
> Kodi confluence skin
>
> diff --git a/package/kodi/Config.in b/package/kodi/Config.in
> index cc747e816..dfe92d2d1 100644
> --- a/package/kodi/Config.in
> +++ b/package/kodi/Config.in
> @@ -55,6 +55,7 @@ menuconfig BR2_PACKAGE_KODI
> select BR2_PACKAGE_FFMPEG_SWSCALE
> select BR2_PACKAGE_FONTCONFIG
> select BR2_PACKAGE_FREETYPE
> + select BR2_PACKAGE_KODI_SKIN_ESTUARY if !BR2_PACKAGE_KODI_SKIN_ENABLED
> select BR2_PACKAGE_LIBASS
> select BR2_PACKAGE_LIBCDIO
> select BR2_PACKAGE_LIBCROSSGUID
> @@ -327,6 +328,40 @@ menu "Screensavers"
> endmenu
>
> menu "Skins"
> +
> +config BR2_PACKAGE_KODI_SKIN_ENABLED
> + bool
> +
> +choice
> + prompt "Default skin"
> +
> +config BR2_PACKAGE_KODI_SKIN_DEFAULT_ESTUARY
> + bool "kodi-skin-estuary"
> + depends on BR2_PACKAGE_KODI_SKIN_ESTUARY
> +
> +config BR2_PACKAGE_KODI_SKIN_DEFAULT_ESTOUCHY
> + bool "kodi-skin-estouchy"
> + depends on BR2_PACKAGE_KODI_SKIN_ESTOUCHY
> +
> +config BR2_PACKAGE_KODI_SKIN_DEFAULT_CONFLUENCE
> + bool "kodi-skin-confluence"
> + depends on BR2_PACKAGE_KODI_SKIN_CONFLUENCE
> +
> +endchoice
> +
> +config BR2_PACKAGE_KODI_SKIN_ESTUARY
> + bool "kodi-skin-estuary"
> + default y
> + help
> + Default skin Estuary
> +
> +config BR2_PACKAGE_KODI_SKIN_ESTOUCHY
> + bool "kodi-skin-estouchy"
> + default y
> + select BR2_PACKAGE_KODI_SKIN_ENABLED
> + help
> + Default skin Estouchy
> +
> source "package/kodi-skin-confluence/Config.in"
> endmenu
>
> diff --git a/package/kodi/kodi.mk b/package/kodi/kodi.mk
> index 0b89bdd86..73e44f861 100644
> --- a/package/kodi/kodi.mk
> +++ b/package/kodi/kodi.mk
> @@ -355,6 +355,64 @@ define KODI_CLEAN_UNUSED_ADDONS
> endef
> KODI_POST_INSTALL_TARGET_HOOKS += KODI_CLEAN_UNUSED_ADDONS
>
> +# Skins estuary and estouchy are installed by default and need to be
> +# removed if they are disabled in buildroot
> +ifeq ($(BR2_PACKAGE_KODI_SKIN_ESTUARY),y)
> +define KODI_CLEAN_SKIN_ESTUARY
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary/media -name *.gif -delete
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary/media -name *.jpg -delete
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary/media -name *.png -delete
> +endef
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_CLEAN_SKIN_ESTUARY
> +else
> +define KODI_REMOVE_SKIN_ESTUARY
> + rm -Rf $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary
> + $(HOST_DIR)/usr/bin/xml ed -L \
> + -d "/addons/addon[text()='skin.estuary']" \
> + $(KODI_ADDON_MANIFEST)
> +endef
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_REMOVE_SKIN_ESTUARY
> +endif
> +
> +ifeq ($(BR2_PACKAGE_KODI_SKIN_ESTOUCHY),y)
> +define KODI_CLEAN_SKIN_ESTOUCHY
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy/media -name *.gif -delete
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy/media -name *.jpg -delete
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy/media -name *.png -delete
> +endef
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_CLEAN_SKIN_ESTOUCHY
> +else
> +define KODI_REMOVE_SKIN_ESTOUCHY
> + rm -Rf $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy
> + $(HOST_DIR)/usr/bin/xml ed -L \
> + -d "/addons/addon[text()='skin.estouchy']" \
> + $(KODI_ADDON_MANIFEST)
> +endef
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_REMOVE_SKIN_ESTOUCHY
> +endif
> +
> +# The default value 'skin.estuary' is stored in
> +# xbmc/system/settings/settings.xml.
> +# If skin estuary is disabled this value needs to be changed to avoid
> +# https://github.com/xbmc/xbmc/blob/32a6916059a0b14ab5fc65cedb17b2615c039918/xbmc/Application.cpp#L1124
> +
> +define KODI_SET_DEFAULT_SKIN_ESTOUCHY
> + $(SED) 's/skin.estuary/skin.estouchy/#g' $(TARGET_DIR)/usr/share/kodi/system/settings/settings.xml
> +endef
> +
> +define KODI_SET_DEFAULT_SKIN_CONFLUENCE
> + $(SED) 's/skin.estuary/skin.confluence/#g' $(TARGET_DIR)/usr/share/kodi/system/settings/settings.xml
> + $(HOST_DIR)/usr/bin/xml ed -L -O --subnode "/addons" \
> + -t elem -n "addon" -v "skin.confluence" \
> + $(KODI_ADDON_MANIFEST)
> +endef
> +
> +ifeq ($(BR2_PACKAGE_KODI_SKIN_DEFAULT_ESTOUCHY),y)
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_SET_DEFAULT_SKIN_ESTOUCHY
> +else ifeq ($(BR2_PACKAGE_KODI_SKIN_DEFAULT_CONFLUENCE),y)
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_SET_DEFAULT_SKIN_CONFLUENCE
> +endif
> +
> define KODI_INSTALL_BR_WRAPPER
> $(INSTALL) -D -m 0755 package/kodi/br-kodi \
> $(TARGET_DIR)/usr/bin/br-kodi
> --
> 2.11.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 2/2] package/kodi: add Kconfig options to remove default skins
2017-04-29 20:36 ` [Buildroot] [PATCH v2 2/2] package/kodi: add Kconfig options to remove default skins Bernd Kuhls
2017-05-01 8:09 ` Yann E. MORIN
@ 2018-03-31 12:45 ` Arnout Vandecappelle
1 sibling, 0 replies; 7+ messages in thread
From: Arnout Vandecappelle @ 2018-03-31 12:45 UTC (permalink / raw)
To: buildroot
Hi Bernd,
Getting back to this very old patch...
On 29-04-17 22:36, Bernd Kuhls wrote:
> The skins estuary and estouchy are new in Kodi 17-Krypton and replace
> the skin confluence, used until Kodi 16-Jarvis.
>
> This patch allows to disable one or both of the new skins, it also takes
> care that at least one skin needs to be enabled. With this patch Kodi
> can be used only with skin confluence.
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
[snip]
> diff --git a/package/kodi/kodi.mk b/package/kodi/kodi.mk
> index 0b89bdd86..73e44f861 100644
> --- a/package/kodi/kodi.mk
> +++ b/package/kodi/kodi.mk
> @@ -355,6 +355,64 @@ define KODI_CLEAN_UNUSED_ADDONS
> endef
> KODI_POST_INSTALL_TARGET_HOOKS += KODI_CLEAN_UNUSED_ADDONS
>
> +# Skins estuary and estouchy are installed by default and need to be
> +# removed if they are disabled in buildroot
> +ifeq ($(BR2_PACKAGE_KODI_SKIN_ESTUARY),y)
> +define KODI_CLEAN_SKIN_ESTUARY
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary/media -name *.gif -delete
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary/media -name *.jpg -delete
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary/media -name *.png -delete
It looks a bit weird that you remove stuff when the skin is enabled, so I think
a comment explaining that is warranted.
> +endef
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_CLEAN_SKIN_ESTUARY
> +else
> +define KODI_REMOVE_SKIN_ESTUARY
> + rm -Rf $(TARGET_DIR)/usr/share/kodi/addons/skin.estuary
> + $(HOST_DIR)/usr/bin/xml ed -L \
We now no longer have the usr/ bit, could you fix that?
> + -d "/addons/addon[text()='skin.estuary']" \
> + $(KODI_ADDON_MANIFEST)
Given that there are now several addons to remove, it might be worthwhile to
factor it:
rm -Rf $(addprefix $(TARGET_DIR)/usr/share/kodi/addons/,\
$(KODI_ADDONS_TO_REMOVE))
$(HOST_DIR)/bin/xml ed -L \
$(patsubst %,-d "/addons/addon[text()='%']",$(KODI_ADDONS_TO_REMOVE)) \
$(KODI_ADDON_MANIFEST)
Probably for a separate patch though.
> +endef
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_REMOVE_SKIN_ESTUARY
> +endif
> +
> +ifeq ($(BR2_PACKAGE_KODI_SKIN_ESTOUCHY),y)
> +define KODI_CLEAN_SKIN_ESTOUCHY
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy/media -name *.gif -delete
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy/media -name *.jpg -delete
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy/media -name *.png -delete
> +endef
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_CLEAN_SKIN_ESTOUCHY
> +else
> +define KODI_REMOVE_SKIN_ESTOUCHY
> + rm -Rf $(TARGET_DIR)/usr/share/kodi/addons/skin.estouchy
> + $(HOST_DIR)/usr/bin/xml ed -L \
> + -d "/addons/addon[text()='skin.estouchy']" \
> + $(KODI_ADDON_MANIFEST)
> +endef
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_REMOVE_SKIN_ESTOUCHY
> +endif
> +
> +# The default value 'skin.estuary' is stored in
> +# xbmc/system/settings/settings.xml.
> +# If skin estuary is disabled this value needs to be changed to avoid
> +# https://github.com/xbmc/xbmc/blob/32a6916059a0b14ab5fc65cedb17b2615c039918/xbmc/Application.cpp#L1124
> +
> +define KODI_SET_DEFAULT_SKIN_ESTOUCHY
Conditional hooks are normally put within the ifeq, like you did above.
> + $(SED) 's/skin.estuary/skin.estouchy/#g' $(TARGET_DIR)/usr/share/kodi/system/settings/settings.xml
#g? Does that exist? I couldn't find an explanation of the # flag in the sed
documentation [1].
> +endef
> +
> +define KODI_SET_DEFAULT_SKIN_CONFLUENCE
> + $(SED) 's/skin.estuary/skin.confluence/#g' $(TARGET_DIR)/usr/share/kodi/system/settings/settings.xml
> + $(HOST_DIR)/usr/bin/xml ed -L -O --subnode "/addons" \
> + -t elem -n "addon" -v "skin.confluence" \
> + $(KODI_ADDON_MANIFEST)
Shouldn't this be added to the manifest regardless of whether it is the default
or not? In that case, it should be part of the kodi-skin-confluence package -
which unfortunately would break check-uniq-files...
Regards,
Arnout
[1] https://www.gnu.org/software/sed/manual/html_node/The-_0022s_0022-Command.html
> +endef
> +
> +ifeq ($(BR2_PACKAGE_KODI_SKIN_DEFAULT_ESTOUCHY),y)
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_SET_DEFAULT_SKIN_ESTOUCHY
> +else ifeq ($(BR2_PACKAGE_KODI_SKIN_DEFAULT_CONFLUENCE),y)
> +KODI_POST_INSTALL_TARGET_HOOKS += KODI_SET_DEFAULT_SKIN_CONFLUENCE
> +endif
> +
> define KODI_INSTALL_BR_WRAPPER
> $(INSTALL) -D -m 0755 package/kodi/br-kodi \
> $(TARGET_DIR)/usr/bin/br-kodi
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 1/2] package/kodi-skin-confluence: new package
2017-04-29 20:36 [Buildroot] [PATCH v2 1/2] package/kodi-skin-confluence: new package Bernd Kuhls
2017-04-29 20:36 ` [Buildroot] [PATCH v2 2/2] package/kodi: add Kconfig options to remove default skins Bernd Kuhls
@ 2017-05-01 7:46 ` Yann E. MORIN
2017-05-07 12:08 ` Bernd Kuhls
2017-05-04 19:10 ` Thomas Petazzoni
2 siblings, 1 reply; 7+ messages in thread
From: Yann E. MORIN @ 2017-05-01 7:46 UTC (permalink / raw)
To: buildroot
Bernd, All,
On 2017-04-29 22:36 +0200, Bernd Kuhls spake thusly:
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Tested-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Yet, a little comment, blow...
> ---
[--SNIP--]
> diff --git a/package/kodi-skin-confluence/kodi-skin-confluence.mk b/package/kodi-skin-confluence/kodi-skin-confluence.mk
> new file mode 100644
> index 000000000..c679ec687
> --- /dev/null
> +++ b/package/kodi-skin-confluence/kodi-skin-confluence.mk
> @@ -0,0 +1,25 @@
> +################################################################################
> +#
> +# kodi-skin-confluence
> +#
> +################################################################################
> +
Like for other Kodi packages, should we add "on the Krypton branch" ?
Regards,
Yann E. MORIN.
> +KODI_SKIN_CONFLUENCE_VERSION = 7044dabb3a818d7ff47b7a34bfb3480f2a9441d9
> +KODI_SKIN_CONFLUENCE_SITE = $(call github,xbmc,skin.confluence,$(KODI_SKIN_CONFLUENCE_VERSION))
> +KODI_SKIN_CONFLUENCE_LICENSE = GPLv2+
> +KODI_SKIN_CONFLUENCE_LICENSE_FILES = LICENSE.txt
> +KODI_SKIN_CONFLUENCE_DEPENDENCIES = kodi
> +KODI_SKIN_CONFLUENCE_CONF_OPTS = -DOVERRIDE_PATHS=1
> +
> +define KODI_SKIN_CONFLUENCE_BUILD_CMDS
> + $(HOST_DIR)/usr/bin/TexturePacker -input $(@D)/media/ -output $(@D)/media/Textures.xbt -dupecheck -use_none
> +endef
> +
> +define KODI_SKIN_CONFLUENCE_INSTALL_TARGET_CMDS
> + mkdir -p $(TARGET_DIR)/usr/share/kodi/addons/skin.confluence
> + cp -dpfr $(@D)/* $(TARGET_DIR)/usr/share/kodi/addons/skin.confluence
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.confluence/media -name *.jpg -delete
> + find $(TARGET_DIR)/usr/share/kodi/addons/skin.confluence/media -name *.png -delete
> +endef
> +
> +$(eval $(generic-package))
> diff --git a/package/kodi/Config.in b/package/kodi/Config.in
> index 6bc81338a..cc747e816 100644
> --- a/package/kodi/Config.in
> +++ b/package/kodi/Config.in
> @@ -326,6 +326,10 @@ menu "Screensavers"
> source "package/kodi-screensaver-stars/Config.in"
> endmenu
>
> +menu "Skins"
> + source "package/kodi-skin-confluence/Config.in"
> +endmenu
> +
> menu "Visualisations"
> source "package/kodi-visualisation-fishbmc/Config.in"
> source "package/kodi-visualisation-goom/Config.in"
> --
> 2.11.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 1/2] package/kodi-skin-confluence: new package
2017-04-29 20:36 [Buildroot] [PATCH v2 1/2] package/kodi-skin-confluence: new package Bernd Kuhls
2017-04-29 20:36 ` [Buildroot] [PATCH v2 2/2] package/kodi: add Kconfig options to remove default skins Bernd Kuhls
2017-05-01 7:46 ` [Buildroot] [PATCH v2 1/2] package/kodi-skin-confluence: new package Yann E. MORIN
@ 2017-05-04 19:10 ` Thomas Petazzoni
2 siblings, 0 replies; 7+ messages in thread
From: Thomas Petazzoni @ 2017-05-04 19:10 UTC (permalink / raw)
To: buildroot
Hello,
On Sat, 29 Apr 2017 22:36:07 +0200, Bernd Kuhls wrote:
> +KODI_SKIN_CONFLUENCE_VERSION = 7044dabb3a818d7ff47b7a34bfb3480f2a9441d9
> +KODI_SKIN_CONFLUENCE_SITE = $(call github,xbmc,skin.confluence,$(KODI_SKIN_CONFLUENCE_VERSION))
> +KODI_SKIN_CONFLUENCE_LICENSE = GPLv2+
You should use SPDX license codes, and GPLv2+ is not a valid SPDX
license code.
In addition, I didn't see anything in the source code that says "or
later". So I've changed this to GPL-2.0.
> +KODI_SKIN_CONFLUENCE_LICENSE_FILES = LICENSE.txt
> +KODI_SKIN_CONFLUENCE_DEPENDENCIES = kodi
> +KODI_SKIN_CONFLUENCE_CONF_OPTS = -DOVERRIDE_PATHS=1
This is not used anywhere in your package... it's a generic-package.
Applied with those issues fixed.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-03-31 12:45 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-04-29 20:36 [Buildroot] [PATCH v2 1/2] package/kodi-skin-confluence: new package Bernd Kuhls
2017-04-29 20:36 ` [Buildroot] [PATCH v2 2/2] package/kodi: add Kconfig options to remove default skins Bernd Kuhls
2017-05-01 8:09 ` Yann E. MORIN
2018-03-31 12:45 ` Arnout Vandecappelle
2017-05-01 7:46 ` [Buildroot] [PATCH v2 1/2] package/kodi-skin-confluence: new package Yann E. MORIN
2017-05-07 12:08 ` Bernd Kuhls
2017-05-04 19:10 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox