-
Notifications
You must be signed in to change notification settings - Fork 6.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
samples: net: tftp-client: Configs cleanup #72919
samples: net: tftp-client: Configs cleanup #72919
Conversation
As native_posix is no longer supported, due to incompatibility with CONFIG_POSIX_API, remove a dedicated config for it. A config for qemu_cortex_m3 needed to enable RNG, as otherwise it's not available for the platform and gives build error. Finally, enable qemu_cortex_m3 build in the CI. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
8e7aca4
to
b6c6c8b
Compare
A support for RDNSS option should not force DNS subsystem being enabled in the system, especially that the option is enabled by default. It should work the other way around - the option is supported only if DNS was enabled by the user. Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
b6c6c8b
to
89d67f2
Compare
@@ -159,7 +159,7 @@ config NET_IPV6_DAD | |||
config NET_IPV6_RA_RDNSS | |||
bool "Support RA RDNSS option" | |||
depends on NET_IPV6_ND | |||
select DNS_RESOLVER | |||
depends on DNS_RESOLVER |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, I wondered exactly this weirdness in #72834 just few minutes ago. Good that it is now fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rlubos Wow! What an unbelievable coincidence - just found the same issue and tried disabling NET_IPV6_RA_RDNSS, as I don't use DNS_RESOLVER in my app. Your solution is definitely better and more desirable. Thank you!
@@ -159,7 +159,7 @@ config NET_IPV6_DAD | |||
config NET_IPV6_RA_RDNSS | |||
bool "Support RA RDNSS option" | |||
depends on NET_IPV6_ND | |||
select DNS_RESOLVER | |||
depends on DNS_RESOLVER |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rlubos Wow! What an unbelievable coincidence - just found the same issue and tried disabling NET_IPV6_RA_RDNSS, as I don't use DNS_RESOLVER in my app. Your solution is definitely better and more desirable. Thank you!
See commit messages for details.
Fixes #72907