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

Improve build logic for JACK-related builds & warn about ASIO #2443

Merged
merged 4 commits into from
Mar 2, 2022

Conversation

hoffie
Copy link
Member

@hoffie hoffie commented Mar 1, 2022

Short description of changes

  • Build: Don't download ASIO for JACK builds
  • Windows: Add warnings about ASIO SDK usage
  • Jamulus.pro: Improve JACK/CoreAudio distinction on macOS
  • Jamulus.pro: Improve JACK/ASIO distinction on Windows

CHANGELOG: Internal: Refactored JACK- and ASIO-related build logic.

Context: Fixes an issue?

Related: #2417

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

No

Status of this Pull Request

  • Windows ASIO build takes ASIO build branch? Yes
  • Windows JACK build takes JACK build branch? Yes
  • Windows JACK build no longer downloads ASIO? Yes (see latest master for contrast)
  • Mac CoreAudio build takes CoreAudio build branch? Yes
  • Mac JACK build takes JACK build branch? @softins?

What is missing until this pull request can be merged?

Reviews.

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

@hoffie hoffie added this to the Release 3.9.0 milestone Mar 1, 2022
@hoffie hoffie mentioned this pull request Mar 1, 2022
15 tasks
Jamulus.pro Show resolved Hide resolved
@hoffie hoffie force-pushed the jack-asio-improvements branch from e1c9fa8 to 364716a Compare March 2, 2022 07:19
Copy link
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.

Thanks - hoping for a green build...

@ann0see ann0see merged commit 02be9dd into jamulussoftware:master Mar 2, 2022
@hoffie hoffie deleted the jack-asio-improvements branch March 19, 2022 21:23
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.

3 participants