freeglut: Use the mesa-glu package when not on Apple or Windows

[jwillikers]

The glu/system package installs the Mesa GLU system package on Linux and FreeBSD. Now that the mesa-glu package is available in Conan, use that instead.

---

• I've read the contributing guidelines.
• I've used a recent Conan client version close to the currently deployed.
• I've tried at least one configuration locally with the conan-center hook activated.

[ghost]

I detected other pull requests that are modifying freeglut/all recipe:

• freeglut: Add support for Wayland #20905

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

[AbrilRBS]

@jwillikers seems like it's not able to find the headers now :)

[jwillikers]

@jwillikers seems like it's not able to find the headers now :)

Yes, looking into it, there's a couple of problems here. That older version doesn't even look for GLU for starters, and even if it did, using the FindOpenGL CMake module doesn't provide the needed granularity to separate the GLU include directories from the generic OpenGL include directories. It might be something that could be improved in the FindOpenGL CMake module.

[jwillikers]

I've opened CMake issue #25627 to address this issue.

[jwillikers]

Looking into this further, it's failing for the CHECK_INCLUDE_FILES function in CMake, which is not what I was expecting. This looks like it will require fixes in both CMake and FreeGLUT. Interestingly, I just hit a compilation error with FreeGLUT 3.4.0 locally, so maybe the CHECK_INCLUDE_FILES issue isn't a problem in newer versions.

[jwillikers]

I'm working on fixing this in CMake and have opened the issue freeglut/freeglut#153 to get this fixed there.

[jwillikers]

Alright, fixed in CMake proper. I've opened freeglut/freeglut#154 to fix this in FreeGLUT. I think that it will be best if #20905 is merged before I attempt to add any patches to fix this here, as that will make porting the upstream patches much easier.

[AbrilRBS]

Thanks a lot @jwillikers! It's great to see :)

I'll focus on approving the PR to ease things a bit for you

[conan-center-bot]

Conan v1 pipeline ✔️

All green in build 3 (602edc99d326a170fa7588d3d11345367f938898):

• freeglut/3.2.1:
  All packages built successfully! (All logs)

• freeglut/3.4.0:
  All packages built successfully! (All logs)

• freeglut/3.2.2:
  All packages built successfully! (All logs)

---

Conan v2 pipeline ✔️

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

All green in build 3 (602edc99d326a170fa7588d3d11345367f938898):

• freeglut/3.4.0:
  All packages built successfully! (All logs)

• freeglut/3.2.2:
  All packages built successfully! (All logs)

• freeglut/3.2.1:
  All packages built successfully! (All logs)

[jwillikers]

@RubenRBS Alright, looks like everything is good to go now!

[uilianries]

LGTM