Bug 46

Summary: Mac: Installer 156 does not set $saconfig right after upgrading.
Product: TestProduct Reporter: Kuanshen An <kuanshen.an>
Component: TestComponentAssignee: Brian Roberge <brian.roberge>
Status: VERIFIED FIXED    
Severity: critical CC: sahil.parbhaker
Priority: Highest    
Version: unspecified   
Hardware: Macintosh   
OS: Mac OS   

Description Kuanshen An 2018-08-01 22:41:13 EDT 
Have no idea if other installer has the issue. I just spot check the build and this is the first isntaller I see the issue.


On Mac
Prepare a Apollo Reader 2.1.2 with following configuration
	$langset,1,5,6
	$marketlev,1,0,1,0
	
Launch installer ID #156 of  build 728
Complete the upgrade
check the following config on Reader

$swver?
$marketlev?
$langset?
$lang?
$uom?
$saconfig?
$iopamp?

Actual result:
2.4.7
CKSM:00000110
CMD OK
1,0,1,0
CKSM:0000015D
CMD OK
10,13,11,6
CKSM:000001F8
CMD OK
10
CKSM:00000078
CMD OK
1
CKSM:00000048
CMD OK
15
CKSM:0000007D
CMD OK
1
CKSM:00000048
CMD OK

==> $saconfig? return 15

Expect result:
$saconfig? should return 14
( for ESAd = 0
ESAc = 1
LSAd = 1
LSAc = 1 in configuration table)
Comment 1 Kuanshen An 2018-08-01 22:45:54 EDT 
$langset? reutn is wrong, too; it was already reported in bug 43.
Comment 2 Brian Roberge 2018-08-02 21:52:41 EDT 
Fixed in 733, Issue with the configuration files
Comment 3 Kuanshen An 2018-08-07 17:59:48 EDT 
not fixed in build 743 ( with latest firmware 2.4.8).

I suspect this might happens to other installers as well ( see bug 50 ).
Comment 4 Brian Roberge 2018-08-07 20:43:36 EDT 
I tried with 156 build 744 and could not reproduce the problem.  $saconfig? returned 14.
Comment 5 Kuanshen An 2018-08-07 23:26:21 EDT 
strange. I got 15 return $saconfig, with the same installer 156 of build 744 on Mac.

I used the build from FTP site. Did you use your local copy? Is possible the error from build system?
Comment 6 Brian Roberge 2018-08-08 15:06:07 EDT 
Randomly not setting bits correctly, fixed in build 745
Comment 7 Kuanshen An 2018-08-09 19:29:16 EDT 
still get 15 in build 748_156 on Mac.
Comment 8 Brian Roberge 2018-08-14 17:24:42 EDT 
Fixed code  Build 760
Comment 9 Brian Roberge 2018-08-17 17:25:33 EDT 
Fixed in build 781
Comment 10 Sahil Parbhaker 2018-08-20 20:30:19 EDT 
Fixed in MAC installer 156 build 787.$saconfig value is reported as 14 and not 15. The other values is correct as well.