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

[devil] build failure #18363

Closed
chaquator opened this issue Jun 10, 2021 · 3 comments · Fixed by #18384
Closed

[devil] build failure #18363

chaquator opened this issue Jun 10, 2021 · 3 comments · Fixed by #18384
Assignees
Labels
category:port-bug The issue is with a library, which is something the port should already support

Comments

@chaquator
Copy link

Host Environment

  • OS: Windows 10 x64
  • Compiler: gcc 10.2.0 mingw64

To Reproduce
Steps to reproduce the behavior:

  1. ./vcpkg install devil
  2. Try to build a project with ILUT
    Somewhere in the C file:
#include <IL/il.h>
#include <IL/ilut.h>

Somewhere in the CMake file:

find_package(DevIL REQUIRED)
target_link_libraries(main PRIVATE ${IL_LIBRARIES} PRIVATE ${ILUT_LIBRARIES})

Failure logs

PS S:\c\C\compute_test\build> make
[ 50%] Building C object CMakeFiles/compute_test.dir/main.c.obj
In file included from S:\c\C\compute_test\main.c:11:
C:/src/vcpkg/installed/x64-windows/include/IL/ilut.h:100:11: fatal error: IL/ilut_config.h: No such file or directory
  100 |  #include <IL/ilut_config.h>
      |           ^~~~~~~~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles\compute_test.dir\build.make:76: CMakeFiles/compute_test.dir/main.c.obj] Error 1
make[1]: *** [CMakeFiles\Makefile2:82: CMakeFiles/compute_test.dir/all] Error 2
make: *** [Makefile:90: all] Error 2

Additional context
This issue goes away when I copy ilut_config.h from DevIL/include/IL into vcpkg/installed/x64-windows/include/IL.
It seems like upon installing DevIL, this file fails to be copied.

@PhoebeHui PhoebeHui added the category:port-bug The issue is with a library, which is something the port should already support label Jun 10, 2021
@PhoebeHui
Copy link
Contributor

@chaquator, thanks for reporting this issue!

It looks the headers hasn't been installed, see https://github.com/DentonW/DevIL/blob/master/DevIL/src-ILUT/CMakeLists.txt#L102, could you also report an issue to Upstream?

@PhoebeHui
Copy link
Contributor

@chaquator, I have submitted a PR to fix this, btw, I have already report this issue to Upstream DentonW/DevIL#96

@chaquator
Copy link
Author

Awesome, thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants