Skip to content

Ci2#68

Open
dk wants to merge 3 commits intoPerl-GPU:masterfrom
dk:ci2
Open

Ci2#68
dk wants to merge 3 commits intoPerl-GPU:masterfrom
dk:ci2

Conversation

@dk
Copy link
Copy Markdown
Contributor

@dk dk commented Mar 1, 2026

Use OGLM_APPLE_GLX for glew-supported XQuartz/GLX GLEW_APPLE_GLX backend.
Now with CI.

dk added 2 commits February 23, 2026 14:43
As agreed in Perl-GPU#66, here's
the next iteration that would allow linking the module with XQuartz libraries.

The intended use is this:

  env:
      OGLM_ADD_DEFINE: '-DGLEW_APPLE_GLX -I/opt/X11/include'
      OGLM_ADD_LIBS:   '-L/opt/X11/lib -lglut'
      OGLM_REPLACE_OTHERLDFLAGS: ''
@dk
Copy link
Copy Markdown
Contributor Author

dk commented Mar 6, 2026

@mohawk2 kindly let me know if the CI is as you imagined it, - this is my best understanding of your requirements

@mohawk2
Copy link
Copy Markdown
Contributor

mohawk2 commented May 2, 2026

On the face of it, this does have CI and passes the tests, thank you.

But I'm having a bit of a problem understanding why OGLM would actually need or even be affected by having the -lglut, since I'm not aware of that providing any symbols this distro would use, nor anything to do with X11. Can you spell that out for me? This thing is supposed to do OpenGL stuff with an already-created context.

@mohawk2
Copy link
Copy Markdown
Contributor

mohawk2 commented May 2, 2026

Having done a bit of googling, this (https://stackoverflow.com/questions/27829330/glew-apple-glx-causes-linking-error-when-building-glew) makes it look like it is indeed needed, which is terrible but here we are. Also it looks like what's needed is -lGL, not -lglut. Do you have any info on that?

I think a synthesis of the approaches here (adding CI) and the one in #67 (generalised env variables to add/replace flags), together with adding installation notes documenting those env variables, is the right way forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants