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.

Sign up for GitHub

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

Jump to bottom

Reforms the library target definitions in the main CMakeLists #27

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Reforms the library target definitions in the main CMakeLists #27

wants to merge 2 commits into from

Conversation

salehjg
Copy link

@salehjg salehjg commented Jun 22, 2023

Reforms the library target definitions in the main CMakeLists to:

  • Allow the project to be used without any hassle of including the published header files, utilizing the modern CMake capabilities.
  • The dependent project will not be needing to add anything to include_directories(); linking the dependent project against cppparser would do the trick.

@salehjg
Added cmake-build-* and .idea to .gitignore
ee0dc78
@salehjg
Changed the main CMakeLists and the way cppparser library is defined.
b02c0b8 
With the new approach the published head files of the library will be automatically provided to any other project that depends on `cppparser`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@salehjg