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

Minetest segfaults on start if png and xml fonts files are removed from the fonts folder. #2551

Closed
Megaf opened this issue Mar 24, 2015 · 6 comments
Labels
Bug Issues that were confirmed to be a bug @ Client / Audiovisuals Low priority

Comments

@Megaf
Copy link
Contributor

Megaf commented Mar 24, 2015

When removing the png and xml files from the fonts folder and compiling with these settings, -DRUN_IN_PLACE=0 -DBUILD_SERVER=0 -DUSE_FREETYPE=0 -DENABLE_FREETYPE=0, Minetest will segfault on start.

reglnx@ToshSAM:~/Games/MinetestFontsRedux$ bin/minetest 
WARNING: system-wide share not found at "/usr/local/share/minetest"
WARNING: system-wide share not found at "/home/reglnx/Games/MinetestFontsRedux/bin/../share/minetest"
WARNING: system-wide share found at "/home/reglnx/Games/MinetestFontsRedux/bin/.."
2015-03-24 17:37:25: ERROR[main]: Subgame specified in default_game [minetest] is invalid.
Irrlicht log: Irrlicht Engine version 1.8.1
Irrlicht log: Linux 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt7-1 (2015-03-01) x86_64
Irrlicht log: Using renderer: OpenGL 3.0
Irrlicht log: Gallium 0.4 on AMD MULLINS: X.Org
Irrlicht log: OpenGL driver version is 1.2 or better.
Irrlicht log: GLSL version: 1.3
2015-03-24 17:37:25: ERROR[main]: FontEngine: Default font file: 
2015-03-24 17:37:25: ERROR[main]:   "/home/reglnx/Games/MinetestFontsRedux/bin/../fonts/lucida_sans"
2015-03-24 17:37:25: ERROR[main]:   required for current screen configuration was not found or was invalid file format.
2015-03-24 17:37:25: ERROR[main]:   Using irrlicht default font.
Segmentation fault

It should still work, but with Irrlicht's default font.

If I compile it with -DRUN_IN_PLACE=0 -DBUILD_SERVER=0 -DUSE_FREETYPE=1 -DENABLE_FREETYPE=1 it will work just as it supposed to.

reglnx@ToshSAM:~/Games/MinetestFontsRedux$ bin/minetest 
WARNING: system-wide share not found at "/usr/local/share/minetest"
WARNING: system-wide share not found at "/home/reglnx/Games/MinetestFontsRedux/bin/../share/minetest"
WARNING: system-wide share found at "/home/reglnx/Games/MinetestFontsRedux/bin/.."
2015-03-24 17:44:43: ERROR[main]: Subgame specified in default_game [minetest] is invalid.
Irrlicht log: Irrlicht Engine version 1.8.1
Irrlicht log: Linux 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt7-1 (2015-03-01) x86_64
Irrlicht log: Using renderer: OpenGL 3.0
Irrlicht log: Gallium 0.4 on AMD MULLINS: X.Org
Irrlicht log: OpenGL driver version is 1.2 or better.
Irrlicht log: GLSL version: 1.3
2015-03-24 17:44:48: ERROR[main]: Provided world path doesn't exist: /home/reglnx/.minetest/worlds/world

To reproduce, just rm minetest/fonts/*xml and rm minetest/fonts/*png

@ShadowNinja ShadowNinja added Bug Issues that were confirmed to be a bug @ Client / Audiovisuals Low priority labels Oct 3, 2015
@ShadowNinja
Copy link
Member

This fixes it, but the resulting font doesn't scale: http://ix.io/laj

@paramat
Copy link
Contributor

paramat commented Mar 31, 2016

If you're removing necessary files, is it a bug? Can this be closed?

@rubenwardy
Copy link
Member

It should fail gracefully with an error, or use irrlicht's inbuilt default font

@ShadowNinja
Copy link
Member

They're necessary now, but they don't really have to be necessary -- all it takes is that one line change to make them optional.

@ShadowNinja
Copy link
Member

Oops.

@Megaf
Copy link
Contributor Author

Megaf commented Apr 2, 2016

This was fixed long ago I think. Mt will no longer cras IF it is compiled with freetype enabled.

@Megaf Megaf closed this as completed Apr 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues that were confirmed to be a bug @ Client / Audiovisuals Low priority
Projects
None yet
Development

No branches or pull requests

4 participants