Opened 12 years ago

Closed 12 years ago

#34083 closed defect (fixed)

kdenline 0.8.2.1 won't build with qt4 v4.8 (qt4-mac-devel)

Reported by: gregory.verret@… Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.0.4
Keywords: Cc: ddennedy (Dan Dennedy), michaelld (Michael Dickens), ryandesign (Ryan Carsten Schmidt)
Port: kdenlive

Description

I don't have the log, sorry but the compile error is about gluOrtho2D. This function is define in glu.h unfortunatly, glu.h has been removed from <QtOpenGL> (don't know why) in version 4.8.

I easily corrected this by changing the use of gluOrtho2D (single use in src/videoglwidget.cpp).

I replaced :

gluOrtho2D(0, width, height, 0);

by this :

glOrtho(0, width, height, 0, -1, 1);

I found out that gluOrtho2D is simply just glOrtho with near = -1 and far = 1 here http://www.opengl.org/sdk/docs/man/xhtml/gluOrtho2D.xml

kdenlive should be working fine with Qt 4.7 but the problem is, qt4-mac (4.7) is about to become 4.8.1 (and devel, Qt 5) in a few days (weeks?).

Greg

Attachments (1)

01-glu-include.diff (466 bytes) - added by ddennedy (Dan Dennedy) 12 years ago.
debian patch

Download all attachments as: .zip

Change History (8)

comment:1 Changed 12 years ago by mf2k (Frank Schima)

Cc: dan@… michaelld@… added

Attaching a patchfile from the build directory would be helpful.

Changed 12 years ago by ddennedy (Dan Dennedy)

Attachment: 01-glu-include.diff added

debian patch

comment:2 Changed 12 years ago by ddennedy (Dan Dennedy)

I have declined continuing to be the kdenlive port maintainer, which is why this port has not been upgraded to the latest release, but maybe this patch from the Debian packager is the better fix.

comment:3 Changed 12 years ago by reid@…

I'm running 10.7.4 fully updated. Problem still exists and this patch still corrects the build issue.

comment:4 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Is this still needed with 0.9.2?

comment:5 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added

Cc Me!

comment:6 Changed 12 years ago by ddennedy (Dan Dennedy)

No, this patch is not needed with the new version 0.9.2 of kdenlive.

comment:7 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.