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

Cmake include paths adds './COMPONENT' and './libraries' #319

Closed
emusgrave opened this issue Jul 25, 2019 · 6 comments · Fixed by #321
Closed

Cmake include paths adds './COMPONENT' and './libraries' #319

emusgrave opened this issue Jul 25, 2019 · 6 comments · Fixed by #321
Labels

Comments

@emusgrave
Copy link

I'm a newb with cmake, but when I installed g3log using it and then included the package in my project, I got errors related to it not finding directories of /COMPONENT and /libraries. It looks like maybe the following line in CPackLists.txt needs to be put before the INCLUDES keyword?

COMPONENT libraries)

Relevant cmake documentation:
https://cmake.org/cmake/help/v3.12/command/install.html#installing-targets

Can anyone else confirm this, or am I just missing something?

@KjellKod
Copy link
Owner

Please let us know what platform you are on and what versions you are using.

@emusgrave
Copy link
Author

emusgrave commented Jul 25, 2019

I'm installing via vcpkg, which is set to use the very latest commit of g3log (1.3.2-95, commit ref 376c417).

vcpkg has installed g3log successfully on both Windows and Clear Linux, but the issue happens when using it in another project.

Cmake is version 3.14.5 on Clear Linux, and 3.14.19050301-MSVC_2 on Windows (Visual Studio 2019).

Edit: Here is the configuration that vcpkg is doing for the install: https://github.com/microsoft/vcpkg/blob/master/ports/g3log/portfile.cmake

@KjellKod
Copy link
Owner

FYI @myd7349 can you check this out?

@myd7349
Copy link
Contributor

myd7349 commented Jul 28, 2019

@KjellKod Sorry for the delay. I will confirm this.

@myd7349
Copy link
Contributor

myd7349 commented Jul 28, 2019

Hi! @emusgrave Thanks for your report. It turns out to be caused by a mistake I made in #312 and I have just created a PR to fix it. Once merged, I will update g3log's portfile in vcpkg.

myd7349 added a commit to myd7349/vcpkg that referenced this issue Jul 28, 2019
Rastaban pushed a commit to microsoft/vcpkg that referenced this issue Jul 29, 2019
* [g3log] Fix KjellKod/g3log#319

* [g3log] Update to 2019-07-29
@myd7349
Copy link
Contributor

myd7349 commented Jul 30, 2019

Hi! @emusgrave This issue has been fixed in vcpkg too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants