Some comments on your patches:
v2-0001-meson-Change-default-of-selinux-feature-option-to.patch
Ok
v2-0002-docs-Document-with-selinux-Dselinux-options-centr.patch
Ok, but "selinux" should be "SELinux" when referring to the product.
v2-0003-meson-docs-Add-doc-html-man-targets.patch
We have make targets "html" and "man", so I suggest we make the meson
targets the same.
v2-0004-meson-Add-world-target.patch
AFAICT, this world target doesn't include the man target. (Again, this
would all work better if we added "man" to "docs".)
v2-0005-docs-meson-Add-documentation-for-important-build-.patch
It's nice to document this, but it's weird that we only document the
meson targets, not the make targets.
v2-0006-meson-Add-help-target-build-docs-from-a-common-so.patch
Here also, making this consistent and uniform with make would be useful.
v2-0007-meson-Add-Dpkglibdir-option.patch
Normally, the pkgFOOdir variables are just FOOdir plus package name. I
don't feel comfortable allowing those to be separately set. We don't
allow that with configure; this just arose from a Debian patch.
The description "location to dynamically loadable modules" is too
narrow. Consider for example, another proposed patch, where we are
doing some preprocessing on postgres.bki at build time. Since that
makes postgres.bki platform dependent, it should really be moved from
share (datadir in configure parlance) to pkglibdir. So then we have
things in there that are not loadable modules. I don't know how that
affects Debian packaging, but this patch might not be the right one.
I suggest we leave this patch for a separate discussion.
