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.
Created on 2014年10月14日 14:21 by rumpelsepp, last changed 2022年04月11日 14:58 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| CAN_RAW_FD_FRAMES.patch | llllllllll, 2015年04月13日 17:02 | review | ||
| CAN_RAW_FD_FRAMES-update.patch | llllllllll, 2015年04月13日 19:50 | review | ||
| CAN_RAW_FD_FRAMES-doc.patch | llllllllll, 2015年04月13日 20:30 | review | ||
| Messages (17) | |||
|---|---|---|---|
| msg229289 - (view) | Author: Stefan Tatschner (rumpelsepp) * | Date: 2014年10月14日 14:21 | |
CAN support was introduced with issue #10141. Python still seems to lack support for CAN FD which is available with the socket option CAN_RAW_FD_FRAMES, see here (chapter 4.1.5): https://www.kernel.org/doc/Documentation/networking/can.txt |
|||
| msg229339 - (view) | Author: Charles-François Natali (neologix) * (Python committer) | Date: 2014年10月14日 19:41 | |
Annoying. I thought CAN_RAW_FD_FRAME would be a macro, which would have made conditional compilation easy, but it's apparently a enum value, which means we have to add a configure-time check... |
|||
| msg240662 - (view) | Author: Joe Jevnik (llllllllll) * | Date: 2015年04月13日 17:02 | |
I have added a patch to update this with conditional compilation. |
|||
| msg240663 - (view) | Author: Joe Jevnik (llllllllll) * | Date: 2015年04月13日 17:08 | |
To show where I got my sources: https://github.com/torvalds/linux/commit/e2d265d3b587f5f6f8febc0222aace93302ff0be There does not appear to be any new structures needed other than supporting the constant. |
|||
| msg240680 - (view) | Author: Larry Hastings (larry) * (Python committer) | Date: 2015年04月13日 17:41 | |
LGTM, just adding Antoine (the expert for the socket module). I'm ready to commit over here. |
|||
| msg240709 - (view) | Author: Berker Peksag (berker.peksag) * (Python committer) | Date: 2015年04月13日 19:19 | |
The new constant should probably be added to https://docs.python.org/3/library/socket.html#constants (with a .. versionadded:: 3.5 directive) |
|||
| msg240716 - (view) | Author: Joe Jevnik (llllllllll) * | Date: 2015年04月13日 19:50 | |
I am not going to be able to write docs on this because I am not totally sure how you would use this; There is an entry in the docs for CAN_* so that might be enough. I have a line now saying that this particular flag is available as of 3.5 though. |
|||
| msg240723 - (view) | Author: Larry Hastings (larry) * (Python committer) | Date: 2015年04月13日 20:03 | |
Follow the instructions in this: https://docs.python.org/devguide/docquality.html And make it look something like this: https://docs.python.org/3/library/socket.html#socket.CAN_BCM |
|||
| msg240736 - (view) | Author: Joe Jevnik (llllllllll) * | Date: 2015年04月13日 20:30 | |
I tried to provide a summary of the Linux page on the topic and then said that it was fully documented there. |
|||
| msg240773 - (view) | Author: Roundup Robot (python-dev) (Python triager) | Date: 2015年04月13日 21:49 | |
New changeset 03ff92b26fa9 by Larry Hastings in branch 'default': Issue #22631: Added Linux-specific socket constant CAN_RAW_FD_FRAMES. https://hg.python.org/cpython/rev/03ff92b26fa9 |
|||
| msg240774 - (view) | Author: Larry Hastings (larry) * (Python committer) | Date: 2015年04月13日 21:49 | |
Patch committed. Thanks, Joe! |
|||
| msg240799 - (view) | Author: Matthias Klose (doko) * (Python committer) | Date: 2015年04月13日 23:02 | |
it would be nice to have these constants backported to both 2.7 and 3.4. It may be good to check for other missing consgtants in this module in 2.7 and 3.4. |
|||
| msg240841 - (view) | Author: Joe Jevnik (llllllllll) * | Date: 2015年04月14日 05:08 | |
So I think the patch could be applied to 3.4; I will look into moving the other back sometime this week. |
|||
| msg240957 - (view) | Author: Larry Hastings (larry) * (Python committer) | Date: 2015年04月14日 17:07 | |
Isn't this a "new feature"? 2.7 is maybe fair game, but 3.4 arguably is not. It should be odd to have a field marked in the documentation as "New in 3.4.4." |
|||
| msg240958 - (view) | Author: Antoine Pitrou (pitrou) * (Python committer) | Date: 2015年04月14日 17:08 | |
It is a new feature. |
|||
| msg240959 - (view) | Author: Larry Hastings (larry) * (Python committer) | Date: 2015年04月14日 17:09 | |
Benjamin, would you like CAN_RAW_FD_FRAMES in the next 2.7? |
|||
| msg240969 - (view) | Author: R. David Murray (r.david.murray) * (Python committer) | Date: 2015年04月14日 18:10 | |
There is no CAN support in 2.7, so no. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:58:09 | admin | set | github: 66821 |
| 2015年04月14日 18:10:37 | r.david.murray | set | nosy:
+ r.david.murray messages: + msg240969 |
| 2015年04月14日 17:09:39 | larry | set | nosy:
+ benjamin.peterson messages: + msg240959 |
| 2015年04月14日 17:08:29 | pitrou | set | messages: + msg240958 |
| 2015年04月14日 17:07:43 | larry | set | messages: + msg240957 |
| 2015年04月14日 05:08:04 | llllllllll | set | messages: + msg240841 |
| 2015年04月13日 23:02:10 | doko | set | nosy:
+ doko messages: + msg240799 |
| 2015年04月13日 21:49:43 | larry | set | status: open -> closed messages: + msg240774 assignee: larry resolution: fixed stage: commit review -> resolved |
| 2015年04月13日 21:49:01 | python-dev | set | nosy:
+ python-dev messages: + msg240773 |
| 2015年04月13日 20:30:10 | llllllllll | set | files:
+ CAN_RAW_FD_FRAMES-doc.patch messages: + msg240736 |
| 2015年04月13日 20:03:07 | larry | set | messages: + msg240723 |
| 2015年04月13日 19:50:18 | llllllllll | set | files:
+ CAN_RAW_FD_FRAMES-update.patch messages: + msg240716 |
| 2015年04月13日 19:19:01 | berker.peksag | set | nosy:
+ berker.peksag messages: + msg240709 stage: commit review |
| 2015年04月13日 17:41:40 | larry | set | nosy:
+ pitrou messages: + msg240680 |
| 2015年04月13日 17:08:16 | llllllllll | set | nosy:
+ larry messages: + msg240663 |
| 2015年04月13日 17:02:05 | llllllllll | set | files:
+ CAN_RAW_FD_FRAMES.patch nosy: + llllllllll messages: + msg240662 keywords: + patch |
| 2014年10月15日 08:31:17 | rumpelsepp | set | title: Feature Request CAN_RAW_FD_FRAME -> Feature Request CAN_RAW_FD_FRAMES |
| 2014年10月14日 19:41:43 | neologix | set | messages:
+ msg229339 components: + Library (Lib), - IO |
| 2014年10月14日 19:01:43 | ned.deily | set | nosy:
+ neologix |
| 2014年10月14日 14:21:28 | rumpelsepp | create | |