Post
Hey, do you know CMake? Can you spare a couple of minutes to do a code review? The Inkscape project sometimes struggles to find experts in some tools to do reviews and you could be a great help just looking over this include construct for unit tests (we're trying to do more unit testing)
Q: Is this ok to include a CMakeLists when putting together a list of cpp files for the unit test builder?
@doctormo another way to do this is to refer not to the set of source files but to the target those files are built under and then to extract the sources from that. it's not clear if you're trying to do something very special since i know testing can involve some metaprogramming
I don't believe the parts of inkscape are split up into their own targets. The CMakeList of each sub directory adds everything to a single build target for the inkscape binary.
This is the first baby steps of splitting things out. Should they be their own target? (and what would that look like?)
A space for Bonfire maintainers and contributors to communicate
