Patchwork [2/6] Add hdmiservice package for the snowball board

login
register
mail settings
Submitter Gregory Hermant
Date Oct. 29, 2012, 8:41 a.m.
Message ID <1351500073-4351-3-git-send-email-gregory.hermant@calao-systems.com>
Download mbox | patch
Permalink /patch/194883/
State Superseded
Headers show

Comments

Gregory Hermant - Oct. 29, 2012, 8:41 a.m.
Signed-off-by: Gregory Hermant <gregory.hermant@calao-systems.com>
---
 package/Config.in                  |    1 +
 package/hdmiservice/Config.in      |   10 ++++++++++
 package/hdmiservice/hdmiservice.mk |   19 +++++++++++++++++++
 3 files changed, 30 insertions(+)
 create mode 100644 package/hdmiservice/Config.in
 create mode 100644 package/hdmiservice/hdmiservice.mk
Arnout Vandecappelle - Oct. 30, 2012, 10:17 p.m.
On 10/29/12 09:41, Gregory Hermant wrote:
[snip]
> diff --git a/package/hdmiservice/hdmiservice.mk b/package/hdmiservice/hdmiservice.mk
> new file mode 100644
> index 0000000..28fc584
> --- /dev/null
> +++ b/package/hdmiservice/hdmiservice.mk
> @@ -0,0 +1,19 @@
> +#############################################################
> +#
> +# hdmiservice
> +#
> +#############################################################
> +HDMISERVICE_VERSION = f75c99d1c52707240a78b4ba78e41d20d3aa3b08
> +HDMISERVICE_SITE = git://igloocommunity.org/git/bsp/hdmiservice.git
> +HDMISERVICE_LICENSE = PROPRIETARY

  The header files say it's MIT.  The copyright file is debian/copyright.

  If you're not sure of the license, please just don't specify it, don't
just put a random value!

> +
> +define HDMISERVICE_BUILD_CMDS
> +	$(MAKE) CC="$(TARGET_CC)" LD="$(TARGET_LD)" -C $(@D)

  Does it work with $(TARGET_CONFIGURE_OPTS) instead of CC= LD= ?
That's the standard pattern.

> +endef
> +
> +define HDMISERVICE_INSTALL_TARGET_CMDS
> +	$(INSTALL) -D -m 0755 $(@D)/hdmiservice.so $(TARGET_DIR)/usr/lib/hdmiservice.so
> +	$(INSTALL) -D -m 0755 $(@D)/hdmistart $(TARGET_DIR)/usr/bin/hdmistart

$(MAKE) $(TARGET_CONFIGURE_OPTS) -C $(@D) DESTDIR=$(TARGET_DIR) install

  We do want the header files as well, so some client can link with
hdmiservice.so.  And you want to install it in staging as well.

> +endef
> +
> +$(eval $(generic-package))

  Regards,
  Arnout
Arnout Vandecappelle - Oct. 30, 2012, 10:19 p.m.
On 10/29/12 09:41, Gregory Hermant wrote:
> @@ -529,6 +529,7 @@ menu "Miscellaneous"
>   source "package/collectd/Config.in"
>   source "package/empty/Config.in"
>   source "package/googlefontdirectory/Config.in"
> +source "package/hdmiservice/Config.in"

  Oh, and Hardware handling is probably a more appropriate menu.

  Regards,
  Arnout

Patch

diff --git a/package/Config.in b/package/Config.in
index 6ebf2ce..097a94d 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -529,6 +529,7 @@  menu "Miscellaneous"
 source "package/collectd/Config.in"
 source "package/empty/Config.in"
 source "package/googlefontdirectory/Config.in"
+source "package/hdmiservice/Config.in"
 source "package/mobile-broadband-provider-info/Config.in"
 source "package/shared-mime-info/Config.in"
 source "package/sound-theme-borealis/Config.in"
diff --git a/package/hdmiservice/Config.in b/package/hdmiservice/Config.in
new file mode 100644
index 0000000..7fb2cf4
--- /dev/null
+++ b/package/hdmiservice/Config.in
@@ -0,0 +1,10 @@ 
+config BR2_PACKAGE_HDMISERVICE
+	bool "hdmiservice"
+	depends on BR2_TOOLCHAIN_HAS_THREADS	
+	help
+	  HDMI userspace control daemon
+
+	  This package contains the HDMI userspace control daemon for the 
+	  snowball board
+	  
+	  http://www.igloocommunity.org
diff --git a/package/hdmiservice/hdmiservice.mk b/package/hdmiservice/hdmiservice.mk
new file mode 100644
index 0000000..28fc584
--- /dev/null
+++ b/package/hdmiservice/hdmiservice.mk
@@ -0,0 +1,19 @@ 
+#############################################################
+#
+# hdmiservice
+#
+#############################################################
+HDMISERVICE_VERSION = f75c99d1c52707240a78b4ba78e41d20d3aa3b08
+HDMISERVICE_SITE = git://igloocommunity.org/git/bsp/hdmiservice.git
+HDMISERVICE_LICENSE = PROPRIETARY
+
+define HDMISERVICE_BUILD_CMDS
+	$(MAKE) CC="$(TARGET_CC)" LD="$(TARGET_LD)" -C $(@D)
+endef
+
+define HDMISERVICE_INSTALL_TARGET_CMDS
+	$(INSTALL) -D -m 0755 $(@D)/hdmiservice.so $(TARGET_DIR)/usr/lib/hdmiservice.so
+	$(INSTALL) -D -m 0755 $(@D)/hdmistart $(TARGET_DIR)/usr/bin/hdmistart
+endef
+
+$(eval $(generic-package))