add support for interface libraries

modern-cmake / cppfront

CMake wrapper around the cppfront repository

BSD 3-Clause "New" or "Revised" License

69 stars 16 forks source link

Closed Green-Sky closed 2 weeks ago

alexreinking commented 3 months ago

Thank you for the PR! I agree there is a gap in support for interface libraries that needs to be filled.

I'm just a bit confused about the SOURCES property on INTERFACE libraries... shouldn't they always use INTERFACE_SOURCES?

Green-Sky commented 3 months ago

Did some reading in the manual and the code, as is, already sets the sources as INTERFACE_SOURCES.

The target_sources() command populates this property with values given to the PUBLIC and INTERFACE keywords.

Green-Sky commented 2 months ago

@alexreinking was there anything else?

alexreinking commented 2 weeks ago

@Green-Sky - thanks! I just merged with the latest main. Should be good to go if GCC 14 likes it.