Skip to content
Peter van Dijk edited this page Nov 30, 2018 · 46 revisions

With reference to https://en.wiktionary.org/wiki/yak_shaving, sometimes somebody will run into something that 'could be nicer' but does not warrant an issue ticket. Instead, we note those things here. Pull requests fixing these things are welcomed! Feel free to add to this list.

The yaks are numbered so a PR can easily reference them.. Please use ~bla bla~ to strike out finished items, adding 'PR #1234after the closing~`.

  • (auth at least) ./configure --help has alignment issues PR #7005
  • (all products) 15:10Z <lieter> another yak I have with confige is our inconsistent use of --enable-X and --with-X 15:11Z <lieter> --enable are _our_ programs's options and --with-X should be external deps PR #7026
  • (auth at least) README has instructions for Debian 8.0, should not say '.0' and should be updated for 9 PR #7027
  • recursor and dnsdist rely on symlinks into the 'main' pdns source tree. Travis should check for dangling symlinks. PR #7028
  • figure out Travis/python/SSL warnings and do something about them PR #7106
  • generally improve quality of Travis logging output
  • in 49f72da134c234603abc5d270f73ea6191dceeff, we imported timegm from dietlibc, presumably(!) because Solaris does not have it. Figure out if that is still relevant today, otherwise, get rid of it.
  • in https://dnsdist.org/reference/config.html?highlight=setup, there is a visible :ref: (to "Healthcheck") that should be a link
  • document that dnsdist newServer() returns the server object
  • parallelize Travis builds or at least check if this is still applicable (also see here)
  • check whether _blank is an issue in our code (targetblank)
  • LGTM alerts
  • Make sure LGTM only builds the right products on the right branches (i.e. no auth builds on a rec branch)
  • We use unsigned int and other int types without size identifiers. It would be nice to switch to properly typed integers (like uint32_t).
  • Remove all references to GOST (some already done, some in #7187, more to do)
  • pdns-4.1.5.tar.gz accidentally contains regression-tests/zones/unit.test, perhaps other undesired files as well. Figure out a way to notice this (a list in git?)
  • some of the Lua examples in the docs, and example *.lua files in the tree, use print(), they should use pdnslog