00158: common.mk is not defining __LINUX__

« 00157 | Issues | 00159 »

Summary: common.mk is not defining __LINUX__

Created: 2009–06–10 14:27

Status: Released

Category: Bug

From: frankie

Version: 2.0

Released_In: 2.1

Description:

in src.dist, the common.mk file should be defining "__LINUX__" in the CFLAGS and is not.

---

Comments

frankie June 10, 2009, at 03:24 PM: Ok.. I fixed this. If you want to fix this yourself, you can add the following code to src.dist/common.mk around line 42:

ifeq ($(HOST_TYPE),Linux)
       CFLAGS += -D__LINUX__
endif

or, you apply this patch file: Attach:0001-Issue-00158-Added-__LINUX__-define-to-common.mk.patch

If you are unfamiliar with applying patch files, just download that file into your edge directory and run:

patch -p1 < 0001-Issue-00158-Added-__LINUX__-define-to-common.mk.patch

---

« 00157 | Issues | 00159 »

Associated Commits

| commit | 6c1a136e4669f329de51ec0f20939289d39d386a link6 | || | Author: | Frank Graffagnino | | Date: | Wed Jun 10 14:41:45 2009 -0500 | | Message: | [@Issue 00158: Added __LINUX__ define to common.mk This symbol needs to be defined if we are compiling on Linux since some of the DOUG headers use this to turn on or off byte swapping. I think we also need a similar check and define for __WIN32__ but I didn't add it here yet. @] |

Affected Files:

src.dist/common.mk | 3 +++
 1 file changed, 3 insertions(+)