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

Compilation Errors #9

Open
0x90shell opened this issue Sep 13, 2023 · 3 comments
Open

Compilation Errors #9

0x90shell opened this issue Sep 13, 2023 · 3 comments

Comments

@0x90shell
Copy link

0x90shell commented Sep 13, 2023

Compiling on Arch I get the following errors related to osdlib:

In file included from ../../../../../src/osd/modules/lib/osdlib_retro.cpp:18:
../../../../../src/osd/modules/lib/osdlib.h: In member function ‘bool osd::virtual_memory_allocation::set_access(std::size_t, std::size_t, unsigned int)’:
../../../../../src/osd/modules/lib/osdlib.h:111:68: error: ‘uint8_t’ in namespace ‘std’ does not name a type; did you mean ‘wint_t’?
  111 |                         return do_set_access(reinterpret_cast<std::uint8_t *>(m_memory) + start, size, access);
      |                                                                    ^~~~~~~
      |                                                                    wint_t
../../../../../src/osd/modules/lib/osdlib.h:111:76: error: expected ‘>’ before ‘*’ token
  111 |                         return do_set_access(reinterpret_cast<std::uint8_t *>(m_memory) + start, size, access);
      |                                                                            ^
../../../../../src/osd/modules/lib/osdlib.h:111:76: error: expected ‘(’ before ‘*’ token
../../../../../src/osd/modules/lib/osdlib.h:111:77: error: expected primary-expression before ‘>’ token
  111 |                         return do_set_access(reinterpret_cast<std::uint8_t *>(m_memory) + start, size, access);
      |                                                                             ^
../../../../../src/osd/modules/lib/osdlib.h:111:89: warning: pointer of type ‘void *’ used in arithmetic [-Wpointer-arith]
  111 |                         return do_set_access(reinterpret_cast<std::uint8_t *>(m_memory) + start, size, access);
      |                                                                              ~~~~~~~~~~~^~~~~~~
make[3]: *** [ocore_retro.make:633: ../../../../libretro/obj/x64/libretro/ocore_retro/src/osd/modules/lib/osdlib_retro.o] Error 1
make[2]: *** [Makefile:19: ocore_retro] Error 2
@0x90shell
Copy link
Author

0x90shell commented Apr 16, 2024

From upstream issue: zach-morris#12 (comment)

It seems the Mame related files and folders need to be updated to more recent releases to ensure compilation on modern dev environments.

@LibretroAdmin

@2play
Copy link

2play commented Mar 11, 2025

@0x90shell
replace /src/osd/modules/lib/osdlib.h with attached (remove from name the .txt)

osdlib.h.txt

run your compile again

2play added a commit to 2play/same_cdi that referenced this issue Mar 11, 2025
Fixes compilation errors as posted on libretro#9
2play added a commit to 2play/same_cdi that referenced this issue Mar 11, 2025
Fix compilation libretro#9
This was referenced Mar 11, 2025
2play added a commit to 2play/same_cdi that referenced this issue Mar 11, 2025
Fixes compilation errors as per libretro#9
@2play
Copy link

2play commented Mar 11, 2025

@0x90shell
if of interest:

  • Tested current git source on RockPro64 Armbian Ubuntu 20.04 (compiles ok without new osdlib.h)

  • Tested current git source on Pi5 Armbian Ubuntu 24.04 (fails without updated osdlib.h)

  • Tested current git source on Pi5 Armbian Ubuntu 24.04 (compiles ok with updated osdlib.h)

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