Thanks for the cleanup, Richard. For consistency, can you please change it to:

%define _fillupdir %{_localstatedir}/adm/fillup-templates

Given it's a backwards compataible macro for old distros, I wanted to be explicit - Not that we're planning on changing _localstatedir any time soon, but if we do we could cause ourselves a mess in the future.

We're using the macro as it's written in this SR in every package across the distro - it would be more inconsistent if we did it differently in pcp.

Can you survive with it like this?