This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: SocketCan support
Type: enhancement Stage: resolved
Components: IO Versions: Python 3.3
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: Riccardo Magliocchetti, amaury.forgeotdarc, giampaolo.rodola, martin.panter, mluis, neologix, ogait87, orsenthil, pitrou, python-dev, r.david.murray, rosslagerwall, slanden, vinay.sajip, vstinner
Priority: normal Keywords: needs review, patch

Created on 2010-10-19 00:49 by slanden, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
socketcan.dpatch slanden, 2010-10-19 00:49 SocketCan patch for 2.6.5
socketcan.patch ogait87, 2011-07-20 01:33 SocketCan patch for 3.3 review
socketcan_v2.patch ogait87, 2011-08-01 20:57 SocketCan patch v2 for 3.3 review
socketcan_v5.patch neologix, 2011-09-23 21:33 review
af_can_undefined.diff neologix, 2011-10-07 17:12 review
Pull Requests
URL Status Linked Edit
PR 23794 open python-dev, 2020-12-16 08:54
Messages (42)
msg119096 - (view) Author: Shawn Landen (slanden) Date: 2010-10-19 00:49
Python lacks support for the AF_CAN family of sockets ( http://en.wikipedia.org/wiki/Controller_area_network http://lwn.net/Articles/253425/)

https://lists.berlios.de/pipermail/socketcan-users/2010-July/001456.html
msg119120 - (view) Author: Senthil Kumaran (orsenthil) * (Python committer) Date: 2010-10-19 04:47
Looks like an interesting feature request. What are the packages or external libraries which may rely on this and how do they deal with SocketCan support at moment for their requirements? (That thread mentions several, but some details may help further).

BTW, this can go in Python 3.2 if it gets in before Nov 2nd week. It wont be going into Python 2.x series.
msg119122 - (view) Author: Shawn Landen (slanden) Date: 2010-10-19 05:39
forgot to include this: http://en.wikipedia.org/wiki/Socketcan

SocketCan is the (currently) Linux-specific Berkley-socket CAN implementation created by Volkswagen. Alot of previous CAN implementations use a rather inferior (explained in can.txt in the Linux sources) character device approach.

So the use case for programs that use SocketCan is raw sockets in C, as it is a rather specialty form of interface. But considering that it fits fairly well into an existing interface, it seems appropriate to add to python.

I clicked 2.7 because that was what the patch was written for.

Our personal use case is for an IVI dash, to access vehicle devices that output familiar metrics like speed, tach, fuel levels, seat belts, etc over CAN.
msg119123 - (view) Author: Shawn Landen (slanden) Date: 2010-10-19 05:42
* PF_CAN family of sockets
msg121125 - (view) Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * (Python committer) Date: 2010-11-13 09:54
The patch looks good at first glance, but is there a way to test the feature?
msg139763 - (view) Author: Tiago Gonçalves (ogait87) Date: 2011-07-04 12:51
There is this "simple" program that can be easily adapted to test the interface. http://old.nabble.com/Socketcan-with-Python-td29286297.html#a29286297
msg139786 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2011-07-04 16:55
The patch looks good to me.
A couple remarks:
- could you please post it as a Mercurial diff? (it makes it easier to
review and apply)

@@ -1151,6 +1151,25 @@ makesockaddr(int sockfd, struct sockaddr
 	}

+               return Py_BuildValue("sh",
+                                    ifname,
+                                    a->can_family);

1) a->can_family would be better as a 'i' to be consistent with the
rest of the code

2) you should use the FS encoding for the interface name, e.g.

               return Py_BuildValue("O&i",
                                    PyUnicode_DecodeFSDefault
                                    ifname,
                                    a->can_family);

(you can have a look at socket_if_nameindex for an example).

@@ -1486,6 +1505,43 @@ getsockaddrarg(PySocketSockObject *s, Py

                       if (!PyArg_ParseTuple(args, "s", &interfaceName))
                               return 0;

3) same thing here, you should use
                       if (!PyArg_ParseTuple(args, "O&", PyUnicode_FSConverter,
                                                            &interfaceName))
                               return 0;

(see socket_if_nametoindex for an example)

