VulkanTutorial icon indicating copy to clipboard operation
VulkanTutorial copied to clipboard

"Canonical" path to stb_image.h is stb/stb_image.h, not stb_image.h

Open amosnier opened this issue 4 years ago • 0 comments

Hi,

As others before me have said, thanks a lot for the tutorial!

My use of the word "canonical" in the subject field could be considered as exaggerated, but my arguments are the following:

  • The libstb-dev package in Debian-based distributions installs its header files under /usr/include/stb. At least, it does so on Ubuntu 21.04, which I'm using right now, and I assume it's always the case.
  • "Dumping" many header files directly under a system include directory would not seem like good practice. In fact, it seems to me that the choice of putting all stb header files under /usr/include/stb is pretty much best practice.

Solving this would of course be very easy: simply search for #include <stb_image.h> in the code base, and replace it by #include <stb/stb_image.h>.

It should not either make the stb installation from source any more difficult. A typical way to do that would be to create the directory /usr/local/include/stb and place the stb header files there.

Best regards,

Alain Mosnier

amosnier avatar Oct 24 '21 09:10 amosnier