Bug #48059 for Mac-SysProfile: Some values are not read

Tue Jul 21 12:42:06 2009 adadov [...] adadov.net - Ticket created

Subject:

Some values are not read

You can find a "patch" here : http://paste.debian.net/42295/ but this patch won't fix everything There is many values on a Mac OS 10.5.7 which are not read by Mac-SysProfile

Wed Jul 22 05:33:44 2009 adadov [...] adadov.net - Status changed from 'new' to 'open'

Wed Jul 22 05:34:04 2009 adadov [...] adadov.net - Correspondence added

--- SysProfile.pm_orig.pm 2005-08-27 01:49:56.000000000 +0200 +++ SysProfile.pm 2009-07-21 18:06:40.000000000 +0200 @@ -39,6 +39,7 @@ } my $raw = $fre || !$pro->{$typ} ? `$conf{bin} $typ` : $pro->{$typ}; my $hdr = ''; + my $hdr2 = ''; for(split /\n/, $raw) { next if m/^\s*$/ || m/^\w/; if(m/^\s{4}\w/) { @@ -46,6 +47,7 @@ $hdr =~ s/^\s+//; $hdr =~ s/:.*$//; $pro->{$typ}->{$hdr} = {}; + $hdr2 = ''; } elsif(m/^\s{6}\w/) { s/^\s+//; s/\s+$//; @@ -55,6 +57,16 @@ } else { $pro->{$typ}->{$k} = $v; } + } elsif(m/^\s{8}\w/ && m/:$/) { + $hdr2 = $_; + $hdr2 =~ s/^\s+//; + $hdr2 =~ s/:.*$//; + $pro->{$typ}->{$hdr}->{$hdr2} = {}; + } elsif(m/^\s{10}\w/ && $hdr2) { + s/^\s+//; + s/\s+$//; + my($k,$v) = split /:\s+/; + $pro->{$typ}->{$hdr}->{$hdr2}->{$k} = $v; } } return $pro->{$typ};

Mon Aug 17 06:37:59 2009 adadov [...] adadov.net - Told changed from Not set to Mon Aug 17 06:37:59 2009

Mon Aug 17 06:40:14 2009 adadov [...] adadov.net - Correspondence added

This Bug is really important for me, I'm blocked in another development using this module. I may fork it if there is no reply quite soon.

Mon Aug 17 09:05:53 2009 dmuey [...] cpan.org - Correspondence added

thanks, I'll try to have a look and update the case by tonight. If you have any tests to add and/or an example of the problem/list of items it misses that'd be excellent.

Mon Aug 17 09:47:38 2009 adadov [...] adadov.net - Correspondence added

From:

adadov [...] adadov.net

You can look for the Memory values or Storages like SATA and PATA which can't be read by the actual version because there is 6 or 8 spaces before datas.

Tue Aug 18 08:00:35 2009 dmuey [...] cpan.org - Correspondence added

Sorry last night was a wash, I'll try again today and tomorrow, thanks!

Thu Aug 20 15:24:50 2009 dmuey [...] cpan.org - Correspondence added

Sorry to leave you all hanging. I'll try again tonight or tomorrow, haven't forgotten about you :) thanks

Sun Aug 23 13:10:44 2009 dmuey [...] cpan.org - Status changed from 'open' to 'resolved'

Sun Aug 23 13:10:48 2009 dmuey [...] cpan.org - Fixed in 0.03 added