Skip to content

tests/nanocoap_cli: allow to build with lwIP #21314

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Mar 23, 2025
Merged

tests/nanocoap_cli: allow to build with lwIP #21314

merged 1 commit into from
Mar 23, 2025

Conversation

benpicco
Copy link
Contributor

Contribution description

Allow to build the nanoCoAP test with lwIP to run it on IPv4-only networks.

Testing procedure

Run the test with LWIP_IPV4=1

2025-03-22 10:06:21,011 # > ifconfig
2025-03-22 10:06:21,016 # Iface ET0 HWaddr: f8:f0:05:a9:ec:19 Link: up State: up
2025-03-22 10:06:21,019 #         Link type: wireless
2025-03-22 10:06:21,024 #         inet addr: 10.53.12.146 mask: 255.255.0.0 gw: 10.53.0.1
2025-03-22 10:06:21,029 # Iface ET1 HWaddr: fc:c2:3d:22:12:3b Link: down State: up
2025-03-22 10:06:21,031 #         Link type: wired
2025-03-22 10:06:21,036 #         inet addr: 0.0.0.0 mask: 0.0.0.0 gw: 0.0.0.0

2025-03-22 10:06:23,855 # > ncget coap://coap.me/
2025-03-22 10:06:24,973 # /test
2025-03-22 10:06:24,974 # /validate
2025-03-22 10:06:25,007 # /hello
2025-03-22 10:06:25,035 # /blåbærsyltetøy
2025-03-22 10:06:25,036 # /sink
2025-03-22 10:06:25,059 # /separate
2025-03-22 10:06:25,060 # /large
2025-03-22 10:06:25,085 # /secret
2025-03-22 10:06:28,075 # /broken
2025-03-22 10:06:28,076 # /weird33
2025-03-22 10:06:28,076 # /weird44
2025-03-22 10:06:28,107 # /weird55
2025-03-22 10:06:28,108 # /weird333
2025-03-22 10:06:28,556 # /weird3333
2025-03-22 10:06:28,557 # /weird33333
2025-03-22 10:06:28,589 # /123412341234123412341234
2025-03-22 10:06:28,617 # /location-query
2025-03-22 10:06:28,643 # /create1
2025-03-22 10:06:28,644 # /large-update
2025-03-22 10:06:28,672 # /large-create
2025-03-22 10:06:28,673 # /query
2025-03-22 10:06:28,701 # /seg1
2025-03-22 10:06:28,701 # /path
2025-03-22 10:06:28,702 # /location1
2025-03-22 10:06:28,727 # /multi-format
2025-03-22 10:06:28,727 # /3
2025-03-22 10:06:28,807 # /4
2025-03-22 10:06:28,807 # /5

Issues/PRs references

@github-actions github-actions bot added the Area: tests Area: tests and testing framework label Mar 22, 2025
@benpicco benpicco requested review from maribu and removed request for aabadie, leandrolanzieri and MichelRottleuthner March 22, 2025 09:08
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Mar 22, 2025
@benpicco benpicco requested a review from mguetschow March 22, 2025 09:09
@riot-ci
Copy link

riot-ci commented Mar 22, 2025

Murdock results

✔️ PASSED

99637ed tests/nanocoap_cli: allow to build with lwIP

Success Failures Total Runtime
19 0 19 01m:39s

Artifacts

@krzysztof-cabaj
Copy link
Contributor

@benpicco do you plan similar feature for examples/networking/coap/nanocoap_server?

@benpicco
Copy link
Contributor Author

That should be easy to add there too

@maribu maribu enabled auto-merge March 23, 2025 05:05
@maribu maribu added this pull request to the merge queue Mar 23, 2025
Merged via the queue into RIOT-OS:master with commit 652d1d3 Mar 23, 2025
27 checks passed
@benpicco benpicco deleted the nanocoap-lwip branch March 23, 2025 09:24
@krzysztof-cabaj krzysztof-cabaj mentioned this pull request Mar 28, 2025
Merged
@mguetschow mguetschow added this to the Release 2025.04 milestone Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maribu maribu maribu approved these changes

@mguetschow mguetschow Awaiting requested review from mguetschow

Assignees
No one assigned
Labels
Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Milestone
Release 2025.04
Development

Successfully merging this pull request may close these issues.
5 participants
@benpicco @riot-ci @krzysztof-cabaj @maribu @mguetschow