Arch Linux

I think ParaView needs the headers to be available (at least it embeds them from their ThirdParty vendored sources if you don’t add pegtl as a system dependency).

Yes, and it needs these only at build time not run time, so pegtl should be moved to makedepends.

By embed, I meant in the actual package/during `ninja install`. But that’s only from memory.

Anyway, I can remove the dep and we will see if anything happens.

DONT remove it, just list it as makedepends.

I'm sure the ninja install does install the headers, but this is cmake, so it would always install the headers as long as the pegtl CMakeLists.txt in the thirdparty directory says to.

This is because add_subdirectory reads the CMakeLists.txt with full, global context and inherits all install() calls. cmake fundamentally does not have sane handling for vendoring dependencies, so it's a good thing we remove them anyway...

EDIT: to be clear, I'm responding to Bruno, with an explanation for why looking at ninja install doesn't say anything about whether it is needed at runtime.

I'm not discussing whether paraview installs the pegtl headers.

I'm just saying that the pegtl package that is listed as dependency in the PKGBUILD should be listed as makedepends as it's only needed when building the package, not when running paraview.