Bug #223

Fix OpenBSD issues

Added by IR4T4 over 5 years ago. Updated over 2 years ago.

Status:New% Done:

0%

Priority:Normal
Assignee:-
Category:General
Target version:ALL
OS:*BSD Arch:

Description

It seems macro '__linux__’ is not active on OpenBSD - see http://www.dev.etlegacy.com/projects/etlegacy/repository/revisions/1b7d0d9ce3aa4d496679b296974fdcf719e7d09d/diff/src/game/g_lua.h

1. There are some locations in the code where the '__linux__’ macro is used only! (simple text search in /src '__linux__’)

2. There is '__FreeBSD_kernel__’ (q_plattform.c 185) and '__BSD__’ (net_ip.c 1562) defined. Isn’t this the same?


Related issues

Related to ET: Legacy Development - Bug #877: Serverbuild on FreeBSD Fixed 27.08.2015

History

#1 Updated by IR4T4 over 5 years ago

  • Description updated (diff)

#2 Updated by IR4T4 about 5 years ago

  • Target version changed from 2.71rc2 to 2.77

#3 Updated by Jacker almost 5 years ago

Why wont we just add:
#if defined('__BSD__’) || defined('__FreeBSD_kernel__’)
ifndef '__linux__’
#define '__linux__’
#endif
#endif

? Without the ' signs ofc..
I know its an ugly solution but, still prettyer than quadtrillion new macros..

#4 Updated by IR4T4 almost 5 years ago

Jacker wrote:

Why wont we just add:
#if defined('__BSD__’) || defined('__FreeBSD_kernel__’)
ifndef '__linux__’
#define '__linux__’
#endif
#endif

? Without the ' signs ofc..
I know its an ugly solution but, still prettyer than quadtrillion new macros..

I’m not sure if we can do that w/o modifing more in q_platform.h. '__BSD__’ macro (which is kind of opposite case of macro '__linux__’ is not active on OpenBSD) doesn’t seem available on all BSD platforms - we set it explicitly in current line 258 of q_plattform.h.

#5 Updated by Radegast over 4 years ago

  • OS set to Linux

#6 Updated by IR4T4 about 4 years ago

  • OS changed from Linux to *BSD

#7 Updated by IR4T4 almost 3 years ago

  • Related to Bug #877: Serverbuild on FreeBSD added

#8 Updated by IR4T4 almost 3 years ago

  • Description updated (diff)

#9 Updated by IR4T4 over 2 years ago

  • Target version changed from 2.77 to ALL

Also available in: Atom PDF