| Re: [AD] Snapshot generation error |
[ Thread Index |
Date Index
| More lists.liballeg.org/allegro-developers Archives
]
On Saturday 14 January 2006 12:34, Stepan Roh wrote:
> Snapshot generation for 4.2 revealed this error:
Ah! That's due to the MacOS X patch I posted yesterday (or would have if my
mail server wasn't blocked) about detecting the endianesse on MacOS X.
This in light of the new Intel-based Macs which have different endianesse
from the older Macs.
If you're not using a Mac, it doesn't define an endianesse, hence the
error.
> gcc -MM -MG -I. -I./include -DSCAN_DEPEND -DALLEGRO_MACOSX src/*.c
> src/unix/*.c src/misc/*.c src/c/*.c demo/*.c > _depend.tmp
> In file included from include/allegro/base.h:40,
> from include/allegro.h:25,
> from src/allegro.c:23:
> include/allegro/internal/alconfig.h:350:5: #error endianess not defined
>
> It repeats itself many times and ends with:
I think that error should not be triggered when SCAN_DEPEND is defined
because we're not interested in compiling the library itself. On the other
hand, it should not matter which one we define in this case.
I can see two possibel fixes: always define ALLEGRO_LITTLE_ENDIAN (say)
when SCAN_DEPEND is true for MacOS X, or disable the error when
SCAN_DEPEND is defined. I think the second is the more sane behavior and
I'll submit a patch accordingly.
Evert