Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix later_api.h is newer than installed later #205

Closed
wants to merge 3 commits into from
Closed

Conversation

shikokuchuo
Copy link
Collaborator

This somehow errors with:

Error in .Internal(dyn.load(x, as.logical(local), as.logical(now), "")) : 
  function 'execLaterFdNative' not provided by package 'later'

after going through the steps here: #204 (comment).

@jcheng5 @wch

@jcheng5
Copy link
Member

jcheng5 commented Nov 27, 2024

Well, you did better than Claude 3.5 Sonnet did--it hallucinated an R_GetErrorHandler and R_SetErrorHandler API--which would've been really cool! 😞

@shikokuchuo
Copy link
Collaborator Author

I haven't found the bug... or why yours seems to work when it's fundamentally the same structure!

@shikokuchuo
Copy link
Collaborator Author

Same error here: https://github.com/r-lib/later/actions/runs/12058747723/job/33626041239?pr=205#step:6:16
But it clearly does work in normal circumstances... there's obviously something I'm missing.

@shikokuchuo
Copy link
Collaborator Author

With the help of https://github.com/jcheng5/laterfdtest I can see now that it is working, and it's simply the error message that's being printed by R_ToplevelExec(), as Joe hinted at all along.

Porting across the helpful message dummy function, that now prints when expected, but after the first strange message. Unfortunately this seems unavoidable. To solve this we'd have to temporarily divert stderr.

@shikokuchuo
Copy link
Collaborator Author

This is superseded by the merged #204. But noting that it is possible to implement this in future - if somehow the trouble of diverting the stderr output from R_ToplevelExec() becomes the lesser evil.

@shikokuchuo shikokuchuo deleted the fix-later-api branch November 28, 2024 15:56
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.

2 participants