On 2022-08-04, at 13:45:04 +1000, Duncan Roe wrote: > On Wed, Aug 03, 2022 at 09:12:41PM +0100, Jeremy Sowden wrote: > > These changes were prompted by Mark Mentovai's request to remove a > > hard-coded `/bin/bash` path from the rule that creates the man-page > > sym-links. Hitherto, the doxygen Makefile has jumped through a > > number of hoops to make sure everything works with `make distcheck` > > and parallel builds. This patch-set makes some doxygen config > > changes that obviate the need for them, fixes a bug in `make clean`, > > updates .gitignore and moves the shell-script out of the Makefile > > into a separate file for ease of maintenance. In the process, the > > hard-coded `/bin/bash` is removed. > > > > One thing I have left is the setting of `-p` when running the > > shell-script. The comment reads "`bash -p` prevents import of > > functions from the environment". Why is this a problem? > > > > Jeremy Sowden (6): > > build: add `make dist` tar-balls to .gitignore > > doc: add .gitignore for Doxygen artefacts > > doc: change `INPUT` doxygen setting to `@top_srcdir@` > > doc: move doxygen config file into doxygen directory > > doc: move man-page sym-link shell-script into a separate file > > doc: fix doxygen `clean-local` rule > > > > .gitignore | 3 +- > > configure.ac | 15 ++++++- > > doxygen/.gitignore | 4 ++ > > doxygen/Makefile.am | 53 +++--------------------- > > doxygen.cfg.in => doxygen/doxygen.cfg.in | 4 +- > > doxygen/finalize_manpages.sh | 40 ++++++++++++++++++ > > 6 files changed, 67 insertions(+), 52 deletions(-) > > create mode 100644 doxygen/.gitignore > > rename doxygen.cfg.in => doxygen/doxygen.cfg.in (91%) > > create mode 100644 doxygen/finalize_manpages.sh > > > > -- > > 2.35.1 > > Thanks for this but ... I think it would be better to use the > netfilter-log model. > > That way there is no new shell script to maintain - libmnl would pick > up libnetfilter-queue's copy. > > I found the time-consuming part was checking the new SYNOPSIS lines to > be sufficient. I have far less time nowadays (no Covid lockdowns, &c.) > but I will try to get it done soon-ish. > > In the meantime I think Mark's 1-line patch to the existing mess is > adequate. Cool. J.
Attachment:
signature.asc
Description: PGP signature
