Skip to content

Avoid non-API R lookup symbols under R-devel#1894

Merged
t-kalinowski merged 3 commits intomainfrom
fix/r-devel-c-api-lookups
Apr 7, 2026
Merged

Avoid non-API R lookup symbols under R-devel#1894
t-kalinowski merged 3 commits intomainfrom
fix/r-devel-c-api-lookups

Conversation

@t-kalinowski
Copy link
Copy Markdown
Member

Summary

Fix the R CMD check note on R-devel by removing compiled-code usage of R_NamespaceRegistry and R_UnboundValue.

Public-facing changes

There are no intended public-facing changes.

Internal changes

  • replace direct namespace lookup with R_FindNamespace()
  • replace lazy refenv lookups that used R_UnboundValue with nullable R_getVarEx(..., FALSE, NULL) lookups
  • cache reticulate namespace bindings during init, including r_convert_dataframe_column and py_resolve_module_proxy
  • add a compiled-code regression test that asserts the built library does not import R_NamespaceRegistry or R_UnboundValue

@t-kalinowski t-kalinowski merged commit e72dea6 into main Apr 7, 2026
13 of 14 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