Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] [BUILDROOT] Update package socketcan and rename socketcan to socketcan_utils
@ 2011-04-20 12:15 kevin.joly at armadeus.com
  2011-04-21  8:28 ` Peter Korsgaard
  0 siblings, 1 reply; 2+ messages in thread
From: kevin.joly at armadeus.com @ 2011-04-20 12:15 UTC (permalink / raw)
  To: buildroot

From: Kevin JOLY <joly.kevin25@gmail.com>


Signed-off-by: Kevin JOLY <joly.kevin25@gmail.com>
---
 patches/buildroot/2010.11/042-socketcan.patch |   48 +++++++++++++++++++++++-
 1 files changed, 46 insertions(+), 2 deletions(-)

diff --git a/patches/buildroot/2010.11/042-socketcan.patch b/patches/buildroot/2010.11/042-socketcan.patch
index 8c77741..7bb885c 100644
--- a/patches/buildroot/2010.11/042-socketcan.patch
+++ b/patches/buildroot/2010.11/042-socketcan.patch
@@ -5,8 +5,8 @@ Signed-off-by: Kevin Joly <joly.kevin25@gmail.com>
 
 Index: buildroot/package/Config.in
 ===================================================================
---- buildroot.orig/package/Config.in	2011-04-20 13:56:09.454599001 +0200
-+++ buildroot/package/Config.in	2011-04-20 13:57:38.958599001 +0200
+--- buildroot.orig/package/Config.in	2011-04-20 14:11:30.206599000 +0200
++++ buildroot/package/Config.in	2011-04-20 14:12:11.566599000 +0200
 @@ -428,6 +428,7 @@
  source "package/samba/Config.in"
  source "package/ser2net/Config.in"
@@ -15,3 +15,47 @@ Index: buildroot/package/Config.in
  source "package/spawn-fcgi/Config.in"
  source "package/squid/Config.in"
  source "package/tcpdump/Config.in"
+Index: buildroot/package/socketcan_utils/Config.in
+===================================================================
+--- /dev/null	1970-01-01 00:00:00.000000000 +0000
++++ buildroot/package/socketcan_utils/Config.in	2011-04-20 09:45:25.000000000 +0200
+@@ -0,0 +1,7 @@
++config BR2_PACKAGE_SOCKETCAN_UTILS
++	bool "socketcan_utils"
++	help
++	  The socketcan_utils package contains several tools for Linux to test
++	  a Controller Area Network.
++
++	  http://developer.berlios.de/projects/socketcan/
+Index: buildroot/package/socketcan_utils/socketcan_utils.mk
+===================================================================
+--- /dev/null	1970-01-01 00:00:00.000000000 +0000
++++ buildroot/package/socketcan_utils/socketcan_utils.mk	2011-04-20 11:52:55.000000000 +0200
+@@ -0,0 +1,27 @@
++#############################################################
++#
++# socketcan_utils
++#
++#############################################################
++SOCKETCAN_UTILS_VERSION = 1235
++SOCKETCAN_UTILS_SOURCE = socketcan-svn-rev$(SOCKETCAN_VERSION).tar.gz
++SOCKETCAN_UTILS_SITE = http://svn.berlios.de/svnroot/repos/socketcan/trunk/
++SOCKETCAN_UTILS_SITE_METHOD = svn
++TARGET_CFLAGS+= -I$(@D)/kernel/2.6/include \
++	    -DSO_RXQ_OVFL=40 \
++	    -DPF_CAN=29 \
++	    -DAF_CAN=PF_CAN
++
++define SOCKETCAN_UTILS_BUILD_CMDS
++	$(MAKE) -C $(@D)/can-utils CC="$(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)"
++endef
++
++define SOCKETCAN_UTILS_INSTALL_TARGET_CMDS
++	$(MAKE) DESTDIR=$(TARGET_DIR) -C $(@D)/can-utils install PREFIX="/usr"
++endef
++
++define SOCKETCAN_UTILS_CLEAN_CMDS
++	$(MAKE) -C $(@D)/can-utils clean
++endef
++
++$(eval $(call GENTARGETS,package,socketcan_utils))
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH] [BUILDROOT] Update package socketcan and rename socketcan to socketcan_utils
  2011-04-20 12:15 [Buildroot] [PATCH] [BUILDROOT] Update package socketcan and rename socketcan to socketcan_utils kevin.joly at armadeus.com
@ 2011-04-21  8:28 ` Peter Korsgaard
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Korsgaard @ 2011-04-21  8:28 UTC (permalink / raw)
  To: buildroot

