Skip to contentSkip to conversation

fb-test-app: new package

PATCH New ludovic.desroches at atmel.com 4 participants 6 messages Oct 5, 2012 – Oct 8, 2012

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 fb-test-app: new package

From: Ludovic Desroches <[email protected]>

---
 package/Config.in                  |  1 +
 package/fb-test-app/Config.in      |  4 ++++
 package/fb-test-app/fb-test-app.mk | 34 ++++++++++++++++++++++++++++++++++
 3 files changed, 39 insertions(+)
 create mode 100644 package/fb-test-app/Config.in
 create mode 100644 package/fb-test-app/fb-test-app.mk
package/Config.in +1 -0
index 50b7631..a1ca25d 100644
@@ -117,6 +117,7 @@ source "package/fbset/Config.in"
 endif
 source "package/fbterm/Config.in"
 source "package/fbv/Config.in"
+source "package/fb-test-app/Config.in"
 source "package/imagemagick/Config.in"
 source "package/linux-fusion/Config.in"
 source "package/lite/Config.in"
package/fb-test-app/Config.in new file +4 -0
new file mode 100644
index 0000000..6b682ce
@@ -0,0 +1,4 @@
+config BR2_PACKAGE_FB_TEST_APP
+	bool "fb-test-app"
+	help
+	  Test suite for Linux framebuffer.
package/fb-test-app/fb-test-app.mk new file +34 -0
new file mode 100644
index 0000000..ce31f05
@@ -0,0 +1,34 @@
+#############################################################
+#
+# fb-test-app
+#
+#############################################################
+
+FB_TEST_APP_VERSION = v1.0.0
+FB_TEST_APP_SITE = git://github.com/prpplague/fb-test-app.git
+FB_TEST_APP_LICENSE = GPLv2
+FB_TEST_APP_LICENSE_FILE = COPYING
+
+define FB_TEST_APP_BUILD_CMDS
+	$(MAKE) CC="$(TARGET_CC)" LD="$(TARGET_LD)" -C $(@D) all
+endef
+
+define FB_TEST_APP_INSTALL_TARGET_CMDS
+	$(INSTALL) -D -m 0755 $(@D)/perf $(TARGET_DIR)/usr/bin/perf
+	$(INSTALL) -D -m 0755 $(@D)/rect $(TARGET_DIR)/usr/bin/rect
+	$(INSTALL) -D -m 0755 $(@D)/fb-test $(TARGET_DIR)/usr/bin/fb-test
+	$(INSTALL) -D -m 0755 $(@D)/offset $(TARGET_DIR)/usr/bin/offset
+endef
+
+define FB_TEST_APP_UNINSTALL_TARGET_CMDS
+	rm -f $(TARGET_DIR)/usr/bin/perf
+	rm -f $(TARGET_DIR)/usr/bin/rect
+	rm -f $(TARGET_DIR)/usr/bin/fb-test
+	rm -f $(TARGET_DIR)/usr/bin/offset
+endef
+
+define FB_TEST_APP_CLEAN_CMDS
+	$(MAKE) -C $(@D) clean
+endef
+
+$(eval $(generic-package))
Signature · 1 line
1.7.11.3
#1 lore raw

PATCH 1/1 fb-test-app: new package

Quoted 1 line · ludovic.desroches at atmel.com

From: Ludovic Desroches<[email protected]>

Add a Signed-off-by line for yourself. This is a short way for you to assert that you are entitled to contribute the patch under buildroot's GPL license. See http://kerneltrap.org/files/Jeremy/DCO.txt for more details.

[snip]

package/fb-test-app/Config.in line 4

Quoted 10 lines
package/fb-test-app/Config.in new file +4 -0
new file mode 100644
index 0000000..6b682ce
@@ -0,0 +1,4 @@
+config BR2_PACKAGE_FB_TEST_APP
+	bool "fb-test-app"
+	help
+	  Test suite for Linux framebuffer.

Could you add an upstream URL to the help text?

Quoted 6 lines
package/fb-test-app/fb-test-app.mk new file +0 -0
new file mode 100644
index 0000000..ce31f05

package/fb-test-app/fb-test-app.mk line 8

Quoted 8 lines
package/fb-test-app/fb-test-app.mk new file +8 -0
@@ -0,0 +1,34 @@
+#############################################################
+#
+# fb-test-app
+#
+#############################################################
+
+FB_TEST_APP_VERSION = v1.0.0
+FB_TEST_APP_SITE = git://github.com/prpplague/fb-test-app.git

FB_TEST_APP_SITE = http://github.com/prpplague/fb-test-app/tarball

downloads a lot faster.

Otherwise, looks great.

Regards,
Arnout

[snip]

Signature · 6 lines
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
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:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
#2 lore raw

PATCH 1/1 fb-test-app: new package

Hi Arnout

Le 10/06/2012 01:14 AM, Arnout Vandecappelle a ?crit :

Quoted 7 lines · ludovic.desroches at atmel.com
Quoted 1 line

