r-lib/processx

`$call` should be a call or `NULL` in error objects

gaborcsardi opened this issue · 3 comments

❯ processx::run("false")
Error in `processx::run("false")`:
! System command 'false' failed
---
Exit status: 1
Stderr: <empty>
---
Type .Last.error to see the more details.

❯ .Last.error$call
[1] "processx::run(\"false\")"

So we should use another field for the call string.

Or maybe just zap the srcrefs if that is the problem?

It is not the only problem, calls may contain large objects.

Supplying a string can be generally useful for non-call contexts. Should we use the field $context? We've also started namespacing fields inside $rlang for $rlang$inherit, I guess that'd be a good idea here too.