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

knot-resolver: fix make config #24060

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

facboy
Copy link
Contributor

@facboy facboy commented May 1, 2024

  • Fix errors in make config preventing e.g. menuconfig from working, and knot-resolver from being included in builds

Maintainer: @miska

Build system: x86/64
Compile tested: rockchip/armv8, nanopi-r4s, be37ab6 snapshot build
Run tested: rockchip/armv8, nanopi-r4s, be37ab6 snapshot build, running on my main router

@facboy
Copy link
Contributor Author

facboy commented May 1, 2024

i forgot this required protobuf-c-compiler to be installed on Ubuntu, because it fixes the 'dnstap enabled by default' config so that it actually builds the dnstap module.

@facboy
Copy link
Contributor Author

facboy commented May 5, 2024

@autobakterie the build docker image doesn't seem to have protobuf-c-compiler installed by default, would it be simplest just to revert this to disable dnstap by default?

@neheb
Copy link
Contributor

neheb commented Jun 23, 2024

PKG_RELEASE bump is needed.

@facboy facboy force-pushed the knot-resolver branch 2 times, most recently from d508699 to 0ee46c7 Compare June 24, 2024 08:16
@facboy
Copy link
Contributor Author

facboy commented Jun 24, 2024

Thx fixed.

* Fix errors in make config preventing e.g. `menuconfig` from working,
  and `knot-resolver` from being included in builds
* Disable `dnstap` by default as the docker build images do not have
  `protobuf-c-compiler` installed, so the build fails

Signed-off-by: Christopher Ng <[email protected]>
Signed-off-by: Christopher Ng <[email protected]>
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.

2 participants