Files
Dario LipicarandClaude Opus 5 bc72ce3902 fix(deps): restore the view teardown hook, and pick up the deduplicated qt-sdk (#214)
* fix(deps): bump logos-view-module so generated view glue regains its teardown hook

#211 switched ui_qml glue generation from logos-qt-generator to
logos-view-generator, but the logos-view-module pin (1f95a75) PREDATES the
reconciliation that gave that emitter the module teardown hook. Measured on the
two revisions:

                                    emitter          context header
  1f95a75 (pinned until now)        0 aboutToUnload  0 maybeUiPluginAboutToUnload
  d6c8885 (this bump)               4 lines          2

So since #211 every view plugin built through this repo has silently lost
aboutToUnload(). It is silent by construction: a generated plugin missing the
hook builds, loads and runs, it is just never asked to finish. Nothing in the
build says a word -- which is the exact failure mode that motivated moving the
emitter and its header into ONE repo under ONE pin in the first place.

The bump also brings logos-view-generator's `--backend` validation: it now
refuses an unrecognised value instead of ignoring the flag, which
lib/modulePreConfigure.nix already documents and depends on.

Verified: nix build .#checks.aarch64-darwin.qml-integration passes on the bump.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

* chore(deps): pick up the qt-sdk that no longer ships logos_ui_plugin_context.h

logos-qt-sdk#42 deleted its copy of the header (and the logos_qt_provider target
that existed only for it), leaving logos-view-module the sole owner. Two
LogosModule.cmake comments described the old world and are corrected.

The BEFORE-ordering of LOGOS_VIEW_INCLUDE_DIR stays. It is no longer what
decides which copy wins -- there is only one copy now -- but this repo pins the
two repos independently, and an older qt-sdk pin (a rollback, a branch, a
consumer overriding the input) brings the duplicate straight back. Belt-and-
braces now, load-bearing again the moment those pins disagree.

Verified with the view-module bump in the same branch:
  nix build .#checks.aarch64-darwin.qml-integration   PASS
  nix build .#checks.aarch64-darwin.qt-host-repoint   PASS

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
2026-08-22 21:02:03 -03:00
..