-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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.
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
Register the DNS service for the accessory with a valid hostname #22010
Register the DNS service for the accessory with a valid hostname #22010
Conversation
4b23c29
to
726c61d
Compare
PR #22010: Size comparison from b4776ad to 726c61d Increases (6 builds for bl602, cc13x2_26x2, psoc6)
Decreases (7 builds for bl602, cc13x2_26x2, cyw30739, nrfconnect, telink)
Full report (32 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, telink)
|
fead8b7
to
6c85214
Compare
PR #22010: Size comparison from b4505ac to 6c85214 Increases (4 builds for esp32, psoc6)
Decreases (4 builds for efr32, psoc6, telink)
Full report (35 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Code looks great. Thanks Vivien. |
6c85214
to
ff0ec51
Compare
I have updated the code a little bit to address some of the comments I made
PR #22010: Size comparison from 0e3bbae to ff0ec51 Increases (3 builds for esp32, nrfconnect, telink)
Decreases (6 builds for bl602, psoc6, qpg, telink)
Full report (46 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
ff0ec51
to
3a84639
Compare
PR #22010: Size comparison from 27f9fb4 to 3a84639 Increases (1 build for telink)
Decreases (2 builds for bl602, nrfconnect)
Full report (14 builds for bl602, k32w, linux, mbed, nrfconnect, qpg, telink)
|
3dfb177
to
e784ea7
Compare
PR #22010: Size comparison from 27f9fb4 to e784ea7 Increases (3 builds for bl602, cc13x2_26x2, telink)
Decreases (4 builds for cc13x2_26x2, esp32, telink)
Full report (35 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
e784ea7
to
47e6055
Compare
PR #22010: Size comparison from 8af13a2 to 47e6055 Increases (4 builds for esp32, nrfconnect, psoc6)
Decreases (5 builds for bl602, nrfconnect, psoc6, telink)
Full report (35 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
- Call DNSServiceRegisterRecord to register a record for the hostname for all interfaces before calling DNSServiceRegister.
47e6055
to
8d80336
Compare
PR #22010: Size comparison from d7dc898 to 8d80336 Increases (7 builds for bl602, cc13x2_26x2, esp32, psoc6, telink)
Decreases (2 builds for cc13x2_26x2, psoc6)
Full report (35 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
…ject-chip#22010) - Call DNSServiceRegisterRecord to register a record for the hostname for all interfaces before calling DNSServiceRegister.
before calling DNSServiceRegister.
Problem
What is being fixed? Examples:
Change overview
Register a DNS service with a valid hostname
Testing
How was this tested? (at least one bullet point required)
*Tested using mac all-clusters-app and chip-tool