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

Pin Inno Setup 6 and Build with Inno Setup 6 on Azure Pipelines #357

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

Conversation

nelsonjchen
Copy link

No description provided.

@AdrianKoshka
Copy link

@nelsonjchen Should I merge this ASAP assuming CI passes?

@nelsonjchen
Copy link
Author

I haven't had time to validate Inno Setup 6 working.

@AdrianKoshka
Copy link

What would need to be done to validate that it is working?

@nelsonjchen
Copy link
Author

nelsonjchen commented Aug 5, 2019

Ooh! That's a checklist. I guess for an installer:

  • Make sure it installs the files on disks and adds an entry to Add or Remove Programs.
  • Make sure the application works.
  • Make sure it uninstalls the files and removes the entry in areas like the Add or Remove Programs
  • definitely more

@shymega shymega added the installer/package Related to the Barrier installer label Aug 14, 2019
@shymega
Copy link

shymega commented Nov 6, 2021

@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.

@nelsonjchen
Copy link
Author

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.

@shymega
Copy link

shymega commented Nov 6, 2021

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.

@shymega
Copy link

shymega commented Nov 6, 2021

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.

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.

@p12tic
Copy link
Member

p12tic commented Nov 7, 2021

Please test whether the produced installer actually works across the Windows versions that we currently support.

@shymega
Copy link

shymega commented Nov 7, 2021

@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...

@nelsonjchen
Copy link
Author

@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/

@shymega
Copy link

shymega commented Nov 7, 2021

That's... really surprising, lol. My connection is pretty bad, so I'll have to download overnight...

@walker0643
Copy link
Member

I agree with the version bump but the batch file modification will be unnecessary after #400 is merged. I suggest testing/merging both together.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
installer/package Related to the Barrier installer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants