Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installer compilation error (Delphi 7) #190

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

emersonline86
Copy link

While trying to compile the installer using Install.bat, the following error occurs:

Compile.pas(1227) Error: Missing operator or semicolon
Frames\FrmInstall.pas(44) Fatal: Could not compile used unit 'Compile.pas'

The call to the procedure Exit at line 1227 in the Compile.pas unit should be made with no parameters.

While trying to compile the installer using Install.bat, the following error occurs:

Compile.pas(1227) Error: Missing operator or semicolon
Frames\FrmInstall.pas(44) Fatal: Could not compile used unit 'Compile.pas'

The call to the procedure Exit at line 1227 in the Compile.pas unit should be made with no parameters.
@obones
Copy link
Member

obones commented Jan 3, 2025

Which version of Delphi are you using to build the installer? This Exit(Value) syntax has been there for quite a while now.

@emersonline86
Copy link
Author

emersonline86 commented Jan 3, 2025

My bad, forgot to montion the targert compiler. I'm using Delphi 7. Until Delphi 2009, there was no orverload of Exit procedure that accepted parameters.

Now it's just passing Win64xResult as the function's result before exiting.

@emersonline86 emersonline86 changed the title Installer compilation error Installer compilation error (Delphi 7) Jan 3, 2025
Calls the Exit procedure with Win64xResult as a parameter only if the compiler version is Delphi 2009 or higher.
Simplified the code so regardless of the Delphi version Win64xResult is passed as the function's result before calling Exit.
Now it only exits when Win64xResult <> 0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants