Skip to content

Prevent setting 'Run Jamulus' if driver not found#2103

Merged
pljones merged 3 commits intojamulussoftware:masterfrom
henkdegroot:disable-run-client
Dec 31, 2021
Merged

Prevent setting 'Run Jamulus' if driver not found#2103
pljones merged 3 commits intojamulussoftware:masterfrom
henkdegroot:disable-run-client

Conversation

@henkdegroot
Copy link
Copy Markdown
Contributor

@henkdegroot henkdegroot commented Nov 1, 2021

Short description of changes

Installer option to run the Jamulus Client after installation will be set to unchecked when the installation is performed without an audio driver

Context: Fixes an issue?

Fixes: #2099

Does this change need documentation? What needs to be documented and how?

No documentation update required

Status of this Pull Request
Local build tested with and without ASIO driver and JACK installed

What is missing until this pull request can be merged?
Nothing

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I tested my code and it does what I want
  • My code follows the style guide
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I've filled all the content above

Comment thread windows/installer.nsi Outdated
@henkdegroot henkdegroot changed the title Disable run client for installation without audio driver Disable run client for windows installation without audio driver Nov 1, 2021
@henkdegroot henkdegroot changed the title Disable run client for windows installation without audio driver Prevent setting 'Run Jamulus' if driver not found Dec 5, 2021
Copy link
Copy Markdown
Collaborator

@pljones pljones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks nice and simple.

Copy link
Copy Markdown
Member

@ann0see ann0see left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks quite simple.

@pljones
Copy link
Copy Markdown
Collaborator

pljones commented Dec 31, 2021

@henkdegroot Are you okay to fix the conflicts?

@henkdegroot
Copy link
Copy Markdown
Contributor Author

@pljones I wasn't sure if I could/should do that or not. I have have resolved the conflicts now.

@ann0see
Copy link
Copy Markdown
Member

ann0see commented Dec 31, 2021

The JACK build failed…

@henkdegroot
Copy link
Copy Markdown
Contributor Author

The JACK build failed…

It failed because JACK could not be installed in the prepare for windows step.
Can you rerun the build workflow?

@henkdegroot
Copy link
Copy Markdown
Contributor Author

@ann0see, pretty annoying but this time NSIS download failed. This seems to be happening very often lately, I don't know why.
Good news is that the jack install did work as normal.

Can you do another re-run please?

@henkdegroot
Copy link
Copy Markdown
Contributor Author

Finally passed the autobuild 👍

@pljones pljones merged commit 85412d0 into jamulussoftware:master Dec 31, 2021
@henkdegroot henkdegroot deleted the disable-run-client branch January 1, 2022 10:46
@pljones pljones added this to the Release 3.8.2 milestone Feb 13, 2022
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.

Prevent automatic start of Jamulus after installation on windows when no audio driver available

4 participants