Skip to content

Conversation

@ann0see
Copy link
Member

@ann0see ann0see commented Mar 12, 2022

Short description of changes
Basically fd7bfe6 split out as separate commit.

CHANGELOG: Windows: The JACK build now no longer gives an ASIO related warning message on an incompatible state

Context: Fixes an issue?
No.

Does this change need documentation? What needs to be documented and how?
No. Bug Fix

Status of this Pull Request
Not yet tested, just ported

What is missing until this pull request can be merged?
Testing and review.

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 12, 2022
Copy link
Member

@hoffie hoffie left a comment

Choose a reason for hiding this comment

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

Basically fd7bfe6 split out as separate commit.

That's matches the commit [edit]content[/edit] this PR contains. You might have meant another one?

Approving solely on the fact that the previous logic simply does not look correct for non-ASIO builds.

@ann0see
Copy link
Member Author

ann0see commented Mar 12, 2022

Yes. Basically it's has the same diff (I just copy and pasted it and didn't cherry-pick). Probably that means it's the same commit?

@ann0see
Copy link
Member Author

ann0see commented Mar 12, 2022

You might have meant another one?

No. I wanted to get the ( WITH_JACK ) into upstream.

Just for my own clarification:
Basically a commit is uniquely determined by the diff?

@hoffie
Copy link
Member

hoffie commented Mar 12, 2022

You might have meant another one?

No. I wanted to get the ( WITH_JACK ) into upstream.

Ok.

Just for my own clarification: Basically a commit is uniquely determined by the diff?

No, sorry for the confusion. I misread the commit IDs. They are not the same. The commit hash is determined by commit metadata (author, date, message, ...) and commit content (a tree object which describes the full content of the repo).

@ann0see
Copy link
Member Author

ann0see commented Mar 12, 2022

Ok. Thanks!

Basically it is a split out of #2168

@ann0see ann0see merged commit db127d8 into jamulussoftware:master Mar 13, 2022
@ann0see ann0see deleted the patch/noJACKWarn branch March 13, 2022 10:25
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