[Box Backup-dev] [PATCH] Win32 merge [02] BoxPlatform.pm and uname -p

Martin Ebourne boxbackup-dev at fluffy.co.uk
Mon Aug 7 12:21:01 BST 2006


Chris Wilson <chris at qwirx.com> wrote:
> It seems that "uname -p", as used in BoxPlatform.pm.in, is not very
> portable. I'm not even sure what $build_cpu is used for, if anything,
> but "uname -m" should work better than "uname -p".

Changing from uname -p to uname -m could be dangerous because they  
mean different things. One is processor type, the other is machine  
architecture. On Solaris for instance they return different results,  
eg. sparc and sun4u respectively.

However, Box isn't using the result of build_cpu so it doesn't matter  
a jot. My guess is that it was used before autoconf (eg. I think for  
endianness - which is generally a processor not architecture thing  
except on ppc) and it was something I missed in the cleanup.

So no to this version of the patch, please remove that code entirely.

Cheers,

Martin.



More information about the Boxbackup-dev mailing list