Skip to content

Installer#377

Open
jimmckeeth wants to merge 7 commits intoVSoftTechnologies:masterfrom
jimmckeeth:Installer
Open

Installer#377
jimmckeeth wants to merge 7 commits intoVSoftTechnologies:masterfrom
jimmckeeth:Installer

Conversation

@jimmckeeth
Copy link
Contributor

Added Delphi 13 support, and created a simple installer with PowerShell and installer documentation. There is no GUI, and can be run interactive in the terminal or automated with parameters.

@vincentparrett
Copy link
Member

Hi Jim

I appreciate the effort - but not sure if I want to merge this - I don't relally like having scripts in repos.

What are the new packages for (vcl/fmx).

@jimmckeeth
Copy link
Contributor Author

The packages are just an easy way to build all the units at once. The FMX vs VCL is just based on what GUI library the files use. It doesn't build the units that require optional 3rd party libraries.

Would you like it better it I made it a Delphi program or a regular installer like InnoSetup?

@vincentparrett
Copy link
Member

InnoSetup would be a better option - but tbh I don't see a need for an installer - there are no components that need installing, you can literally just clone the repo and update you library or project search path.

@jimmckeeth
Copy link
Contributor Author

I like using pre-compiled units and my general rule of thumb is if I have to do something more than once then it had better be automated. Saves me time and then I don't need to worry about making mistakes.

Do you prefer InnoSetup or a Delphi command-line program?

@vincentparrett
Copy link
Member

Innosetup would be better - but then someone has to build it (and compile for each delphi version) and create releases - that will take me a while to get to as I am already heavily over subscribed.

@jimmckeeth
Copy link
Contributor Author

I replaced it with an InnoSetup installer. It should support Delphi 2010 through 13.1, and it compiles the units at install time for the selected platforms. I guess technically it won't work with community edition...

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