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

Building error: initializer-string for 'char [32]' is too long [-fpermissive] #1591

Closed
Kelvfimer opened this issue Nov 10, 2023 · 6 comments
Closed

Comments

@Kelvfimer
Copy link

Hello

Starting commit f179eb0 compilation reports error

/build/fbneoSA-7200efefe322f153be2e1164be50323db6d5c720/src/dep/generated/driverlist.h:59991:1: error: initializer-string for 'char [32]' is too long [-fpermissive]
make[1]: *** [makefile.sdl2:562: burn.o] Error 1
make: *** [makefile:113: sdl2] Error 2

it was expected to be fixed on 870438d

however it is still failing on latest commit 7200efe

Platform odroid n2+ CMAKE 3.24.2 gcc 12 Linux 4.9.269 #1 SMP PREEMPT Thu Nov 9 06:40:40 CET 2023 aarch64 GNU/Linux

make flags sdl2 RELEASEBUILD=1

Pls can you help me?

thx
KR

@barbudreadmon
Copy link
Collaborator

Hi,

please post your driverlist.h

@Kelvfimer
Copy link
Author

Hello @barbudreadmon here it is.

thx for your support
driverlist.zip

@Kelvfimer
Copy link
Author

BTW I see changing

struct game_sourcefile_entry {
char game_name[256];
char sourcefile[256];
};

in /src/dep/scripts/gamelist.pl it fixes the issue.

@barbudreadmon

@barbudreadmon
Copy link
Collaborator

changing the char length is beyond the point, the problem here is that your sourcefile_table is storing crap, i.e

static game_sourcefile_entry sourcefile_table[] = {
	{ "88games", "/home/kelv/zznew/EmuELEC/build.EmuELEC-Amlogic-ng.aarch64-4/build/fbneoSA-7200efefe322f153be2e1164be50323db6d5c720/konami/d_88games.cpp"},
	{ "flagrall", "/home/kelv/zznew/EmuELEC/build.EmuELEC-Amlogic-ng.aarch64-4/build/fbneoSA-7200efefe322f153be2e1164be50323db6d5c720/pst90s/d_1945kiii.cpp"},

instead of

static game_sourcefile_entry sourcefile_table[] = {
	{ "88games", "konami/d_88games.cpp"},
	{ "flagrall", "pst90s/d_1945kiii.cpp"},

I have absolutely no idea how you are doing this, your instructions don't allow me to reproduce it, and our buildbot doesn't have this issue either.
It seems to me you are doing something you are not mentioning.

@barbudreadmon
Copy link
Collaborator

barbudreadmon commented Nov 11, 2023

So it does happen when the makefile is moved elsewhere, which seems to be a thing with emuelec's build system.
37e23e3 should take care of this.

@Kelvfimer
Copy link
Author

@barbudreadmon thx so much for taking in account the path for driverlist . The change you did make the result. Kind Regards!!!

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