Ticket #396 (assigned patch) — at Version 1

Opened 13 years ago

Last modified 11 years ago

32/64-bit cpu in smolder report

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

Description (last modified by rurban) (diff)

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:09:50.171875000 +0100
@@ -100,8 +100,12 @@
 }

 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},
+        'Architecture' => $arch,
         'Compiler'     => _get_compiler_version(),
         'DEVEL'        => $PConfig{DEVEL},
         'Optimize'     => ($PConfig{optimize} || 'none'),

Change History

Changed 13 years ago by rurban

  • owner set to rurban
  • status changed from new to assigned
  • description modified (diff)
Note: See TracTickets for help on using tickets.