Skip to content

Fix #1376#1412

Merged
zariiii9003 merged 2 commits into
hardbyte:developfrom
zariiii9003:fix_1376
Oct 31, 2022
Merged

Fix #1376#1412
zariiii9003 merged 2 commits into
hardbyte:developfrom
zariiii9003:fix_1376

Conversation

@zariiii9003

Copy link
Copy Markdown
Collaborator

Closes #1376

@umlaeute Could you run your tests with this branch?

@codecov

codecov Bot commented Oct 15, 2022

Copy link
Copy Markdown

Codecov Report

Merging #1412 (248b40a) into develop (0c82d2c) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1412   +/-   ##
========================================
  Coverage    65.27%   65.27%           
========================================
  Files           81       81           
  Lines         8777     8777           
========================================
  Hits          5729     5729           
  Misses        3048     3048           

@zariiii9003

Copy link
Copy Markdown
Collaborator Author

ping @umlaeute

@umlaeute

umlaeute commented Oct 31, 2022

Copy link
Copy Markdown
Contributor

I can confirm that this PR fixes the test/test_socketcan.py::SocketCANTest::test_build_bcm_header test on BigEndian (s390x).

unfortunately the testsuite still fails on s390x, but this time with test/test_vector.py::test_get_channel_configs (which is a new test, that was not present in the 4.0.0. release).
EDIT since I believe this new test failure to be unrelated to the original one (#1376), i've opened a separate ticket #1424.

from my side, the fix looks good.

@zariiii9003

Copy link
Copy Markdown
Collaborator Author

Thank you for the feedback. I'll create another PR for that, too.

@zariiii9003 zariiii9003 merged commit a344a13 into hardbyte:develop Oct 31, 2022
@zariiii9003 zariiii9003 deleted the fix_1376 branch October 31, 2022 13:08
j-c-cook added a commit to j-c-cook/python-can that referenced this pull request Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

big endian support broken(?)

2 participants