Skip to content
This repository has been archived by the owner on Apr 17, 2022. It is now read-only.

fails to build with GLEW 1.12.0 #4300

Closed
wzdev-ci opened this issue Jul 1, 2015 · 9 comments
Closed

fails to build with GLEW 1.12.0 #4300

wzdev-ci opened this issue Jul 1, 2015 · 9 comments

Comments

@wzdev-ci
Copy link
Contributor

wzdev-ci commented Jul 1, 2015

resolution_duplicate type_bug | by pabs


warzone2100 fails to build with GLEW 1.12.0, it seems like the GLEW API has changed:

g++ -DHAVE_CONFIG_H -I. -I../..  -I/usr/include/libpng12    -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -DNDEBUG -DWZ_DATADIR="\"/usr/share/games/warzone2100\"" -DLOCALEDIR="\"/usr/share/locale\"" -I../.. -I/usr/include/libdrm  -DQT_SHARED -I/usr/include/qt4 -I/usr/include/qt4/QtNetwork -I/usr/include/qt4 -I/usr/include/qt4/QtScript -I/usr/include/qt4 -I/usr/include/qt4/QtCore  -D_FORTIFY_SOURCE=2 -g -Wno-enum-compare -Wall -Wextra -Wno-unused-parameter -Wno-sign-compare -Wcast-align -Wwrite-strings -Wpointer-arith -Wno-format-security -DQT_SHARED -I/usr/include/qt4 -I/usr/include/qt4/QtNetwork -I/usr/include/qt4 -I/usr/include/qt4/QtScript -I/usr/include/qt4 -I/usr/include/qt4/QtCore  -g -O2 -fPIE -fstack-protector-strong -Wformat -Werror=format-security -fstack-protector -c -o screen.o screen.cpp
screen.cpp: In function 'void screen_EnableMissingFunctions()':
screen.cpp:264:22: error: invalid conversion from 'PFNGLSHADERSOURCEARBPROC {aka void (*)(unsigned int, int, const char**, const int*)}' to 'PFNGLSHADERSOURCEPROC {aka void (*)(unsigned int, int, const char* const*, const int*)}' [-fpermissive]
   __glewShaderSource = __glewShaderSourceARB;
                      ^
Makefile:423: recipe for target 'screen.o' failed

Issue migrated from trac:4300 at 2022-04-16 11:50:48 -0700

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jul 1, 2015

Per changed blocking which not transferred by tractive

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jul 1, 2015

Per changed blockedby which not transferred by tractive

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jul 1, 2015

Per commented


This bug appears inside a hideous hack to support pre-OpenGL 2.0 systems, which presumes too much about GLEW internals. I would not mind ripping it out. Or if Debian ripped it out before distributing - just remove the whole function and one place where it is called.

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jul 1, 2015

Per commented


This code is all gone in master, BTW.

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jul 1, 2015

Per Inge Mathisen <per.mathisen@...> commented


In Warzone2100/warzone2100@d512d1d:

#CommitTicketReference repository="" revision="d512d1dc9857e3c2bbfa0a7b5856bdf420412b8f"
3.1: Remove hack to support some pre-OpenGL-2.0 systems that made
assumptions about internals in GLEW that breaks with GLEW 1.12.

See #4300 reported by pabs

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jul 1, 2015

vexed changed status from new to closed

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jul 1, 2015

vexed changed resolution from `` to duplicate

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jul 1, 2015

vexed commented


Actually, this was already reported.
duplicate of #4218

@wzdev-ci wzdev-ci closed this as completed Jul 1, 2015
@wzdev-ci
Copy link
Contributor Author

vexed committed [6]


In Warzone2100/warzone2100@6a6c611:

#CommitTicketReference repository="" revision="6a6c611ca3d9d6745e5343f2ee041892772ccae6"
Resync/backport the code base:

3.1: Remove hack to support some pre-OpenGL-2.0 systems that made
assumptions about internals in GLEW that breaks with GLEW 1.12.

See #4300 reported by pabs

(cherry picked from commit d512d1dc9857e3c2bbfa0a7b5856bdf420412b8f)

Conflicts:
	lib/ivis_opengl/screen.cpp

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant