-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Pin Inno Setup 6 and Build with Inno Setup 6 on Azure Pipelines #357
base: master
Are you sure you want to change the base?
Conversation
@nelsonjchen Should I merge this ASAP assuming CI passes? |
I haven't had time to validate Inno Setup 6 working. |
What would need to be done to validate that it is working? |
Ooh! That's a checklist. I guess for an installer:
|
Signed-off-by: Dom Rodriguez <[email protected]>
Signed-off-by: Dom Rodriguez <[email protected]>
@nelsonjchen I'm going back through PRs. This branch is out of date with our master branch, so I'm just going to rebase, and we can see if CI passes. Not going to merge yet until we can get your checklist completed. |
bcf4c4d
to
bbf142c
Compare
Signed-off-by: Dom Rodriguez <[email protected]>
Whew, it's been far too long. I haven't had time to work on this issue and I've kind of lost my familiarity since then. Please do whatever you want to this PR and/or even replace it completely if you would like. |
I've also bumped the Inno Setup version to the latest. We'll see if this passes. If @p12tic agrees that we should update Inno Setup, then we can merge, once everything is implemented. |
No worries. I've been dedicating more time to Barrier in the last 24 hours than I have for some time now. I'm sorting out PRs, and adjusting our CI infra. |
Please test whether the produced installer actually works across the Windows versions that we currently support. |
@p12tic Sure. I don't have access to any other Windows machine other than Windows 11 though - we may well need to ask the Barrier community to help... |
If you ever need quick, legal, and temporary virtual machines of Windows, I usually look at this: https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/ |
That's... really surprising, lol. My connection is pretty bad, so I'll have to download overnight... |
I agree with the version bump but the batch file modification will be unnecessary after #400 is merged. I suggest testing/merging both together. |
No description provided.