Re: [pcp] pcp updates: container/man build fix

[Lukas Berk <lberk@xxxxxxxxxx>]

Hey,

Nathan Scott <nathans@xxxxxxxxxx> writes:
>> [...]
>>  src/include/builddefs.in |    8 +++++++-
>>  5 files changed, 23 insertions(+), 2 deletions(-)
[...]
> The added HAVE_MANPAGES there looks like a no-op, or is it needed in some
> subtle way?  (note that final && is after a variable assignment).  Is the
> intent perhaps to have a separate
>
> +    test $(HAVE_MANPAGES) || MAN_PAGES=""; \
>
> before the "for" loop?

Was essentially to skip that loop if the variable is empty, I've tweaked
it again with an 'if ... fi' conditional to make it more clear, been using
it all day to build pcp with and without containers.  Is in my master
branch upstream.

Cheers,

Lukas