Skip to content

Renamed SDL_ttf.h to SDL2_ttf.h#20

Open
ellisvelo wants to merge 5 commits into
Failproofshark:masterfrom
ellisvelo:master
Open

Renamed SDL_ttf.h to SDL2_ttf.h#20
ellisvelo wants to merge 5 commits into
Failproofshark:masterfrom
ellisvelo:master

Conversation

@ellisvelo
Copy link
Copy Markdown

Hello,

I ran into a small problem when c2ffi is trying to execute because it is looking for SDL2_ttf.h referenced in sdl2-ttf.asd, but the file doesn't exist. This pull request renames SDL_ttf.h to SDL2_ttf.h so that c2ffi executes successfully. I wasn't sure if sdl2-ttf.asd should change or the header filename because some of the spec files start with SDL2 and others start with SDL. Thanks for looking into this.

TTF_SetFontOutline outline argument represents the pixel size of the outline and
not a packed bit array like TTF_SetFontStyle.
@justjoheinz
Copy link
Copy Markdown

@ellisvelo I just ran into the same problem, wanted to make a PR and noticed that the maintainer did not do anything on github since 2020. I think it is time to fork these repositories. You already did an awesome job with the cl-sdl2 library and M1 architecture. Would you be willing to?

I also would like to get https://github.com/TatriX/cl-sdl2-tutorial running again and link the tutorial to the cl-sdl2 repo, so that more people are enabled to write graphical apps in Common Lisp.

@ellisvelo
Copy link
Copy Markdown
Author

Thanks for the compliment @justjoheinz. 👍 I do have a fork of this repo (https://github.com/ellisvelo/cl-sdl2-ttf), but I think that I should make a PR for the Sharp Lispers and combine efforts: https://github.com/sharplispers/cl-sdl2-ttf

That's a good idea about the cl-sdl2-tutorial because I think the LazyFoo tutorials have changed too. Go for it!

Sometimes there is a compilation failure with sbcl because nreverse is
used on a macro argument.
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.

2 participants