[Buildroot] [PATCH v2 1/1] package/kodi: Extended comment about non-RPi options

[Buildroot] [PATCH v2 1/1] package/kodi: Extended comment about non-RPi options

[Bernd Kuhls]

Suggested by Thomas:
http://lists.busybox.net/pipermail/buildroot/2017-April/190767.html

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: fixed logic typo in subject

 package/kodi/kodi.mk | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/package/kodi/kodi.mk b/package/kodi/kodi.mk
index 12dc3117f8..80c7c1b074 100644
--- a/package/kodi/kodi.mk
+++ b/package/kodi/kodi.mk
@@ -147,7 +147,10 @@ ifeq ($(BR2_PACKAGE_RPI_USERLAND),y)
 KODI_CONF_OPTS += -DCORE_SYSTEM_NAME=rbpi
 KODI_DEPENDENCIES += rpi-userland
 else
-# these options only exist on non-rpi systems
+# The following options are defined only in
+# project/cmake/scripts/linux/ArchSetup.cmake,
+# rbpi uses its own ArchSetup.cmake, located in
+# project/cmake/scripts/rbpi/
 KODI_CONF_OPTS += -DENABLE_LDGOLD=OFF
 ifeq ($(BR2_PACKAGE_LIBAMCODEC),y)
 KODI_CONF_OPTS += -DENABLE_AML=ON
-- 
2.11.0

[Buildroot] [PATCH v2 1/1] package/kodi: Extended comment about non-RPi options

[Thomas Petazzoni]

Hello,

On Sat, 29 Apr 2017 15:05:08 +0200, Bernd Kuhls wrote:
> Suggested by Thomas:
> http://lists.busybox.net/pipermail/buildroot/2017-April/190767.html
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> v2: fixed logic typo in subject

I've applied to master, but after significantly rewording. I found that
your explanation in the e-mail was much clearer than the one you
proposed in this patch, so I've reworded with something closer to what
you said in the e-mail.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com