id	summary	reporter	owner	description	type	status	priority	milestone	component	version	severity	resolution	keywords	cc	lang	patch	platform
396	32/64-bit cpu in smolder report	rurban		"I miss crucial 32/64bit info on the smolder reports.
I want to add it to the cpuarch, similar to amd64 or ia64.

So ppc is either pcc32 or ppc64, alpha is alpha32 or alpha64, ...
sun4 should be sparc32 or sparc64. Just to make it clear.

Ok to apply?

{{{
--- parrot-svn.orig/lib/Parrot/Harness/Smoke.pm	2009-03-03 19:54:35.671875000 +0100
+++ parrot-svn/lib/Parrot/Harness/Smoke.pm	2009-03-03 20:32:42.390625000 +0100
@@ -100,8 +100,14 @@
 }
 
 sub collect_test_environment_data {
+    # rename sun4 to sparc
+    my $arch = $PConfig{cpuarch} eq 'sun4' ? 'sparc' : $PConfig{cpuarch};
+    # add the 32/64 bit suffix to the cpuarch
+    if ($PConfig{cpuarch} !~ /\d$/) {
+      $arch = $PConfig{cpuarch} . (8*$PConfig{wordsize});
+    }
     return (
-        'Architecture' => $PConfig{cpuarch},
+        'Architecture' => $arch,
         'Compiler'     => _get_compiler_version(),
         'DEVEL'        => $PConfig{DEVEL},
         'Optimize'     => ($PConfig{optimize} || 'none'),
}}}"	patch	closed	normal		website		medium	fixed	64bit	Coke		applied