On Saturday 07 July 2012 16:05:06 Vladimir Yakovlev wrote:
> --- a/tools/mkenvimage.c
> +++ b/tools/mkenvimage.c
> @@ -45,6 +45,13 @@
> 
>  #define CRC_SIZE sizeof(uint32_t)
> 
> +#ifdef __MINGW32__
> +#define FILE_PERM            (S_IRUSR | S_IWUSR)
> +#else
> +#define FILE_PERM            (S_IRUSR | S_IWUSR | S_IRGRP |\
> +                                          S_IWGRP)
> +#endif

err, i'd rather you didn't do this.  we've tried very hard to avoid sprinkling 
arch-specific crap (like __MINGW32__) in random files. instead, i'd like to see 
in (either in os_support.h or mingw_support.h):
#ifndef S_IRGRP
# define S_IRGRP 0
#endif
#indef S_IWGRP
# define S_IWGRP 0
#endif

then no one has to care about mingw, and we don't have to add complicated 
defines that disconnect the locations of the code (the define and the open()).
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to