dnsdist: Remove autotools#17010
Conversation
737255a to
2092b6e
Compare
This should fix the Fuzz errors in PowerDNS/pdns#17010
This should fix the Fuzz errors in PowerDNS/pdns#17010
|
From The |
|
I wonder why the codeql one does not have that and it does work for the autotools build. |
48753e6 to
8f42213
Compare
This should fix the Fuzz errors in PowerDNS/pdns#17010
This should fix the Fuzz errors in PowerDNS/pdns#17010 --------- Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com> Co-authored-by: Remi Gacogne <remi.gacogne@powerdns.com>
d850bf5 to
3279417
Compare
|
Ah, looks like we might have to add python-yaml to the oss-fuzz environment, or perhaps disable YAML support there? |
I think we should keep yaml in the fuzz build, should we ever want to fuzz the parser? |
|
That requires building the Rust-based static library, so we need a Rust compiler. So perhaps it makes sense to not do that for now, and cross that bridge if we decide to fuzz serde (which would probably not happen in dnsdist anyway)? |
|
you're right, let's disable it. |
3279417 to
241de62
Compare
b6b6aec to
74cfcd1
Compare
|
google/oss-fuzz#15546 will help with the fuzzing issues, and I opened pieterlexis#18 to simplify a bit the handling of dnsdist's fuzzing targets. |
5d24c2a to
c93958e
Compare
8bfe756 to
b2c008c
Compare
Coverage Report for CI Build 26573471660Coverage increased (+4.0%) to 71.023%Details
Uncovered ChangesNo uncovered changes found. Coverage Regressions104 previously-covered lines in 15 files lost coverage.
Coverage Stats💛 - Coveralls |
4c157c5 to
15de42c
Compare
Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>
Catch2 >= 3 is not available in 22.04.
690cd49 to
5f26cd7
Compare
Short description
As part of removing some technical debt, this PR removes the possibility to build dnsdist using autotools.
Draft PR to iron out all the CI issues.
Checklist
I have: