Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

After Commit 031bcf8 Retroarch no longer compiles on Amlogic devices #8448

Closed
shantigilbert opened this issue Mar 11, 2019 · 6 comments
Closed

Comments

@shantigilbert
Copy link
Contributor

Description

After Commit 031bcf8 Retroarch no longer compiles on Amlogic devices

Using the following params

HAVE_NETWORKING=1
HAVE_LAKKA=1
HAVE_ZARCH=0
HAVE_QT=0
HAVE_OPENGL1=0

Configure options:

--disable-vg \
--disable-sdl \
--disable-ssl \
 --enable-opengles \
 --disable-kms \
 --disable-x11 \
 --enable-mali_fbdev \
--disable-qt \
--enable-neon \
 --enable-zlib \
--enable-freetype \
 --disable-discord"

compilation fails with the following errors:

LD retroarch
obj-unix/release/gfx/video_driver.o:video_driver.c:function video_driver_find_driver: error: undefined reference to 'video_gl_core'
obj-unix/release/gfx/video_driver.o:video_driver.c:function video_driver_find_driver: error: undefined reference to 'video_gl_core'
obj-unix/release/gfx/video_driver.o:video_driver.c:function video_driver_find_driver: error: undefined reference to 'video_gl_core'
obj-unix/release/gfx/video_driver.o:video_driver.c:function video_driver_find_driver: error: undefined reference to 'video_gl_core'
obj-unix/release/gfx/video_driver.o:video_driver.c:video_drivers: error: undefined reference to 'video_gl1'
obj-unix/release/gfx/font_driver.o:font_driver.c:function font_init_first: error: undefined reference to 'gl_core_raster_font'
obj-unix/release/gfx/font_driver.o:font_driver.c:function font_init_first: error: undefined reference to 'gl_core_raster_font'
obj-unix/release/gfx/font_driver.o:font_driver.c:function font_init_first: error: undefined reference to 'gl1_raster_font'
obj-unix/release/gfx/font_driver.o:font_driver.c:function font_init_first: error: undefined reference to 'gl1_raster_font'
obj-unix/release/menu/menu_driver.o:menu_driver.c:menu_display_ctx_drivers: error: undefined reference to 'menu_display_ctx_gl1'
obj-unix/release/menu/menu_driver.o:menu_driver.c:menu_display_ctx_drivers: error: undefined reference to 'menu_display_ctx_gl_core'
collect2: error: ld returned 1 exit status
make: *** [Makefile:196: retroarch] Error 1

I also tried passing HAVE_OPENGL_CORE=0 with the same result.

Expected behavior

Compile

Actual behavior

Fails

Steps to reproduce the bug

  1. Try to compile using mali_fbdev on amlogic

Bisect Results

This started after commit 031bcf8

Commit 0ff2832 compiles and works fine.

Environment information

  • OS: CoreELEC
@shantigilbert shantigilbert changed the title After Commit 031bcf8cbcb963a2416a5fa0b5f329ac1b334958 Retroarch no longer compiles on Amlogic devices After Commit 031bcf8 Retroarch no longer compiles on Amlogic devices Mar 11, 2019
@orbea
Copy link
Contributor

orbea commented Mar 12, 2019

Probably broke here. e961dad

Try ./configure --disable-opengl1

@shantigilbert
Copy link
Contributor Author

@orbea Thanks, that along with --disable-opengl_core worked.

I guess HAVE_OPENGL1=0 and HAVE_OPENGL_CORE=0 has no effect.

Will close this issue.

@orbea
Copy link
Contributor

orbea commented Mar 12, 2019

You can use ./configure --help to see all of the supported configure arguments, its usually preferred to use these instead of make arguments.

Ideally it should work on arm without those arguments, but I haven't had a chance to try to improve the build process for the gl drivers yet.

@shantigilbert
Copy link
Contributor Author

shantigilbert commented Mar 12, 2019

Thanks, I knew about the --help but to be honest for some reason I thought that the make arguments did the same :(

@orbea
Copy link
Contributor

orbea commented Mar 12, 2019

In this case the make arguments were being overridden by config.mk which is created by configure so they didn't work.

@shantigilbert
Copy link
Contributor Author

Ahh makes sense now that you explain it!

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

No branches or pull requests

2 participants