Skip to content

DNSServer had some major changes, but the version was not incremented #9947

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

Closed
1 task done
TheBlackEagle198 opened this issue Jun 28, 2024 · 3 comments · Fixed by #10041
Closed
1 task done

DNSServer had some major changes, but the version was not incremented #9947

TheBlackEagle198 opened this issue Jun 28, 2024 · 3 comments · Fixed by #10041
Labels
Status: Awaiting triage Issue is waiting for triage

Comments

@TheBlackEagle198
Copy link

Board

ESP32

Device Description

ttgo-lora32-v1 with on-board tft display

Hardware Configuration

not relevant

Version

latest master (checkout manually)

IDE Name

PlatformIO

Operating System

Windows10

Flash frequency

40Mhz

PSRAM enabled

yes

Upload speed

115200

Description

Hi,
It seems like there's a commit which migrates DNSServer to a AsyncUDP implementation, however the version number of the package is not incremented: d912710.

This ended up being a problem, since the package manager from PlatformIO didn't index these changes (not sure how this works), and version 2.0.0 is still using WiFiUdp.

I came to this discovering while troubleshooting the older version of the library (I had this issue: E][WiFiUdp.cpp:185] endPacket(): could not send data: 12), and the newer version fixed it.

Sketch

#include <WiFiUdp.h>
#pragma once
#include <AsyncUDP.h>

Debug Message

.

Other Steps to Reproduce

No response

I have checked existing issues, online documentation and the Troubleshooting Guide

  • I confirm I have checked existing issues, online documentation and Troubleshooting guide.
@TheBlackEagle198 TheBlackEagle198 added the Status: Awaiting triage Issue is waiting for triage label Jun 28, 2024
@Jason2866
Copy link
Collaborator

Apart from the fact that a version change should be done, the issue you encountered was a Platformio "hickup". Since installing a new framework deletes completely the old one including all libraries. So code has changed and the stored checksums from cache does not match anymore and the code will be compiled.

@TheBlackEagle198
Copy link
Author

I agree, I just thought I'd report the version increment as I'm not sure how versioning works on these modules.

@me-no-dev
Copy link
Member

they generally do not matter, because as @Jason2866 said, the core comes already packaged with them and the correct library would be compiled. You got a problem in your build environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Awaiting triage Issue is waiting for triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants