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: expand org-babel-async insert async insert form #88

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vidbina
Copy link

@vidbina vidbina commented Jun 26, 2022

Bumped into the following issue:

error in process sentinel: async-handle-result: Symbol’s value as variable is void: org-babel-async-content

I reckon that the symbol is unknown in the async process since binding of a value to the symbol happens outside of the lambdas. Within the definition of ob-async-org-babel-execute-src-block, most let-bound symbols are resolved using substitution (e.g.: ,SYMBOL-NAME) to ensure that the values known to the caller are expanded into the forms in the callees (async handlers/lambdas).

Very curious to have more guidance on how to resolve the issue in a "cleaner" manner in case I'm approaching "patching this" in a suboptimal manner.

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