Opened 9 years ago

Last modified 9 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)

stdout.diff​ (2.2 KB ) - added by diver 9 years ago.
LIBSDL.kmk.diff​ (824 bytes ) - added by Valery V. Sedletski 9 years ago.
kBuild LIBSDK.kmk fix
LIBSDL.kmk.2.diff​ (1.2 KB ) - added by dmik 9 years ago.
SDL.diff​ (2.4 KB ) - added by Valery V. Sedletski 9 years ago.
My variant of stdout.diff (configure patch for using both standard and SDK SDL)
stdout_2.diff​ (2.3 KB ) - added by diver 9 years ago.
ny version

Download all attachments as: .zip

Change History (11)

by diver, 9 years ago

Attachment: stdout.diff​ added

comment:1 by diver, 9 years ago

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.

by Valery V. Sedletski, 9 years ago

Attachment: LIBSDL.kmk.diff​ added

kBuild LIBSDK.kmk fix

comment:2 by Valery V. Sedletski, 9 years ago

I attached LIBSDL.kmk patch, to use rpm SDL from /@unixroot/usr prefix, instead of PATH_SDK_LIBSDL.

comment:3 by dmik, 9 years ago

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.

by dmik, 9 years ago

Attachment: LIBSDL.kmk.2.diff​ added

by Valery V. Sedletski, 9 years ago

Attachment: SDL.diff​ added

My variant of stdout.diff (configure patch for using both standard and SDK SDL)

by diver, 9 years ago

Attachment: stdout_2.diff​ added

ny version

comment:4 by diver, 9 years ago

this is fixed now in r113, but we leave it open until a new kbuild is out

comment:6 by diver, 9 years ago

Milestone: → Enhanced
Note: See TracTickets for help on using tickets.