Skip to content

Fix compilation error if DNS features are disabled #1243

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

marcusb
Copy link

@marcusb marcusb commented Apr 16, 2025

The code would fail to compile unless one of ipconfigUSE_DNS_CACHE,
ipconfigUSE_MDNS, ipconfigUSE_LLMNR, and ipconfigDNS_USE_CALLBACKS
were set.

Checklist:

  • I have tested my changes. No regression in existing tests.
  • I have modified and/or added unit-tests to cover the code changes in this Pull Request.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@marcusb marcusb requested a review from a team as a code owner April 16, 2025 04:09
@marcusb marcusb changed the title Fix compilation error if DNS cache is disabled Fix compilation error if DNS features are disabled Apr 16, 2025
@marcusb marcusb force-pushed the no-dns-cache branch 2 times, most recently from b31d2ee to 8cdf1fc Compare April 22, 2025 23:54
@jasonpcarroll
Copy link
Member

Hi @marcusb,

Sorry for the late reply. I will ping someone from the team to take a look.

Best,
Jason Carroll

@marcusb
Copy link
Author

marcusb commented Apr 23, 2025

I just got the unit tests running locally and figured out why they're failing, so I think I can fix it.

@AniruddhaKanhere
Copy link
Member

@marcusb let us know if you need any help. Thanks

The code would fail to compile unless one of ipconfigUSE_DNS_CACHE,
ipconfigUSE_MDNS, ipconfigUSE_LLMNR, and ipconfigDNS_USE_CALLBACKS
were set.
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