Differential D13475
Improve blacklistd support in mail/sendmail port Authored by • lidl on Dec 13 2017, 4:47 AM. Tags None Referenced Files
Subscribers
Details Rather than hacking the support into main.c, properly construct Now requires a configuration file option, or command line option to
Diff Detail
Event TimelineComment Actions instead of hard define(`bldSOURCES') is sould be better to extend the the m4 macro in a seperate line, Comment Actions Since the blacklistd support now needs to be enabled at runtime, Adjust how blacklistd.c is added to the sendmail/Makefile.m4 file, Fixed an error where a non-blacklistd related change to Makefile.m4 Comment Actions in case blacklist is not used, the extra calls to sm_io_getinfo are not covered my the #ifdef fd = sm_io_getinfo( ... ) Is this intentional? Comment Actions Pretty much, yes. I didn't think it was worthwhile cluttering up the files with extra
Comment Actions Ok, I see where my testing lead me to believe this worked. Turns out the 'make config' Is there a way to alter the DEFAULT_OPTIONS such that they are different on Comment Actions There are no simple ways, no. If blacklistd has is not supported before 11, what you can do is: OPTIONS_EXCLUDE_FreeBSD_10= BLACKLISTD Comment Actions Is FreeBSD10 the only other OS version that is still supported for building /usr/ports Comment Actions FreeBSD10 and FreeBSD11 are supported for building /usr/ports so we can add to OPTIONS:DEFAULT and disabled it on FreeBSD10. The build on FreeBSD10 still failed: ../../sendmail/sendmail.h:60:10: fatal error: 'blacklist.h' file not found ^ 1 error generated. I had to skip the include in sendmail.h: I have a cleaned up patch in testing, but i conflicts with the recent smtputf8 commit. | ||||||||||||||||||||||||||||||||||