Patchwork [v3,1/2] Adding yavta (Yet Another V4L2 Test Application) as a package

login
register
mail settings
Submitter kpa_info@yahoo.fr
Date Jan. 10, 2013, 12:55 p.m.
Message ID <1357822531-8938-1-git-send-email-kpa_info@yahoo.fr>
Download mbox | patch
Permalink /patch/210998/
State Superseded
Headers show

Comments

kpa_info@yahoo.fr - Jan. 10, 2013, 12:55 p.m.
From: Patrick Gerber <kpa_info@yahoo.fr>


Signed-off-by: Patrick Gerber <kpa_info@yahoo.fr>
---
 package/multimedia/Config.in       |    1 +
 package/multimedia/yavta/Config.in |    6 ++++++
 package/multimedia/yavta/yavta.mk  |   21 +++++++++++++++++++++
 3 files changed, 26 insertions(+)
 create mode 100644 package/multimedia/yavta/Config.in
 create mode 100644 package/multimedia/yavta/yavta.mk
Thomas Petazzoni - Jan. 10, 2013, 1 p.m.
Dear kpa_info@yahoo.fr,

On Thu, 10 Jan 2013 13:55:30 +0100, kpa_info@yahoo.fr wrote:
> From: Patrick Gerber <kpa_info@yahoo.fr>
> 
> 
> Signed-off-by: Patrick Gerber <kpa_info@yahoo.fr>

For both:

Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Tested-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Note that I did only a build test, not a runtime test. But I did the
build test with a minimal toolchains:

 * One test with a uClibc toolchain with just thread support, but no
   rpc, no ipv6, no largefile, no C++, no nothing.

 * On test with a uClibc toolchain without thread support, but with
   rpc, ipv6, largefile, C++, etc.

This should help Peter bring those packages in.

The only nitpick I could do is on the commit title. We normally do:

	yavta: new package
	media-ctl: new package

But I'm really nitpicking here :-)

Thomas

Patch

diff --git a/package/multimedia/Config.in b/package/multimedia/Config.in
index 273d9bc..da558ca 100644
--- a/package/multimedia/Config.in
+++ b/package/multimedia/Config.in
@@ -24,4 +24,5 @@  source "package/multimedia/pulseaudio/Config.in"
 source "package/multimedia/tidsp-binaries/Config.in"
 source "package/multimedia/vorbis-tools/Config.in"
 source "package/multimedia/wavpack/Config.in"
+source "package/multimedia/yavta/Config.in"
 endmenu
diff --git a/package/multimedia/yavta/Config.in b/package/multimedia/yavta/Config.in
new file mode 100644
index 0000000..3cca5c1
--- /dev/null
+++ b/package/multimedia/yavta/Config.in
@@ -0,0 +1,6 @@ 
+config BR2_PACKAGE_YAVTA
+	bool "yavta"
+	help
+	  Yet Another V4L2 Test Application
+
+	  http://git.ideasonboard.org/yavta.git  
diff --git a/package/multimedia/yavta/yavta.mk b/package/multimedia/yavta/yavta.mk
new file mode 100644
index 0000000..1aa5574
--- /dev/null
+++ b/package/multimedia/yavta/yavta.mk
@@ -0,0 +1,20 @@ 
+#############################################################
+#
+# yavta
+#
+#############################################################
+
+YAVTA_VERSION = 82ff2efdb9787737b9f21b6f4759f077c827b238
+YAVTA_SITE = git://git.ideasonboard.org/yavta.git
+YAVTA_LICENSE = GPLv2
+YAVTA_LICENSE_FILES = COPYING.GPL
+
+define YAVTA_BUILD_CMDS
+	$(MAKE) $(TARGET_CONFIGURE_OPTS) -C $(@D) 
+endef
+
+define YAVTA_INSTALL_TARGET_CMDS
+	$(INSTALL) -m 0755 -D $(@D)/yavta $(TARGET_DIR)/usr/bin/yavta
+endef
+
+$(eval $(generic-package))