linux-can.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] autotools: define N_SLCAN, if not already declared
@ 2015-03-08 14:00 Yegor Yefremov
  2015-03-08 14:57 ` Oliver Hartkopp
  0 siblings, 1 reply; 2+ messages in thread
From: Yegor Yefremov @ 2015-03-08 14:00 UTC (permalink / raw)
  To: linux-can; +Cc: socketcan, Yegor Yefremov

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
---
 configure.ac | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/configure.ac b/configure.ac
index 3b695d3..633ba81 100644
--- a/configure.ac
+++ b/configure.ac
@@ -83,6 +83,9 @@ AC_CHECK_DECL(PF_CAN,,
 AC_CHECK_DECL(AF_CAN,,
     [AC_DEFINE([AF_CAN], [PF_CAN], [AF_CAN])]
 )
+AC_CHECK_DECL(N_SLCAN,,
+    [AC_DEFINE([N_SLCAN], [17], [N_SLCAN])]
+)
 
 
 #
-- 
2.1.0


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

* Re: [PATCH] autotools: define N_SLCAN, if not already declared
  2015-03-08 14:00 [PATCH] autotools: define N_SLCAN, if not already declared Yegor Yefremov
@ 2015-03-08 14:57 ` Oliver Hartkopp
  0 siblings, 0 replies; 2+ messages in thread
From: Oliver Hartkopp @ 2015-03-08 14:57 UTC (permalink / raw)
  To: Yegor Yefremov, linux-can

Good idea!

Actually a colleague recently had the problem that N_SLCAN was not defined on 
some older Redhat 6.x machine with 2.6.32 Kernel and 'even older' includes.

The patch from Jeremiah which was removing this private define

https://github.com/linux-can/can-utils/commit/ca7990919717b8e2c841552139ad221e58e8bb69

created the compile issue. So your patch fixes this problem in the correct way :-)

Acked-by: Oliver Hartkopp <socketcan@hartkopp.net>

Thanks,
Oliver

On 08.03.2015 15:00, Yegor Yefremov wrote:
> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
> ---
>   configure.ac | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index 3b695d3..633ba81 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -83,6 +83,9 @@ AC_CHECK_DECL(PF_CAN,,
>   AC_CHECK_DECL(AF_CAN,,
>       [AC_DEFINE([AF_CAN], [PF_CAN], [AF_CAN])]
>   )
> +AC_CHECK_DECL(N_SLCAN,,
> +    [AC_DEFINE([N_SLCAN], [17], [N_SLCAN])]
> +)
>
>
>   #
>

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

end of thread, other threads:[~2015-03-08 14:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-08 14:00 [PATCH] autotools: define N_SLCAN, if not already declared Yegor Yefremov
2015-03-08 14:57 ` Oliver Hartkopp

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).