Skip to content
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

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

miodvallat
Copy link
Contributor

Short description

As suggested in #12953, there is need to support libdecaf any longer.

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled this code
  • tested this code (quickly)
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)

@coveralls
Copy link

coveralls commented Dec 4, 2024

Pull Request Test Coverage Report for Build 12182335989

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • 42 unchanged lines in 11 files lost coverage.
  • Overall coverage increased (+0.3%) to 64.712%

Files with Coverage Reduction New Missed Lines %
pdns/validate.cc 1 68.14%
pdns/dnssecinfra.cc 2 76.3%
pdns/recursordist/sortlist.cc 2 72.94%
pdns/sstuff.hh 2 56.83%
modules/lmdbbackend/lmdbbackend.cc 2 73.83%
pdns/recursordist/test-syncres_cc2.cc 3 88.85%
pdns/stubresolver.cc 3 77.58%
pdns/recursordist/rec-main.cc 3 62.4%
pdns/shuffle.cc 4 39.71%
pdns/recursordist/syncres.cc 10 79.34%
Totals Coverage Status
Change from base Build 12178254343: 0.3%
Covered Lines: 125741
Relevant Lines: 163437

💛 - Coveralls

@omoerbeek
Copy link
Member

Recursor build failure due to missed auto* case: https://github.com/PowerDNS/pdns/actions/runs/12156498965/job/33900317485?pr=14926

@miodvallat
Copy link
Contributor Author

I suppose I need to keep an empty m4/pdns_with_libdecaf.m4 for autoreconf to operate correctly.

@omoerbeek
Copy link
Member

Hmm, that should not be needed, I'll take a look why autoconf wants that file later today.

@miodvallat
Copy link
Contributor Author

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.

@omoerbeek
Copy link
Member

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.

@omoerbeek
Copy link
Member

Of course the issue doesn't reproduce locally. I avoid gaining too much auto* knowledge, so the proposed workaround it is.

@miodvallat
Copy link
Contributor Author

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.

@Habbie
Copy link
Member

Habbie commented Dec 5, 2024

Recursor build failure due to missed auto* case: https://github.com/PowerDNS/pdns/actions/runs/12156498965/job/33900317485?pr=14926

this is -so weird-

@omoerbeek
Copy link
Member

Recursor build failure due to missed auto* case: https://github.com/PowerDNS/pdns/actions/runs/12156498965/job/33900317485?pr=14926

this is -so weird-

caching is indeed hard... does the weirdness prevent merging?

@Habbie
Copy link
Member

Habbie commented Dec 5, 2024

I figured it out. The right fix is git rm pdns/recursordist/m4/pdns_with_libdecaf.m4, I am quite sure :)

@Habbie
Copy link
Member

Habbie commented Dec 5, 2024

(no caching was involved at all)

@miodvallat
Copy link
Contributor Author

Ah, I missed it due it being a symbolic link. Good catch!

@miodvallat
Copy link
Contributor Author

(no caching was involved at all)

It was DNS.

@omoerbeek
Copy link
Member

I suspect I did my test with the workaround already in and didn't pay attention.

@Habbie Habbie merged commit 583c7b2 into PowerDNS:master Dec 6, 2024
77 checks passed
@miodvallat miodvallat deleted the feature/12953 branch December 6, 2024 09:15
omoerbeek added a commit to omoerbeek/pdns that referenced this pull request Dec 20, 2024
omoerbeek added a commit that referenced this pull request Dec 20, 2024
rec: zap dangling symlink, leftover of #14926
omoerbeek added a commit to omoerbeek/pdns that referenced this pull request Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants