Skip to content
Snippets Groups Projects

Move pressure-vessel-adverb subprocess supervision into common code

Merged Simon McVittie requested to merge wip/smcv/into-library2 into main

Continuation of !671 (merged) and !672 (merged), could be useful for steamrt/tasks#387.

  • !671 (merged)

  • !672 (merged)

  • pv-adverb: Move low-level subprocess management into common code

    There's no reason why this particularly needs to be pressure-vessel-specific, and since commit 7627197e "steam-runtime-tools: Separate out modules not in the shared library" we can share code between pressure-vessel/ and bin/ without enlarging the shared library.

  • pv-adverb: Factor out more subprocess management as SrtProcessManager

    No functional change intended other than a few debug messages, but this will let us recycle this code for a new steam-runtime-supervisor which does not depend on pressure-vessel-specific code and can easily be included in the scout runtime.

    Helps: steamrt/tasks#387

Edited by Simon McVittie

Merge request reports

Pipeline #78350 passed

Pipeline passed for 680280c5 on wip/smcv/into-library2

Approval is optional

Merged by Ludovico de NittisLudovico de Nittis 1 year ago (Feb 21, 2024 10:40am UTC)

Merge details

  • Changes merged into with 0c9e8244.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading