Make EnergyPlus suitable as a cmake subproject #7825
Conversation
This was a regression after changing CMAKE_SOURCE_DIR to PROJECT_SOURE_DIR
|
@kbenne I am not sure why CI was having trouble with your version of this. I pulled your branch, merged in develop, and it was happy right away. If it's all the same to you I'll just merge this in and call it good. Anyone have any objections to this? It seems like a trivial beneficial change. |
Myoldmopar
added
the
Refactoring
|
Yay! @kbenne CI finally responded on all platforms to your changes, but in this branch. Maybe @nrel-bot-3 is mad at you for kicking him out of your basement? Anyway, this looks good. I'm going to merge develop locally and make sure everything is good and then this can merge. |
|
Built locally with develop pulled in, ran unit tests, CI is happy on all platforms, can't ask for much more with this type of change. All is well. Merging. Hopefully this makes @kbenne and many other people have a much happier experience using EnergyPlus "as a project". |
Pull request overview
#7616 Recreated to see if CI is happier now. Also pulled develop in and resolved conflicts.