Refuse to build if there are no backends for DNSSD at all. The messages for the OPTIONAL backeds mention that one is required, but it wasn't enforced. KDE CI has been building with no backend configured for some time.
Details
- Reviewers
- None
Configure without as backend:
cmake -DCMAKE_DISABLE_FIND_PACKAGE_Avahi=1 \ -DCMAKE_DISABLE_FIND_PACKAGE_DNSSD=1
note that the backend requirements are optional, but one is required.
Diff Detail
- Repository
- R272 KDNSSD
- Branch
- require-backend
- Lint
No Linters Available - Unit
No Unit Test Coverage
Something to consider, though, is that it might perhaps be useful to have kdnssd build -- without a backend -- for environments where there is no real dnssd and any service discovery returns "not found". Requiring a backend makes that impossible.
(So something else to do is to install avahi on the openSUSE and other Linux KDE CI machines)
CMakeLists.txt | ||
---|---|---|
67–68 | wrong place |
CMakeLists.txt | ||
---|---|---|
67–68 | Hi. If you're posting a review, please provide constructive input. If there are policies or guidelines about places, please link to them. If there aren't, but you would prefer a change, please describe the (outline of the) change. If you mean "there should be an additional space in the else() statement to match the text formatting from line 57", then write that. |
CMakeLists.txt | ||
---|---|---|
67–68 | your else() follows if (IS_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/po"), I thought the comment was clear. Your current patch means: if there's no "po" subdir, throw a fatal error with an unrelated message. |
Use the right place (sorry Christophe, to use a Dutch phrase:
I must have been looking with my nose).
I'm with you that having a possibility to just compile kdnssd without any backend may be of value so i would either not commit this or improve the cmake so there's a way to say I_KNOW_WHAT_I_AM_DOING_KEEP_GOING