View Single Post
  #4  
Old 28-08-2021, 08:43
Cesar82's Avatar
Cesar82 Cesar82 is offline
Registered User
 
Join Date: May 2011
Location: Brazil
Posts: 1,073
Thanks: 1,814
Thanked 2,302 Times in 786 Posts
Cesar82 is on a distinguished road
Quote:
Originally Posted by oo7 View Post
Conversion Updated

[Changes]
1.--Now both merging and non merging of records.ini methods are supported.
2.--removed "records_modder.exe" and "dous.oo7".


@Cesar82


Its because in the earlier version, IDK why while copying the records from src to tmp, the key Task=1 is added to records and also the setup.exe is recognized as "trojan wacatac b ml".

The records problem was solved by deleting the tmp folder and about "trojan wacatac b ml", I still don't know why it is recognized as "trojan wacatac b ml"
First of all I want to say that your design was a great job.

Very strange. Here my antivirus does not detect anything.

If you compiled your script using Inno Setup 6, try recompiling the script with Inno Setup 5.59. If you use IS Enhanced it will not be possible.
Usually builds created with Inno Setup 6 are detected as false positives by some antiviruses.

P.S:
I noticed that you used an IniFile.exe executable to remove the keys from the Records.ini file by calling the Exec function.
You could have used Inno Setup's built-in removing ini file entries, such as:
Code:
function InitializeSetup(): Boolean;
var
  I: Integer;
begin
  if FileExists(ExpandConstant('{src}\Records.ini')) then
  begin
    for I := 1 to 50 do
      DeleteIniEntry('Record' + IntToStr(I), 'Task', ExpandConstant('{src}\Records.ini'));
  end;
  Result := False;
end;

Last edited by Cesar82; 28-08-2021 at 08:45.
Reply With Quote
The Following User Says Thank You to Cesar82 For This Useful Post:
oo7 (28-08-2021)