Skip to content

SDL2_image and SDL2_ttf Toolchain fix#2

Open
bremedios wants to merge 2 commits intoknulli-cfw:knulli-mainfrom
bremedios:sdl2-cross_toolchain_fixes
Open

SDL2_image and SDL2_ttf Toolchain fix#2
bremedios wants to merge 2 commits intoknulli-cfw:knulli-mainfrom
bremedios:sdl2-cross_toolchain_fixes

Conversation

@bremedios
Copy link

Fixes an issue where programs using the toolchain to compile against the SDL2_image and SDL2_ttf images will fail if they are using CMAKE.

SDL2_image

  • Fix that is already included in the SDL2 build script was added to the SDL2_image

SDL2_ttf

  • Fix that is already included in the SDL2 build script was added to the SDL2_image
  • Manually fixed _sdl2ttf_extra_static_libraries to use -lfreetype and -lharfbuzz

that use SDL_image or SDL_ttf and are using the CMAKE build system.

A relocation script was not required after these changes at least
when working against SDL2, SDL2_image and SDL2_ttf.

SDL2_image
    - Copied cmake fix from SDL2 into SDL2_image.
    - Using the toolchain works for static and shared linkage.

SDL2_ttf
    - Same fix as SDL2_image
    - This library also has a bad extra static libs setup where
      it contains more than just static libraries.  REGEX alone
      was not sufficient to resolve, replaced the extra libs with
      the necessary libraries.
@bremedios
Copy link
Author

bremedios commented Nov 11, 2024

Notes / Open Issues:

  • This still builds images for the h700
  • The toolchain is not fixed for the A133, but appears to be fixed for the h700.
  • This does not build for the atm7039 (Original RG35XX)

This has been compiled against the h700

  • Build compiles
  • Toolchain works for cross compilation of program using SDL2, SDL2_image and SDL2_ttf

This has been compiled against the A133

  • Build compiles
  • Toolchain DOES NOT WORK for cross compilation of program using SDL2, SDL2_image and SDL2_ttf**

@bremedios bremedios marked this pull request as draft November 11, 2024 17:29
@bremedios bremedios marked this pull request as ready for review November 12, 2024 01:50
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

Successfully merging this pull request may close these issues.

1 participant