Skip to content

ZD15937 - Add more detail on configuring wolfSSL for CAN-bus example in README#380

Merged
anhu merged 1 commit intowolfSSL:masterfrom
tim-weller-wolfssl:can-bus-readme-update
Apr 4, 2023
Merged

ZD15937 - Add more detail on configuring wolfSSL for CAN-bus example in README#380
anhu merged 1 commit intowolfSSL:masterfrom
tim-weller-wolfssl:can-bus-readme-update

Conversation

@tim-weller-wolfssl
Copy link
Copy Markdown
Contributor

Fixes: zd15937

Add ./configure shell command to enable ISOTP support in wolfSSL for can-bus example.

@tim-weller-wolfssl tim-weller-wolfssl requested a review from anhu April 4, 2023 15:18
@tim-weller-wolfssl tim-weller-wolfssl marked this pull request as ready for review April 4, 2023 15:18
Comment thread can-bus/README.md Outdated
You need to have wolfSSL installed on your computer prior to building, this will need to be built with `WOLFSSL_ISOTP` defined to provide ISO-TP functionality.
You need to have wolfSSL installed on your computer prior to building, this will need to be built with `WOLFSSL_ISOTP` defined to provide ISO-TP functionality:
```sh
./configure CFLAGS="-DWOLFSSL_ISOTP"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See ZD ticket. There are more stuff to add here (autogen, make , etc., etc.)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes made based on text in ZD ticket.

Copy link
Copy Markdown
Member

@anhu anhu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a bit more detail please.

Copy link
Copy Markdown
Member

@anhu anhu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

line 35: client --> server

Copy link
Copy Markdown
Member

@anhu anhu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!! :)

@anhu anhu merged commit 9688f1b into wolfSSL:master Apr 4, 2023
@tim-weller-wolfssl tim-weller-wolfssl deleted the can-bus-readme-update branch April 12, 2023 11:47
yota22721 pushed a commit to yota22721/wolfssl-examples that referenced this pull request Jan 25, 2025
…-update

ZD15937 - Add more detail on configuring wolfSSL for CAN-bus example in README
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.

3 participants