Remove BYPRODUCTS from cmake custom commands

Try / Tempest

3d graphics engine

MIT License

83 stars 24 forks source link

Remove BYPRODUCTS from cmake custom commands #19

Closed ChristophHaag closed 3 years ago

ChristophHaag commented 3 years ago

fixes ninja build error

ninja: error: build.ninja:5663: multiple rules generate lib/MoltenTempest/Engine/sprv/builtin_shader.h [-w dupbuild=err]

cmake documentation sounds like BYPRODUCTS is supposed to be used for intermediate files, not final outputs. It expliclity says "Byproducts may not be listed as outputs" https://cmake.org/cmake/help/latest/policy/CMP0058.html

Try commented 3 years ago

Hi, @ChristophHaag! Thanks for the fix