>>>>> "kevin" == kevin joly <kevin.joly@armadeus.com> writes:

 kevin> From: Kevin JOLY <joly.kevin25@gmail.com>
 kevin> Signed-off-by: Kevin JOLY <joly.kevin25@gmail.com>
 kevin> ---
 kevin>  patches/buildroot/2010.11/042-socketcan.patch |   48 +++++++++++++++++++++++-
 kevin>  1 files changed, 46 insertions(+), 2 deletions(-)

 kevin> diff --git a/patches/buildroot/2010.11/042-socketcan.patch b/patches/buildroot/2010.11/042-socketcan.patch
 kevin> index 8c77741..7bb885c 100644
 kevin> --- a/patches/buildroot/2010.11/042-socketcan.patch
 kevin> +++ b/patches/buildroot/2010.11/042-socketcan.patch
 kevin> @@ -5,8 +5,8 @@ Signed-off-by: Kevin Joly <joly.kevin25@gmail.com>
 
 kevin>  Index: buildroot/package/Config.in
 kevin>  ===================================================================
 kevin> ---- buildroot.orig/package/Config.in	2011-04-20 13:56:09.454599001 +0200
 kevin> -+++ buildroot/package/Config.in	2011-04-20 13:57:38.958599001 +0200
 kevin> +--- buildroot.orig/package/Config.in	2011-04-20 14:11:30.206599000 +0200
 kevin> ++++ buildroot/package/Config.in	2011-04-20 14:12:11.566599000 +0200
 kevin>  @@ -428,6 +428,7 @@
 kevin>   source "package/samba/Config.in"
 kevin>   source "package/ser2net/Config.in"
 kevin> @@ -15,3 +15,47 @@ Index: buildroot/package/Config.in
 kevin>   source "package/spawn-fcgi/Config.in"
 kevin>   source "package/squid/Config.in"
 kevin>   source "package/tcpdump/Config.in"
 kevin> +Index: buildroot/package/socketcan_utils/Config.in
 kevin> +===================================================================
 kevin> +--- /dev/null	1970-01-01 00:00:00.000000000 +0000
 kevin> ++++ buildroot/package/socketcan_utils/Config.in	2011-04-20 09:45:25.000000000 +0200
 kevin> +@@ -0,0 +1,7 @@
 kevin> ++config BR2_PACKAGE_SOCKETCAN_UTILS
 kevin> ++	bool "socketcan_utils"
 kevin> ++	help
 kevin> ++	  The socketcan_utils package contains several tools for Linux to test
 kevin> ++	  a Controller Area Network.
 kevin> ++
 kevin> ++	  http://developer.berlios.de/projects/socketcan/

The svn directory is called can-utils, so I prefer to use that instead.

 kevin> +Index: buildroot/package/socketcan_utils/socketcan_utils.mk
 kevin> +===================================================================
 kevin> +--- /dev/null	1970-01-01 00:00:00.000000000 +0000
 kevin> ++++ buildroot/package/socketcan_utils/socketcan_utils.mk	2011-04-20 11:52:55.000000000 +0200
 kevin> +@@ -0,0 +1,27 @@
 kevin> ++#############################################################
 kevin> ++#
 kevin> ++# socketcan_utils
 kevin> ++#
 kevin> ++#############################################################
 kevin> ++SOCKETCAN_UTILS_VERSION = 1235
 kevin> ++SOCKETCAN_UTILS_SOURCE = socketcan-svn-rev$(SOCKETCAN_VERSION).tar.gz
 kevin> ++SOCKETCAN_UTILS_SITE = http://svn.berlios.de/svnroot/repos/socketcan/trunk/
 kevin> ++SOCKETCAN_UTILS_SITE_METHOD = svn
 kevin> ++TARGET_CFLAGS+= -I$(@D)/kernel/2.6/include \
 kevin> ++	    -DSO_RXQ_OVFL=40 \
 kevin> ++	    -DPF_CAN=29 \
 kevin> ++	    -DAF_CAN=PF_CAN
 kevin> ++

You're changing TARGET_CFLAGS globally here, and can-utils' configure
already checks for PF_CAN / AF_CAN and defines those if not provided by
C library.

 kevin> ++define SOCKETCAN_UTILS_BUILD_CMDS
 kevin> ++	$(MAKE) -C $(@D)/can-utils CC="$(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)"
 kevin> ++endef
 kevin> ++
 kevin> ++define SOCKETCAN_UTILS_INSTALL_TARGET_CMDS
 kevin> ++	$(MAKE) DESTDIR=$(TARGET_DIR) -C $(@D)/can-utils install PREFIX="/usr"
 kevin> ++endef
 kevin> ++
 kevin> ++define SOCKETCAN_UTILS_CLEAN_CMDS
 kevin> ++	$(MAKE) -C $(@D)/can-utils clean
 kevin> ++endef

 kevin> ++
 kevin> ++$(eval $(call GENTARGETS,package,socketcan_utils))

can-utils uses autotools, so it makes more sense to use AUTOTARGETS
here.

I've made a can-utils package a few months ago as well, but kept back
from committing it as can-utils was supposed to get a release shortly - 
but that hasn't happened, so I'll commit it now.

Thanks for your contribution.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-04-21  8:28 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-20 12:15 [Buildroot] [PATCH] [BUILDROOT] Update package socketcan and rename socketcan to socketcan_utils kevin.joly at armadeus.com
2011-04-21  8:28 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox