trusteddomainproject / OpenDKIM

Other
97 stars 52 forks source link

Delete dkim_res_nslist(), rbl_res_nslist() #99

Open rhansen opened 4 years ago

rhansen commented 4 years ago

These functions are useless: If HAVE_RES_SETSERVERS is defined they don't compile due to a handful of obvious bugs. If HAVE_RES_SETSERVERS is not defined they are no-ops. For over 6 years HAVE_RES_SETSERVERS was never defined by configure.ac due to a silly typo. Because nobody seemed to notice that they were no-ops, I doubt anyone will miss them when they're gone.

mskucherawy commented 4 years ago

LGTM; are we going to add the RELEASE_NOTES text in here or in a later pull request?

rhansen commented 4 years ago

I added a couple of lines to RELEASE_NOTES under 2.11.0.

rhansen commented 3 years ago

Friendly ping.

rhansen commented 3 years ago

Friendly ping @mskucherawy