On 2019-08-13, at 11:55:29 +0200, Pablo Neira Ayuso wrote: > On Wed, Aug 07, 2019 at 03:16:57PM +0100, Jeremy Sowden wrote: > > A number of netfilter header files are on the header-test blacklist > > becuse they cannot be compiled stand-alone. There are two main > > reasons for this: missing inclusions of other headers, and missing > > conditionals checking for CONFIG_* symbols. > > > > The first six of these patches rectify these omissions, the seventh > > removes some unnecessary "#ifdef __KERNEL__" checks, and the last > > removes all the NF headers from the blacklist. > > > > I've cc'ed Masahiro Yamada because the last patch removes 74 lines > > from include/Kbuild and may conflict with his kbuild tree. > > Series applied, one comment below. > > > Jeremy Sowden (8): > > netfilter: inlined four headers files into another one. > > netfilter: added missing includes to a number of header-files. > > netfilter: added missing IS_ENABLED(CONFIG_BRIDGE_NETFILTER) checks to > > header-file. > > netfilter: added missing IS_ENABLED(CONFIG_NF_TABLES) check to > > header-file. > > netfilter: added missing IS_ENABLED(CONFIG_NF_CONNTRACK) checks to > > some header-files. > > netfilter: added missing IS_ENABLED(CONFIG_NETFILTER) checks to some > > header-files. > > netfilter: removed "#ifdef __KERNEL__" guards from some headers. > > kbuild: removed all netfilter headers from header-test blacklist. > > Would you mind if - before pushing this out - I do this string > replacement for the patch subject? > > s/added/add > s/removed/remove > s/inlined/inline > > I was told present tense is preferred for description. Otherwise, I'll > leave them as is. I adopted past tenses because at the point at which one is reading the description of a commit, one is usually reading about old behaviour and what has been done to change it. However, I wasn't aware that there was a preference and I am happy to switch to the present tense instead, so by all means feel free to change them. J.
Attachment:
signature.asc
Description: PGP signature