From: Ludovic Desroches<[email protected]>

Add a Signed-off-by line for yourself. This is a short way for you to assert that you are entitled to contribute the patch under buildroot's GPL license. See http://kerneltrap.org/files/Jeremy/DCO.txt for more details.

Sorry for the omission.

Quoted 16 lines

[snip]

package/fb-test-app/Config.in line 4

Quoted 11 lines
package/fb-test-app/Config.in new file +4 -0
new file mode 100644
index 0000000..6b682ce
@@ -0,0 +1,4 @@
+config BR2_PACKAGE_FB_TEST_APP
+    bool "fb-test-app"
+    help
+      Test suite for Linux framebuffer.

Could you add an upstream URL to the help text?

Yes I'll add the github url.

Quoted 18 lines
Quoted 7 lines
package/fb-test-app/fb-test-app.mk new file +0 -0
new file mode 100644
index 0000000..ce31f05

package/fb-test-app/fb-test-app.mk line 8

Quoted 8 lines
package/fb-test-app/fb-test-app.mk new file +8 -0
@@ -0,0 +1,34 @@
+#############################################################
+#
+# fb-test-app
+#
+#############################################################
+
+FB_TEST_APP_VERSION = v1.0.0
+FB_TEST_APP_SITE = git://github.com/prpplague/fb-test-app.git

FB_TEST_APP_SITE = http://github.com/prpplague/fb-test-app/tarball

Quoted 1 line

downloads a lot faster.

Ok I'll change it.

Regards

Ludovic

Quoted 10 lines

Otherwise, looks great.

Regards,
Arnout

[snip]

#3 lore raw

PATCH 1/1 fb-test-app: new package

In reply to ludovic.desroches at atmel.com

Hi Ludovic,

package/fb-test-app/fb-test-app.mk line 17

Quoted 2 lines · ludovic.desroches at atmel.com
package/fb-test-app/fb-test-app.mk new file +2 -0
+define FB_TEST_APP_INSTALL_TARGET_CMDS
+	$(INSTALL) -D -m 0755 $(@D)/perf $(TARGET_DIR)/usr/bin/perf

This might collide with the Linux kernel's perf utility.

package/fb-test-app/fb-test-app.mk line 21

Quoted 4 lines
package/fb-test-app/fb-test-app.mk new file +4 -0
+	$(INSTALL) -D -m 0755 $(@D)/rect $(TARGET_DIR)/usr/bin/rect
+	$(INSTALL) -D -m 0755 $(@D)/fb-test $(TARGET_DIR)/usr/bin/fb-test
+	$(INSTALL) -D -m 0755 $(@D)/offset $(TARGET_DIR)/usr/bin/offset
+endef

baruch

Signature · 3 lines
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
#4 lore raw

PATCH 1/1 fb-test-app: new package

Hi Baruch,

Le 10/06/2012 11:03 PM, Baruch Siach a ?crit :

Quoted 8 lines

Hi Ludovic,

package/fb-test-app/fb-test-app.mk line 17

Quoted 2 lines · ludovic.desroches at atmel.com
package/fb-test-app/fb-test-app.mk new file +2 -0
+define FB_TEST_APP_INSTALL_TARGET_CMDS
+	$(INSTALL) -D -m 0755 $(@D)/perf $(TARGET_DIR)/usr/bin/perf

This might collide with the Linux kernel's perf utility.

I can add fb-test prefix to to the binary names, isn't it?

Regards

Ludovic

Quoted 7 lines

package/fb-test-app/fb-test-app.mk line 21

Quoted 4 lines
package/fb-test-app/fb-test-app.mk new file +4 -0
+	$(INSTALL) -D -m 0755 $(@D)/rect $(TARGET_DIR)/usr/bin/rect
+	$(INSTALL) -D -m 0755 $(@D)/fb-test $(TARGET_DIR)/usr/bin/fb-test
+	$(INSTALL) -D -m 0755 $(@D)/offset $(TARGET_DIR)/usr/bin/offset
+endef

baruch

#5 lore raw

PATCH 1/1 fb-test-app: new package

Quoted 7 lines · ludovic.desroches

Hi Baruch,

Le 10/06/2012 11:03 PM, Baruch Siach a ?crit :

Quoted 4 lines

Hi Ludovic,

On Fri, Oct 05, 2012 at 03:31:39PM +0200,
ludovic.desroches at atmel.com wrote:

Quoted 8 lines
Quoted 6 lines
Quoted 3 lines
(quoted diff) +2 -0
+define FB_TEST_APP_INSTALL_TARGET_CMDS
+	$(INSTALL) -D -m 0755 $(@D)/perf

$(TARGET_DIR)/usr/bin/perf

This might collide with the Linux kernel's perf utility.

I can add fb-test prefix to to the binary names, isn't it?

Sounds good, yes. So you would have:

fb-test-perf
fb-test
fb-test-rect
fb-test-offset

Best regards,

Thomas

Signature · 4 lines
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com