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

We need to update the OBS dependency to 29. #360

Merged
merged 2 commits into from
Jun 30, 2023
Merged

We need to update the OBS dependency to 29. #360

merged 2 commits into from
Jun 30, 2023

Conversation

umireon
Copy link
Member

@umireon umireon commented Jun 30, 2023

I finally finally concluded that we have to use OBS 29 with the new build system and Qt6.
It is nearly impossible to build both with Qt6 and with OBS 28.

See #359

I think our plugin may work with OBS 28 when the build dependency is OBS 29.

@umireon umireon requested a review from royshil June 30, 2023 13:33
@umireon
Copy link
Member Author

umireon commented Jun 30, 2023

I cannot install OBS 28 on my machine because all the OBS distribution is already OBS 29.
I think this means OBS 28 is already legacy and we have to advise users to update to OBS 29+.

@umireon
Copy link
Member Author

umireon commented Jun 30, 2023

I cannot spare any more time to investigate to make this plugin buildable with OBS 28 binaries.

@umireon
Copy link
Member Author

umireon commented Jun 30, 2023

We can still provide an option that OBS 28 users can use older versions of this plugin.

Copy link
Collaborator

@royshil royshil left a comment

Choose a reason for hiding this comment

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

OK!
no problem. we have older versions that work with v28

@umireon umireon merged commit 39ce395 into main Jun 30, 2023
@umireon umireon deleted the umireon.29.1.2 branch June 30, 2023 14:21
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.

2 participants