Made CMakeLists.txt CPM-compatible#254
Conversation
|
I'll fix that linting error in a sec |
|
@sdeleon28 , apologies for the delay on this one. In crunch time at current shop. I'll do my utmost to take a first pass over this tonight! |
|
Any updates here? Would love to have this merged :) |
|
I'm for it, but we had originally held off on merging this because we had come up with a different way that we wanted to structure our CMake here. In particular, I remember @JoshMarler suggesting that the root level project shouldn't add juce, that we can delegate that to whomever calls |
Implemented the fix detailed in #253
In summary this PR:
REACTJUCE_JS_LIBRARYfrom client code