HaikuPorts
  • Login
  • Preferences
  • Help/Guide
  • Wiki
  • Timeline
  • Roadmap
  • View Tickets
  • Search
  • Port Log
  • Blog

Context Navigation

  • ← Previous Ticket
  • Back to Query
  • Next Ticket →

Ticket #399 (closed enhancement: fixed)

Opened 5 years ago

Last modified 4 years ago

[cmake] add OpenGL headers and libs path; replace lua math lib

Reported by: michaelvoliveira Owned by: scottmc
Priority: normal Milestone: Haiku Optional Packages
Component: dev-util/cmake Version: 1.0
Severity: major Keywords: cmake opengl lua
Cc:

Description

Duggan is doing a port of Ogre3D for Haiku. It uses cmake as build system but our cmake port not recognize the GL headers nor GL libs.

On the other hand, for Wesnoth port, cmake don't find lua math library (libm), of course.
So I use the same lua for it.. and all works fine.

Attachments

cmake-2.8.1.2.patch​ (2.1 KB) - added by michaelvoliveira 5 years ago.
augiedoggie
cmake-2.8.1.patch​ (3.6 KB) - added by michaelvoliveira 5 years ago.
recreated with augiedoggie suggestions, for GL, GLUT and LUA

Download all attachments as: .zip

Change History

comment:1 Changed 5 years ago by augiedoggie

This patch is not acceptable.

For one thing it will break the FindLua51.cmake package for everyone besides Haiku. You need to add a check like 'AND NOT BEOS' to the 'IF' statement.

In FindOpenGL.cmake it would probably be better to use the gcc specific library path '/boot/develop/lib/x86/'

I'ts not a good idea to remove the UNIX definition in the Haiku.cmake module. Besides the fact that it's the wrong way to do it, some ports are already using this. Despite what some would like to believe, Haiku is pretty close to being UNIX as far as cmake is concerned.

Changed 5 years ago by michaelvoliveira

  • attachment cmake-2.8.1.2.patch​ added

augiedoggie

Changed 5 years ago by michaelvoliveira

  • attachment cmake-2.8.1.patch​ added

recreated with augiedoggie suggestions, for GL, GLUT and LUA

comment:2 Changed 4 years ago by scottmc

  • Resolution set to fixed
  • Status changed from new to closed

Applied in r1333.

Note: See TracTickets for help on using tickets.

Download in other formats:

  • Comma-delimited Text
  • Tab-delimited Text
  • RSS Feed

Trac Powered

Powered by Trac 0.13dev-r10686
By Edgewall Software.

Visit the Trac open source project at
http://trac.edgewall.org/