Compilation fails on 32 bit Solaris
Bug #404506 reported by
Trond Norbye
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Gearman |
Fix Released
|
Medium
|
Eric Day | ||
Bug Description
cc -xc99=all -DHAVE_CONFIG_H -I.. -I.. -I/usr/local -I/home/
"../config.h", line 204: macro redefined: _FILE_OFFSET_BITS (E_MACRO_REDEFINED)
cc: acomp failed for test_gearmand.c
We need to include config.h _before_ any system headers, because config.h doesn't test if the macro is set before trying to define it.
Revision history for this message
| Monty Taylor (mordred) wrote Re: [Bug 404506] [NEW] Compilation fails on 32 bit Solaris | #1 |
| Changed in gearmand: | |
| assignee: | nobody → Eric Day (eday) |
| importance: | Undecided → Medium |
| milestone: | none → 0.10 |
| status: | New → Fix Committed |
| Changed in gearmand: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Trond Norbye wrote:
> Public bug reported:
>
> cc -xc99=all -DHAVE_CONFIG_H -I.. -I.. -I/usr/local -I/home/
> "../config.h", line 204: macro redefined: _FILE_OFFSET_BITS (E_MACRO_REDEFINED)
> cc: acomp failed for test_gearmand.c
>
> We need to include config.h _before_ any system headers, because
> config.h doesn't test if the macro is set before trying to define it.
Agree. We also need to include config.h before any other headers for
many other reasons too.