-
Notifications
You must be signed in to change notification settings - Fork 921
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
[feature] remove support for libdecaf #14926
Conversation
Pull Request Test Coverage Report for Build 12182335989Details
💛 - Coveralls |
Recursor build failure due to missed auto* case: https://github.com/PowerDNS/pdns/actions/runs/12156498965/job/33900317485?pr=14926 |
I suppose I need to keep an empty |
Hmm, that should not be needed, I'll take a look why autoconf wants that file later today. |
I agree that this shouldn't be needed, but if keeping a mostly empty file helps, then it doesn't hurt to carry that empty file for a while. |
Sure, if I don't find the solution quickly, an empty file it is. |
Of course the issue doesn't reproduce locally. I avoid gaining too much auto* knowledge, so the proposed workaround it is. |
Of course! You can't have both auto* knowledge and otto* knowledge. |
this is -so weird- |
caching is indeed hard... does the weirdness prevent merging? |
I figured it out. The right fix is |
(no caching was involved at all) |
Ah, I missed it due it being a symbolic link. Good catch! |
64eefc6
to
bc37490
Compare
It was DNS. |
I suspect I did my test with the workaround already in and didn't pay attention. |
rec: zap dangling symlink, leftover of #14926
(cherry picked from commit 472b550)
Short description
As suggested in #12953, there is need to support libdecaf any longer.
Checklist
I have: