search

troglobit / mdnsd

Jeremie Miller's original mdnsd
BSD 3-Clause "New" or "Revised" License
57 stars 35 forks source link

Potential fork with many build improvements #40

Closed prince-chrismc closed 4 years ago

prince-chrismc commented 4 years ago

I came across a fork of this repository while searching for MDNS-DS solutions...

I wanted to share it's advances since it provides many of the features/compatibility improvements many could benefit from.

Differences to the base repo (https://github.com/troglobit/mdnsd):

  • Use CMake for the build
  • Strict compilation flags for better portability
  • Support of Linux, MinGW, OS X and Windows
  • Continuous Integration
troglobit commented 4 years ago

Hi Chris. I'm aware of Stefan's fork, and I've tried several times in the past to merge his changes, but as you can see in your pull request we've diverged a lot and it would be a monumental task to stitch it all back together. Most of the relevant changes have been merged though, and this project has also added a lot of features that Stefan's fork doesn't have, e.g., support for /etc/mdns.d/foo.service records for admins to control what services are published.

Also, I don't consider it to be an improvement to use CMake. That is not my tribe at all. As for the other advances you mention, have you tried building this project on MinGW, or mac OS? We also have CI setup, and we use the static code analysis tool Coverity. Recently a German team did extensive fuzzing of the interfaces and fixed a lot of nasty holes.