[PATCH 3/3] build: make wafsamba WORDS_BIGENDIAN test cross-compile friendly
Gustavo Zacarias
gustavo at zacarias.com.ar
Tue Apr 1 03:24:40 MDT 2014
On 03/31/2014 10:18 PM, Andrew Bartlett wrote:
> I would prefer you did as suggested, and unified the whole test. In
> particular, coping over just the big-endian handling and assuming that
> this macro being absent is authoritative for little-endian seems
> dangerous, we should always do the fallback the ccan code does.
Ok, i didn't know if it was ok to condense the whole endian test in
wafsamba (seemed the appropiate thing to do, reason why i mentioned it).
I think i'll keep the separate HAVE_X_ENDIAN and WORDS_BIGENDIAN
definitions separate though right?
Since it seems you're using foreign codebases and it's a oneliner to do
so rather than modifying them.
Thanks.
Regards.
More information about the samba-technical
mailing list