Skip to content

add support for OpenBSD#196

Merged
danhedron merged 1 commit intorwengine:masterfrom
qbit:master
Aug 12, 2016
Merged

add support for OpenBSD#196
danhedron merged 1 commit intorwengine:masterfrom
qbit:master

Conversation

@qbit
Copy link
Contributor

@qbit qbit commented Aug 11, 2016

This PR lets OpenRW build on OpenBSD! Game starts and is playable!

elseif(CMAKE_SYSTEM_NAME STREQUAL "OpenBSD")
add_definitions(-DRW_OPENBSD)
include_directories("/usr/X11R6/include")
set(OPENRW_PLATFORM_LIBS iconv)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We depend on iconv on all platforms but will get rid of it soon (so this hack is fine).

@JayFoxRox
Copy link
Collaborator

JayFoxRox commented Aug 11, 2016

This might be an absolutely stupid question, but we support FreeBSD already, wouldn't compiling for that also work for OpenBSD? (e.g. can't we just call it RW_BSD?)


//Edit (because the old comment was outdated, just a heads up):
"We depend on iconv on all platforms but will get rid of it soon (so this hack is fine)."

@mulander
Copy link

@JayFoxRox FreeBSD and OpenBSD are different kernels, user lands and available third party Libraries. While they do share some common ancestry code they are different systems with different feature sets. It might compile now but could soon become problematic with one OS lacking features of the other or implementing them in a vastly different way.

@krytarowski
Copy link
Contributor

After merging it I will push NetBSD support.

@JayFoxRox
Copy link
Collaborator

JayFoxRox commented Aug 11, 2016

PC did not explode, also did not forcefully download OpenBSD and shoved it on my harddrive.
Still works on Arch Linux.

Code review process was very hard and long but also passed:

LGTM

PS: I want at least one more LGTM from an OpenBSD user on this, possibly also MacOS or even Windows, I've already mentioned that on IRC.

//Edit: All 3 next "LGTM" were with the same version as the new one, only the commit name has changed between this and mulander's "LGTM"

@krytarowski
Copy link
Contributor

OK from NetBSD user :)

@christoph-heiss
Copy link
Contributor

+1 for macOS.

@mulander
Copy link

Waiting for GTA 3 game data to download before doing a test run on OpenBSD -current amd64

@mulander
Copy link

Tested on OpenBSD/amd64 -current

The game builds without issues and can be played. LGTM from an OpenBSD user :)

$ ./rwgame/rwgame
I [Game] Game directory: /home/mulander/Games/GTA3/
I [Game] Build: a8db29cd
Max UBO Size: 65536
UBO Alignment: 16
Max batch size: 512
I [Renderer] OpenGL Renderer Version: 3.3 (Core Profile) Mesa 11.2.2 (GLSL 3.30) Vendor: Intel Open Source Technology Center
I [Data] Loaded 42 zones from /home/mulander/Games/GTA3/data/gta3.zon
I [Game] Started
Error opening sound file "/home/mulander/Games/GTA3//audio/City.wav": System error : No such file or directory.
I [World] Loaded cutscene: BET
I [World] Loaded cutscene: JB
I [World] Creating Vehicle ID 116 (POLICAR)
I [World] Creating Vehicle ID 117 (ENFORCR)
I [World] Creating Vehicle ID 130 (RUMPO)
I [World] Creating Vehicle ID 96 (PATRIOT)
I [World] Creating Vehicle ID 110 (TAXI)
I [World] Creating Vehicle ID 105 (CHEETAH)
I [World] Creating Vehicle ID 92 (STINGER)
I [World] Creating Vehicle ID 105 (CHEETAH)
I [World] Creating Vehicle ID 92 (STINGER)
I [World] Creating Vehicle ID 111 (KURUMA)
I [World] Creating Vehicle ID 110 (TAXI)
I [World] Creating Vehicle ID 111 (KURUMA)
I [World] Creating Vehicle ID 110 (TAXI)
I [World] Creating Vehicle ID 111 (KURUMA)
I [World] Creating Vehicle ID 111 (KURUMA)
I [World] Creating Vehicle ID 111 (KURUMA)
I [World] Creating Vehicle ID 93 (LINERUN)
I [World] Creating Vehicle ID 108 (MOONBM)
I [World] Creating Vehicle ID 116 (POLICAR)
I [World] Creating Vehicle ID 116 (POLICAR)
I [World] Creating Vehicle ID 91 (IDAHO)
I [World] Creating Vehicle ID 130 (RUMPO)
I [World] Creating Vehicle ID 91 (IDAHO)
I [World] Creating Vehicle ID 109 (ESPERAN)
I [Game] Beginning cleanup
I [Game] Stopping work queue
I [Game] Cleaning up scripts
I [Game] Cleaning up renderer
I [Game] Cleaning up world
I [Game] Cleaning up state
I [Game] Cleaning up window
I [Game] Cleaning up work queue
I [Game] Done cleaning up

@qbit qbit changed the title add fixes for OpenBSD add support for OpenBSD Aug 12, 2016
@danhedron
Copy link
Member

+1 from me.

@danhedron danhedron merged commit 160598b into rwengine:master Aug 12, 2016
@brightersun99
Copy link

glad to see this never got anywhere

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.

7 participants