Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replaced CMAKE_CFG_INTDIR with $<CONFIG> #687

Merged
merged 4 commits into from
Oct 16, 2023

Conversation

JDuffeyBQ
Copy link
Collaborator

  • CMAKE_CFG_INTDIR is deprecated
  • Fixes packaging with Visual Studio generator
  • Removed file(MAKE_DIRECTORY) calls that use variables with $ since it can't use generator expressions and the calls aren't required

@imikejackson imikejackson force-pushed the fix/cmake_install_configuration branch 2 times, most recently from a5e910b to 9b7db66 Compare September 14, 2023 13:06
@imikejackson imikejackson force-pushed the fix/cmake_install_configuration branch 2 times, most recently from c80eb2f to 9345841 Compare September 29, 2023 23:30
* CMAKE_CFG_INTDIR is deprecated
* Fixes packaging with Visual Studio generator
* Removed file(MAKE_DIRECTORY) calls that use variables with
$<CONFIG> since it can't use generator expressions and the calls
aren't required

Signed-off-by: Jared Duffey <[email protected]>
@imikejackson imikejackson force-pushed the fix/cmake_install_configuration branch from 9345841 to fcbd51f Compare October 13, 2023 22:17
@imikejackson imikejackson merged commit 677ddc6 into develop Oct 16, 2023
7 checks passed
@imikejackson imikejackson deleted the fix/cmake_install_configuration branch October 16, 2023 01:14
imikejackson added a commit to imikejackson/simplnx that referenced this pull request Oct 20, 2024
* Replaced CMAKE_CFG_INTDIR with $<CONFIG>
* CMAKE_CFG_INTDIR is deprecated
* Fixes packaging with Visual Studio generator
* Removed file(MAKE_DIRECTORY) calls that use variables with $<CONFIG> since it can't use generator expressions and the calls aren't required
* Use $<CONFIG> or "." depending if the generator is multi-config
* Visual Studio fixes build, install, help generation
* Single and multi-config install rule fixes

---------

Signed-off-by: Jared Duffey <[email protected]>
Signed-off-by: Michael Jackson <[email protected]>
Co-authored-by: Michael Jackson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants