-
-
Notifications
You must be signed in to change notification settings - Fork 419
Update with OBS Template #1190
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
Update with OBS Template #1190
Conversation
…requirement byt he plugin (not fully tested yet)
|
This is ready for review / testing. There are a LOT of changes, including changes to default github actions and code formatting process. please check the latest artefacts and try on your system, report any bug that are not in master branch, |
|
This is no longer. WIP and should be tested on the major OS. |
Follow OBS for log entries guideline (non-capitalized) Update with plugin-template code where missing
|
This requires OBS 31. there are some possible clean-up to be pushed as well later with the work on the new config files |
…istroAV into obs_31_min_req_update
|
An Important point on this PR: The obs-plugintemplate moved away from providing & maintaining build script for innosetup outside of the CI environment. This brings us to a choice:
-or-
Planned features could be implemented at:
|
This aim to use the updated OBS Plugin Template.
There are a major lack of documentation about the changes and this process is highly manual.
Goal is:
No other changes should be made as well (no new features / bug fix etc).
There are a bunch of changes that would impact the obs_log but this is too complex and will be dealt in another PR once this is merged.
Impacts:
Windows build process has change, this PR will wipe the tweaks from {PR 1134](Auto remove obs-ndi - Windows #1134) > Could be looked at to be managed by a script. (if deemed relevant still)
MacOS build as well will not have the scripts (postinstall) as well as PR Remove obs-ndi on macos after install of DistroAV #1158 & PR Delete obs-ndi plugin on macos after installing DistroAV #1136
This could be re-implemented via specific PR is relevant.