Skip to content

Export ossia::nodes::automation and ossia::curve - #967

Merged
jcelerier merged 1 commit into
masterfrom
bugfix/export-automation-curve
Oct 27, 2026
Merged

jcelerier merged 1 commit into
masterfrom
bugfix/export-automation-curve

Conversation

@jcelerier

Copy link
Copy Markdown
Member

Both are header-only, so with -fvisibility=hidden every shared object that uses them gets its own local typeinfo. A dynamic_cast from graph_node / curve_abstract in another shared object compares typeinfo by address on ELF and fails. This shows up in score's unit tests (test_unit_curve_edition, test_unit_scriptable_controls) on a shared-plugin build, which read the automation node and the curve its executor built.

Exports them the same way as their header-only siblings simple_mapping and domain_base. No effect on static builds (the macro expands to nothing there).

Needed by ossia/score's bugfix/shader-ledger PR, which bumps the submodule to this commit.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JeL19ofVgPe9u9q7J1V9ex

Both are defined in headers only, so with -fvisibility=hidden every
shared object that uses them gets its own local typeinfo. A
dynamic_cast from graph_node / curve_abstract in another shared object
(score's unit tests reading the automation node and curve its executor
built) compares typeinfo by address on ELF and fails. Export them like
their header-only siblings simple_mapping and domain_base.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JeL19ofVgPe9u9q7J1V9ex
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 26, 2026 22:56 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 26, 2026 22:56 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 26, 2026 22:56 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 26, 2026 22:56 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 26, 2026 22:56 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 26, 2026 22:56 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 26, 2026 22:56 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 26, 2026 22:56 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 26, 2026 22:56 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 26, 2026 22:56 — with GitHub Actions Inactive
@jcelerier
jcelerier merged commit dd8e435 into master Oct 27, 2026
45 of 47 checks passed

This branch was previously deployed

1 inactive deployment
Apple Certificate — 32b92884 Deployed Oct 26, 2026 by jcelerier via CPP (macOS) #1284
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