Ticket #396 (new patch) — at Initial Version

Opened 13 years ago

Last modified 11 years ago

32/64-bit cpu in smolder report

Reported by: rurban Owned by:
Priority: normal Milestone:
Component: none Version:
Severity: medium Keywords:
Cc: Coke Language:
Patch status: Platform:

Description

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, ... Ok to apply?

diff -u lib/Parrot/Harness/Smoke.pm~ lib/Parrot/Harness/Smoke.pm
--- lib/Parrot/Harness/Smoke.pm~        2009-02-18 00:01:18.000000000 +0100
+++ lib/Parrot/Harness/Smoke.pm 2009-03-03 15:05:07.812500000 +0100
@@ -100,6 +100,10 @@
 }

 sub collect_test_environment_data {
+    # add the 32/64 bit suffix to the cpuarch
+    my $arch = $PConfig{cpuarch} =~ /\d$/
+      ? $PConfig{cpuarch}
+      : $PConfig{cpuarch} . (8*$PConfig{wordsize});
     return (
         'Architecture' => $PConfig{cpuarch},
         'Compiler'     => _get_compiler_version(),
Note: See TracTickets for help on using tickets.