Skip to content

ci: also build in/against ossia score (dev / SDK / JIT)#3

Merged
jcelerier merged 3 commits into
mainfrom
add-score-ci
Jun 17, 2026
Merged

ci: also build in/against ossia score (dev / SDK / JIT)#3
jcelerier merged 3 commits into
mainfrom
add-score-ci

Conversation

@jcelerier

Copy link
Copy Markdown
Member

Summary

  • Adds a score job calling ossia/actions/.github/workflows/score-addon.yml@master so the template is also validated as an ossia score add-on — the dev (in-tree score build), SDK (latest + continuous score SDK), and JIT tracks.
  • release: false — the templates ship no release.sh.
  • Renames the existing standalone back-end job cistandalone for clarity; its back-end toggles are unchanged.

This brings the templates to the same CI shape as a ported avendish-based score addon: score + standalone + portability.

🤖 Generated with Claude Code

jcelerier and others added 3 commits June 17, 2026 05:16
Add a `score` job calling the score-addon.yml reusable workflow alongside the
standalone back-end build, so the template is validated as an ossia score add-on
too (dev tree + SDK + JIT tracks). release left off: the templates ship no
release.sh. Renames the existing standalone job ci -> standalone for clarity.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
In an in-tree score build the bootstrap generator emits a Plugin_QtInterface
class named after the addon NAME and bakes in VERSION. NAME equalled the avnd
object class, so the generated class redefined it; and VERSION 1.0.0 was emitted
as an invalid C++ float literal. Rename NAME (keeping the object CLASS and the
standalone project()/artifact name) and use an integer VERSION.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…header

- Set CMAKE_MSVC_RUNTIME_LIBRARY at the top of the addon so the object
  library and every back-end external share the /MT runtime the Max
  external needs (mixing /MT and /MD trips MSVC LNK2038).
- Include UI.hpp from the class header so <Class>::ui is a complete type
  for the score plugin generator (it includes the class header, not the
  MAIN_FILE, and needs the full ui to build the process layer).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@jcelerier jcelerier merged commit ad2eca5 into main Jun 17, 2026
43 of 58 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