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

Update CI workflow #84

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Update CI workflow #84

wants to merge 9 commits into from

Conversation

norihiro
Copy link
Owner

Description

This PR includes these changes.

  • cmake: Move generated files under build directory
  • .github(macOS): Fix condition to notarize and deprecated set-output
  • .github(Fedora): Update RPM build flow for Fedora 38, 39, and 40
  • .github: Fix parameter for chown
  • Implement in-tree build
  • ci/macos: Remove unused files

How Has This Been Tested?

CI will test.

General checklist

  • The commit is reviewed by yourself.
  • The code is tested.
  • Document is up to date or not necessary to be changed.
  • The commit is compatible with repository's license.

Also fixes `PKG_SUFFIX` was not set correctly by `option`.
The notarytool should be skipped if the necessary variables are not set.
Also fixed the deprecated `set-output` command in the workflow file.
The versions 36 and 37 were no longer supported.
This commit also adds a script to generate the release-version specific
files so that it's easier to maintain the target release versions.
The package name should have the prefix `obs-studio-plugin-`.
Now the latest obs-studio is available from Fedora Project. It is not
necessary to install obs-studio from RPM Fusion.
- Replaces CMAKE_PROJECT_NAME with PROJECT_NAME.
- Replaces CMAKE_PROJECT_VERSION with PROJECT_VERSION.
- Replaces `OBS::obs-frontend-api` with `OBS::frontend-api`, which looks
  the standard naming in obs-studio repository.
- Adds a condition that checks CMAKE_SOURCE_DIR.
  - Avoids finding packages such as libobs and obs-frontend-api.
  - Does not create installer files.
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.

None yet

1 participant