4)
@@ -1486,6 +1505,43 @@ getsockaddrarg(PySocketSockObject *s, Py

+                       if (!strcmp(interfaceName, "any")) {
+                               ifr.ifr_ifindex = 0;

To be consistent with the convention used IPv4/IPv6
INADDR_ANY/in6addr_any, I would prefer if you replaced the any
interface name by "" (empty string)

5)
@@ -69,6 +69,20 @@ typedef int socklen_t;

+#ifdef HAVE_LINUX_CAN_H
+#include <linux/can.h>
+#ifndef PF_CAN
+# define PF_CAN 29
+#endif
+#ifndef AF_CAN
+# define AF_CAN PF_CAN
+#endif
+#endif

I don't see how PF_CAN or AF_CAN could not be defined lin <linux/can.h>.
And if they're not defined, it's probably not a good idea to define
them arbitrarily...

6)
+#ifdef HAVE_LINUX_CAN_H
+       struct sockaddr_can* can;
+#endif

it should be struct sockaddr_can can here, not a pointer.

7)
- you should update Doc/library/socket.rst

8)
- could you add a test for SocketCan in Lib/test/socket.py ?
It would be nice to have a basic test which can be run on all the
kernels supporting PF_CAN (typically just creating a socket, binding
to it and closing it), and another more complete test if there's a CAN
interface (sending and receiving a packet, probably using the
loopback).
Take into account that SOCK_RAW are priviledged and restricted to root
or CAP_SYS_RAW.
msg140713 - (view) Author: Tiago Gonçalves (ogait87) Date: 2011-07-20 01:33
This patch should be easy to verify and includes documentation and the test cases.

ogait87@mypc:~/cpython$ hg sum
parent: 71435:6f9d917df541 tip
 Fix test_multiprocessing failure under Windows.
branch: default
commit: 7 modified
update: (current)
msg141058 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2011-07-24 20:54
Thanks for updating your patch.
I've done a review, available here:
http://bugs.python.org/review/10141/show
msg141534 - (view) Author: Tiago Gonçalves (ogait87) Date: 2011-08-01 20:57
I think that this time i have included almost every proposed changes.

For the unitTest I did not included the condition "os.getuid() == 0" because of this information in the Linux documentation "3.3 network security issues (capabilities)" http://www.mjmwired.net/kernel/Documentation/networking/can.txt#212. For example, the Ubuntu 10.10 does not require to be root to access socketCAN network interfaces.

For the "return Py_BuildValue("O&h", PyUnicode_DecodeFSDefault, ifname, a->can_family);" i have kept the "h" because in the header file the "a->can_family" is defined as an short int.
msg144416 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2011-09-22 21:06
Here's an updated patch, with more tests.
Please review!
msg144434 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2011-09-23 00:17
socketcan_v4.patch:
 - dummy question: why an address is a tuple with 1 string instead of just the string? Does AF_UNIX also uses a tuple of 1 string?
 - the example should also use struct.pack() to create the frame, I don't like hardcoded BLOB
 - in test_socket: _have_socket_can() interprets permission denied as "CAN is not supported", it would be nice to provide a better skip message. Create maybe a decorator based?
 - _have_socket_can(): you may move s.close() outside the try block (add maybe a "else:" block?) because you may hide a real bug in .close()
 - data += b'\0' * (8 - can_dlc): I prefer data = data.ljust(8, '\x00')
 - you might add frame encoder/decoder in your example
 - if (!strcmp(PyBytes_AS_STRING(interfaceName), "")) hum..... PyBytes_GET_SIZE(intername)==0 should be enough
 - you truncate the interface name, it can be surprising, I would prefer an error (e.g. "interface name too long: 20 characters, the maximum is 10 characters" ?)
 - (oh no! don't include horrible configure diff in patches for the bug tracker :-p)

In which Linux version was CAN introduced?
msg144476 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2011-09-23 21:33
>  - dummy question: why an address is a tuple with 1 string instead of just
> the string? Does AF_UNIX also uses a tuple of 1 string?

I think the reason behind the tuple is future proofing.
Here's the definition of `struct sockaddr_can` in my Linux box's headers:
"""
/**
 * struct sockaddr_can - the sockaddr structure for CAN sockets
 * @can_family:  address family number AF_CAN.
 * @can_ifindex: CAN network interface index.
 * @can_addr:    protocol specific address information
 */
struct sockaddr_can {
    sa_family_t can_family;
    int         can_ifindex;
    union {
        /* transport protocol class address information (e.g. ISOTP) */
        struct { canid_t rx_id, tx_id; } tp;

        /* reserved for future CAN protocols address information */
    } can_addr;
};
"""

By making it a tuple, it will be easier to extend the address that
must be passed to bind(2), should it ever evolve, in a backward
compatible way. Well, that's just a guess (I'm by no means a SocketCAN
expert :-).

