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

Merged
merged 11 commits into from
Jul 15, 2024
Merged

Update CI workflow #84

merged 11 commits into from
Jul 15, 2024

Conversation

norihiro
Copy link
Owner

@norihiro norihiro commented May 14, 2024

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.

norihiro added 5 commits July 15, 2024 23:18
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.
norihiro added 5 commits July 15, 2024 23:19
Also removes the build flow for Fedora 38.
- 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.
@norihiro
Copy link
Owner Author

Tested on these two environments.

  • macOS on Apple Silicon
  • Fedora 39

@norihiro norihiro merged commit 325faa0 into main Jul 15, 2024
22 checks passed
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.

1 participant