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

--html-to-org-with-pandoc reports error from pandoc instead of "Pandoc failed". #47

Merged
merged 3 commits into from
Dec 20, 2023

Conversation

c1-g
Copy link
Contributor

@c1-g c1-g commented Nov 26, 2021

Here is the patch that makes the process from pandoc replace the
buffer string with its error message and show it throught the capture
buffer.

This is intended to be used to debug #42.

@alphapapa
Copy link
Owner

Thanks, but this patch would leave the temp file on disk. You should probably use unwind-protect.

@c1-g
Copy link
Contributor Author

c1-g commented Feb 16, 2023

@alphapapa Hi, I added unwind-protect like you requested. Sorry I took so long to reply.

@alphapapa alphapapa force-pushed the verbose-pandoc-error branch from dcae2a5 to 9537d3b Compare December 20, 2023 14:22
@alphapapa alphapapa merged commit abc2c03 into alphapapa:master Dec 20, 2023
0 of 5 checks passed
@alphapapa
Copy link
Owner

Thanks for your patience. I squashed, rebased, updated the changelog, and merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants