On 18/01/18 15:29, Allan McRae wrote: > On 18/01/18 14:59, Allan McRae wrote: >> On 14/01/18 02:49, Andrew Gregory wrote: >>> Parsing pacman's configuration file is non-trivial and extremely >>> difficult to do correctly from scripts; even our own do it incorrectly. >>> pacman-conf is a dedicated tool specifically to allow scripts to parse >>> config files, getting the same value that pacman itself would use. >>> >>> Signed-off-by: Andrew Gregory <andrew.gregor...@gmail.com> >> >> The automake changes resulted in a big warning that is fixed by enabling >> subdir-objects in configure.ac and ignoring the .dirstamp file it creates. > > > Crap... that breaks "make distcheck". More specifically "make > distclean" as it tries to remove the generated object files for the > common files twice.
Turns out automake with this subdir-objects option (which will become default next release) is buggy as hell. Some patches have landed upstream, but we can't rely on them in the pacman codebase. Options here.... 1) host pacman-conf in the src/pacman directory 2) Use the makefile to copy the needed src/pacman files into src/util/pacman-conf/... and they get compiled twice. 3) create a small library using the needed files and static link it into pacman-conf To be clear, I only included #3 as I saw several projects do this! I see #1 as the easiest. And probably most correct - the overlap in source used here is such that they should be located in the same place. Allan