>  - the example should also use struct.pack() to create the frame, I don't
> like hardcoded BLOB

Done.

>  - in test_socket: _have_socket_can() interprets permission denied as "CAN
> is not supported", it would be nice to provide a better skip message. Create
> maybe a decorator based?

AFAICT, it shouldn't fail with EPERM or so.
Also, I'm not sure what the message would look like, and it's probably
a bit overkill.

>  - _have_socket_can(): you may move s.close() outside the try block (add
> maybe a "else:" block?) because you may hide a real bug in .close()

Changed that.

>  - data += b'\0' * (8 - can_dlc): I prefer data = data.ljust(8, '\x00')

Hum... Done.

>  - you might add frame encoder/decoder in your example

Done.

>  - if (!strcmp(PyBytes_AS_STRING(interfaceName), "")) hum.....
> PyBytes_GET_SIZE(intername)==0 should be enough

Done.

>  - you truncate the interface name, it can be surprising, I would prefer an
> error (e.g. "interface name too long: 20 characters, the maximum is 10
> characters" ?)

I changed that, and added a test. Also, note that AF_PACKET suffers
from the same problem.
I'll submit a separate patch.

>  - (oh no! don't include horrible configure diff in patches for the bug
> tracker :-p)

Yeah, I usually take care of that, but forgot this time.

> In which Linux version was CAN introduced?
>

Apparently, 2.6.25. Note that we don't need
@support.requires_linux_version() though, it should be catched by
HAVE_SOCKET_CAN (also, you can't use it as a class decorator...).

Here's the updated patch. It passes on all the buildbots (of course,
it's only relevant on Linux).
msg144777 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2011-10-02 16:45
So, Victor, what do you think of the last version?
This patch has been lingering for quite some time, and it's really a
cool feature.
msg144959 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2011-10-05 17:04
I don't have much to say about the patch, given that I don't know anything about CAN and my system doesn't appear to have a "vcan0" interface. I think it's ok to commit and refine later if something turns out insufficient.
msg144960 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2011-10-05 17:09
> I don't have much to say about the patch, given that I don't know
> anything about CAN and my system doesn't appear to have a "vcan0"
> interface.

I had never heard about it before this issue, but the protocol is really simple.

If you want to try it out (just for fun :-), you just have to do the following:
# modprobe vcan
# ip link add dev vcan0 type vcan
# ifconfig vcan0 up
msg144961 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2011-10-05 17:17
> I had never heard about it before this issue, but the protocol is really simple.
> 
> If you want to try it out (just for fun :-), you just have to do the following:
> # modprobe vcan
> # ip link add dev vcan0 type vcan
> # ifconfig vcan0 up

Ah, thanks! Can you add a comment about that in test_socket.py?
I can confirm that all tests pass ok on my Linux system (kernel
2.6.38.8-desktop-5.mga).
msg145028 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2011-10-06 17:45
New changeset e767318baccd by Charles-François Natali in branch 'default':
Issue #10141: socket: add SocketCAN (PF_CAN) support. Initial patch by Matthias
http://hg.python.org/cpython/rev/e767318baccd
msg145029 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2011-10-06 18:28
New changeset a4af684bb54e by Victor Stinner in branch 'default':
Issue #10141: Don't use hardcoded frame size in example, use struct.calcsize()
http://hg.python.org/cpython/rev/a4af684bb54e
msg145032 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2011-10-06 19:47
Committed.
Matthias, Tiago, thanks!
msg145070 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2011-10-07 11:48
From python-dev:
"""
I work on Ubuntu Jaunty for my cpython development work - an old version, I
know, but still quite serviceable and has worked well for me over many months.
With the latest default cpython repository, however, I can't run the regression
suite because the socket module now fails to build:

gcc -pthread -fPIC -g -O0 -Wall -Wstrict-prototypes -IInclude -I.
  -I./Include -I/usr/local/include -I/home/vinay/projects/python/default -c
  /home/vinay/projects/python/default/Modules/socketmodule.c
  -o build/temp.linux-i686-3.3-pydebug/home/vinay/projects/python/default
  /Modules/socketmodule.o
.../Modules/socketmodule.c: In function
‘makesockaddr’:
.../Modules/socketmodule.c:1224: error: ‘AF_CAN’ undeclared (first use in
this function)
.../Modules/socketmodule.c:1224: error: (Each undeclared identifier is
reported only once
.../Modules/socketmodule.c:1224: error: for each function it appears in.)
.../Modules/socketmodule.c: In function
‘getsockaddrarg’:
.../Modules/socketmodule.c:1610: error: ‘AF_CAN’ undeclared (first use in
this function)
.../Modules/socketmodule.c: In function ‘getsockaddrlen’:
.../Modules/socketmodule.c:1750: error: ‘AF_CAN’ undeclared (first use in
this function)

On this system, AF_CAN *is* defined, but in linux/socket.h, not in sys/socket.h.
From what I can see, sys/socket.h includes bits/socket.h which includes
asm/socket.h, but apparently linux/socket.h isn't included.
"""

Vinay, what happens if you replace in Modules/socketmodule.h:
"""
#ifdef HAVE_LINUX_CAN_H
#include <linux/can.h>
#endif
"""

with

"""
#ifdef HAVE_LINUX_CAN_H
#include <linux/socket.h>
#include <linux/can.h>
#endif
"""
msg145081 - (view) Author: Vinay Sajip (vinay.sajip) * (Python committer) Date: 2011-10-07 14:55
I added the line in the suggested place:

#ifdef HAVE_LINUX_TIPC_H
# include <linux/tipc.h>
#endif

#ifdef HAVE_LINUX_CAN_H
#include <linux/socket.h>  /* the line I added - line 76 */
#include <linux/can.h>
#endif

#ifdef HAVE_LINUX_CAN_RAW_H
#include <linux/can/raw.h>
#endif

Strangely, it seems to make no difference! I copied out the actual gcc line from the make file and ran it from the command line to just pre-process:

gcc -pthread -fPIC -g -O0 -Wall -Wstrict-prototypes -IInclude -I. -I./Include -I/usr/local/include -I/home/vinay/projects/python/default -E /home/vinay/projects/python/default/Modules/socketmodule.c >/tmp/tmp.c

(I assume /usr/include is always searched, as it's not explicitly named in a -I parameter.)

Looking at the tmp.c file produced, here's the relevant section:

# 182 "/usr/include/linux/tipc.h" 3 4
struct sockaddr_tipc {
 unsigned short family;
 unsigned char addrtype;
 signed char scope;
 union {
  struct tipc_portid id;
  struct tipc_name_seq nameseq;
  struct {
   struct tipc_name name;
   __u32 domain;
  } name;
 } addr;
};
# 73 "/home/vinay/projects/python/default/Modules/socketmodule.h" 2




# 1 "/usr/include/linux/can.h" 1 3 4

After the tipc.h include, the can.h include comes next, as if I hadn't added the linux/socket.h line at all! What is this I don't even ...

I pasted the whole socketmodule.h file to a gist here:

https://gist.github.com/1270436

Tell me I'm not going mad!
msg145083 - (view) Author: Vinay Sajip (vinay.sajip) * (Python committer) Date: 2011-10-07 15:01
Just to test, I added the full absolute path name in socketmodule.h:

#ifdef HAVE_LINUX_CAN_H
#include "/usr/include/linux/socket.h"
#include <linux/can.h>
#endif

Now, the snippet from pre-processing is:

# 182 "/usr/include/linux/tipc.h" 3 4
struct sockaddr_tipc {
 unsigned short family;
 unsigned char addrtype;
 signed char scope;
 union {
  struct tipc_portid id;
  struct tipc_name_seq nameseq;
  struct {
   struct tipc_name name;
   __u32 domain;
  } name;
 } addr;
};
# 73 "/home/vinay/projects/python/default/Modules/socketmodule.h" 2



# 1 "/usr/include/linux/socket.h" 1
# 77 "/home/vinay/projects/python/default/Modules/socketmodule.h" 2
# 1 "/usr/include/linux/can.h" 1 3 4
# 41 "/usr/include/linux/can.h" 3 4

This implies that the linux/socket.h file is not being read at all. First part of linux/socket.h is:

#ifndef _LINUX_SOCKET_H
#define _LINUX_SOCKET_H
msg145088 - (view) Author: Vinay Sajip (vinay.sajip) * (Python committer) Date: 2011-10-07 15:10
Ok, I found that linux/socket.h *is* actually included earlier, from /usr/include/linux/netlink.h. However, the AF_CAN definition appears only under the condition

#if defined(__KERNEL__) || !defined(__GLIBC__) || (__GLIBC__ < 2)

...
#define AF_CAN		29	/* Controller Area Network      */
...

#endif /* not kernel and not glibc */
#endif /* _LINUX_SOCKET_H */

which would imply that on this system at least, the AF_CAN definition is supposed to come from elsewhere. I did a recursive grep in /usr/include: the only place AF_CAN is defined is linux/socket.h.
msg145093 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2011-10-07 16:03
> which would imply that on this system at least, the AF_CAN definition is 
> supposed to come from elsewhere.

Yes, from <bits/socket.h>.
Looks like a crappy libc version: <linux/can.h> is present, but AF_CAN is not defined.
Just for fun, is PF_CAN defined?

You might try the following in configure.in:
"""
# On Linux, can.h and can/raw.h require sys/socket.h
AC_CHECK_HEADERS(linux/can.h linux/can/raw.h,,,[
#ifdef HAVE_SYS_SOCKET_H
#include <sys/socket.h>
#ifndef AF_CAN
# error "AF_CAN not defined"
#endif
#endif
])
"""
msg145100 - (view) Author: Vinay Sajip (vinay.sajip) * (Python committer) Date: 2011-10-07 16:31
PF_CAN is defined by

#define PF_CAN		AF_CAN

in linux/socket.h :-(

Making the change in configure.in didn't lead to any change: no error when I ran configure (which is ./configure --with-py-debug in my case), and when I build, the same error (AF_CAN undefined) occurs. Just to eliminate any extraneous variables and be absolutely sure, the program

#include <sys/socket.h>
#include <stdio.h>

int main(int argc, char ** argv)
{
    printf("AF_CAN is %d\n", AF_CAN);
}

also fails with the same error.
msg145101 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2011-10-07 16:33
> Making the change in configure.in didn't lead to any change: no error
> when I ran configure

Did you run autoconf (or autoreconf) before?
msg145103 - (view) Author: Vinay Sajip (vinay.sajip) * (Python committer) Date: 2011-10-07 16:46
> Did you run autoconf (or autoreconf) before?

D'oh! I didn't realise I had to, though in retrospect it should have been obvious ... autoconf isn't even installed on my system, and I can't install it at the moment because of some problem with the Ubuntu repos for Jaunty.
msg145107 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2011-10-07 17:12
Here's a better patch.
It only touches socketmodule.c, so no need for autoconf, just use make.
If it works on your box, I'll test it on the custom buildbots before pushing it for good (if I learned one thing, it's to never underestimate the potential for broken headers/libc).
msg145109 - (view) Author: Vinay Sajip (vinay.sajip) * (Python committer) Date: 2011-10-07 17:23
That did the trick - I can now build the socket module. Thanks!

I noticed that the module initialisation code uses #ifdef AF_CAN and #ifdef PF_CAN rather than #ifdef HAVE_LINUX_CAN_H :-)
msg145143 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2011-10-07 20:44
New changeset d4ce850b06b7 by Charles-François Natali in branch 'default':
Issue #10141: fix socketmodule compilation on Linux systems with <linux/can.h>
http://hg.python.org/cpython/rev/d4ce850b06b7
msg145165 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2011-10-08 10:56
Working fine on the buildbots and Vinay's box, closing!
msg214213 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2014-03-20 12:44
New changeset df427bf067d7 by Vinay Sajip in branch '3.4':
Issue #10141: updated new usages of AF_CAN to be in #ifdef AF_CAN rather than #ifdef HAVE_LINUX_CAN_H to allow compilation on older Linuxes.
http://hg.python.org/cpython/rev/df427bf067d7
msg214262 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2014-03-20 19:17
Vinay, your change just reverted this: http://bugs.python.org/issue20065

Basically, AF_CAN being defined doesn't imply that CAN_RAW is defined.

So compilation will now fail - again - on those hosts.
msg214308 - (view) Author: Vinay Sajip (vinay.sajip) * (Python committer) Date: 2014-03-20 23:37
Sorry if I messed up - I just applied the same logic as I thought you had done earlier (replacing #ifdef HAVE_LINUX_CAN_H with #ifdef AF_CAN), and everything compiled OK after my changes.

Are you saying that an additional clause for CAN_RAW being defined should be added around where it is used? Would that sort things out? I'd rather not just revert my change, as that would mean I couldn't compile the SSL module.
msg214328 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2014-03-21 06:43
> Are you saying that an additional clause for CAN_RAW being defined should
be added around where it is used? Would that sort things out?

Yes.

> I'd rather not just revert my change, as that would mean I couldn't
compile the SSL module.

I don't get it: how could the previous code prevent the SSL module from
being built?
What error were you getting?
msg214343 - (view) Author: Vinay Sajip (vinay.sajip) * (Python committer) Date: 2014-03-21 11:02
> What error were you getting?

That AF_CAN was undefined (even though HAVE_LINUX_CAN_H is). This is on Ubuntu Jaunty, which I use for my Python core development.

Note the change you made in d4ce850b06b7 to fix this problem when it occurred before - it's the same approach as my recent change that we're talking about. It seems that in 3.4, some more instances of AF_CAN usage were added, but wrapped in #ifdef HAVE_LINUX_CAN_H, which led to the new compilation failures.

I'll make the changes to take CAN_RAW into account.
msg214349 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2014-03-21 11:45
New changeset 9dc199b921eb by Vinay Sajip in branch '3.4':
Issue #10141, Issue 20065: Changed #if to take CAN_RAW into account.
http://hg.python.org/cpython/rev/9dc199b921eb

New changeset 20cced06acdd by Vinay Sajip in branch 'default':
Issue #10141, Issue 20065: Merged from 3.4.
http://hg.python.org/cpython/rev/20cced06acdd
msg214391 - (view) Author: Charles-François Natali (neologix) * (Python committer) Date: 2014-03-21 17:03
> That AF_CAN was undefined (even though HAVE_LINUX_CAN_H is). This is on
Ubuntu Jaunty, which I use for my Python core development.

How dear...

The latest change should be OK.
msg255792 - (view) Author: Martin Panter (martin.panter) * (Python committer) Date: 2015-12-02 22:16
Sounds like the last problem has been fixed, so can we close this?
msg297725 - (view) Author: Riccardo Magliocchetti (Riccardo Magliocchetti) * Date: 2017-07-05 11:08
I have an issue related to this while trying to compile statically Python 3.6.1 against a static musl.

The problem is that i have AF_CAN defined because it's defined in linux/socket.h but by not having HAVE_LINUX_CAN_H defined in pyconfig.h the header which contains the definition of struct sockaddr_can is not included. So i think (at least for linux) using AF_CAN for the conditionals is wrong and the HAVE_LINUX_CAN_H should be used instead.

I think the same applies for CAN_RAW and CAN_BCM because they are defined in the generic linux/can.h and not in a feature specific header.
msg297795 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2017-07-06 03:54
This issue is closed.  Please open a new issue for your problem and proposal.
History
Date User Action Args
2022-04-11 14:57:07adminsetgithub: 54350
2020-12-16 08:54:40python-devsetpull_requests: + pull_request22650
2017-07-06 03:54:49r.david.murraysetnosy: + r.david.murray
messages: + msg297795
2017-07-05 11:08:09Riccardo Magliocchettisetnosy: + Riccardo Magliocchetti
messages: + msg297725
2015-12-04 09:13:48vinay.sajipsetstatus: open -> closed
2015-12-02 22:16:32martin.pantersetnosy: + martin.panter
messages: + msg255792
2014-03-21 17:03:00neologixsetmessages: + msg214391
2014-03-21 11:45:40python-devsetmessages: + msg214349
2014-03-21 11:02:20vinay.sajipsetmessages: + msg214343
2014-03-21 06:43:01neologixsetmessages: + msg214328
2014-03-20 23:37:51vinay.sajipsetmessages: + msg214308
2014-03-20 19:17:59neologixsetstatus: closed -> open

messages: + msg214262
2014-03-20 12:44:01python-devsetmessages: + msg214213
2011-10-08 10:56:55neologixsetstatus: open -> closed
resolution: fixed
messages: + msg145165

stage: needs patch -> resolved
2011-10-07 20:44:49python-devsetmessages: + msg145143
2011-10-07 17:23:28vinay.sajipsetmessages: + msg145109
2011-10-07 17:12:36neologixsetfiles: + af_can_undefined.diff

messages: + msg145107
2011-10-07 16:46:54vinay.sajipsetmessages: + msg145103
2011-10-07 16:33:06pitrousetmessages: + msg145101
2011-10-07 16:31:25vinay.sajipsetmessages: + msg145100
2011-10-07 16:03:55neologixsetmessages: + msg145093
2011-10-07 15:10:44vinay.sajipsetmessages: + msg145088
2011-10-07 15:01:41vinay.sajipsetmessages: + msg145083
2011-10-07 14:55:32vinay.sajipsetmessages: + msg145081
2011-10-07 11:48:41neologixsetstatus: closed -> open

nosy: + vinay.sajip
messages: + msg145070

resolution: fixed -> (no value)
stage: resolved -> needs patch
2011-10-06 19:47:47neologixsetstatus: open -> closed
resolution: fixed
messages: + msg145032

stage: commit review -> resolved
2011-10-06 18:28:02python-devsetmessages: + msg145029
2011-10-06 17:45:32python-devsetnosy: + python-dev
messages: + msg145028
2011-10-05 17:17:18pitrousetmessages: + msg144961
2011-10-05 17:09:23neologixsetmessages: + msg144960
2011-10-05 17:04:15pitrousetmessages: + msg144959
2011-10-05 16:47:27neologixsetnosy: + pitrou
2011-10-02 16:45:32neologixsetmessages: + msg144777
2011-09-23 21:35:17neologixsetfiles: - socketcan_v4.patch
2011-09-23 21:33:35neologixsetfiles: + socketcan_v5.patch

messages: + msg144476
2011-09-23 00:17:40vstinnersetmessages: + msg144434
2011-09-22 21:06:50neologixsetfiles: + socketcan_v4.patch

nosy: + vstinner
messages: + msg144416

keywords: + needs review
stage: patch review -> commit review
2011-08-01 20:57:53ogait87setfiles: + socketcan_v2.patch

messages: + msg141534
2011-07-24 20:54:38neologixsetmessages: + msg141058
2011-07-20 01:33:10ogait87setfiles: + socketcan.patch
keywords: + patch
messages: + msg140713
2011-07-20 01:20:21mluissetnosy: + mluis
2011-07-04 16:55:26neologixsetmessages: + msg139786
2011-07-04 13:29:44pitrousetnosy: + neologix, rosslagerwall

versions: - Python 3.2
2011-07-04 12:51:58ogait87setnosy: + ogait87
messages: + msg139763
2010-11-13 09:54:34amaury.forgeotdarcsetnosy: + amaury.forgeotdarc
messages: + msg121125
2010-10-19 09:34:44giampaolo.rodolasetnosy: + giampaolo.rodola
2010-10-19 05:42:16slandensetmessages: + msg119123
2010-10-19 05:39:18slandensetmessages: + msg119122
2010-10-19 04:47:18orsenthilsetversions: + Python 3.2, - Python 2.6
nosy: + orsenthil

messages: + msg119120

stage: patch review
2010-10-19 00:49:25slandencreate