Skip to content

Fix resolver stdin handling for --with dependencies - #22

Merged
t-kalinowski merged 1 commit into
mainfrom
locate-dynamic-package-library
Jun 3, 2026
Merged

t-kalinowski merged 1 commit into
mainfrom
locate-dynamic-package-library

Conversation

@t-kalinowski

Copy link
Copy Markdown
Member

Summary

  • Read resolver input from file("stdin") so ir run --with ... reaches the embedded R resolver through Rscript’s real stdin stream.
  • Add a CLI regression that exercises the public ir run path against the real R resolver and verifies a --with package is materialized into the runtime library.
  • This fixes the case where dependency specs were being skipped by the resolver, leaving the dynamic package library empty for user code.

Notes

  • The change is internal-only in the resolver path; the user-facing behavior is that --with packages now resolve and load as expected.

@t-kalinowski
t-kalinowski marked this pull request as ready for review June 3, 2026 21:07
@t-kalinowski
t-kalinowski merged commit 07d4dd2 into main Jun 3, 2026
4 checks passed
@t-kalinowski
t-kalinowski deleted the locate-dynamic-package-library branch July 7, 2026 12:48
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