Opened 8 years ago
Last modified 8 years ago
#55 new defect
sdl
| Reported by: | diver | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | Enhanced |
| Component: | Common Tasks | Keywords: | |
| Cc: |
Description
when using the rpm version of SDL only, VBox is not buildable anymore. The reason is, that in configure some "$OS" = "os2" are. And those are then obsolete. But even when this is fixed, the /@unixroot/usr/share/kbuild/sdks/LIBSDL.kmk will not like what it gets. So in the end both need a fix.
Attachments (5)
Change History (11)
Changed 8 years ago by
| Attachment: | stdout.diff added |
|---|
comment:1 Changed 8 years ago by
comment:2 Changed 8 years ago by
I attached LIBSDL.kmk patch, to use rpm SDL from /@unixroot/usr prefix, instead of PATH_SDK_LIBSDL.
comment:3 Changed 8 years ago by
To go the Linux way with the newest SDL RPM, we should patch LIBSDL.kmk like the attached one. This will keep the possibility to use a manual PATH_SDK_LIBSDL override (that implies a non-standard non-Linux directory structure) in non-VBox applications. We may actually keep this support in VBox as well just in case - the configure diff should be a bit different then. I suggest that Valeriy will try my LIBSDL.kmk patch, alter configure accordingly, check that all works, attach a new configure diff here for review (or via pastebin) and then commit.
Changed 8 years ago by
| Attachment: | LIBSDL.kmk.2.diff added |
|---|
Changed 8 years ago by
My variant of stdout.diff (configure patch for using both standard and SDK SDL)
comment:4 Changed 8 years ago by
this is fixed now in r113, but we leave it open until a new kbuild is out
comment:6 Changed 8 years ago by
| Milestone: | → Enhanced |
|---|
with the attached diff configure runs and the sdl frontend gets built.
the only change still needed is the one in sdl-config. As there is a strange include path provided.