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

Patching synctex misses input or includes #3

Closed
harmsk opened this issue Oct 11, 2016 · 1 comment
Closed

Patching synctex misses input or includes #3

harmsk opened this issue Oct 11, 2016 · 1 comment

Comments

@harmsk
Copy link

harmsk commented Oct 11, 2016

patchSynctex() does not patch Rnw files included with \input{} or \include{} in the main Rnw file.

Working example: knitr-with-input.tar.gz

make rRestore
make
@EmmanuelCharpentier
Copy link
Owner

  1. patchSynctex doesn't patch the .Rnw files ; it patches the corresponding .synctex[.gz] files...

  2. Indeed, this will have trouble with included parts (the "right way" to do this is to use children chunks, see the docs).

But Yihui Xie has decided not to support concordance with these children chunks. See his comments in the issues of his github repository. There is little one can do without forking knitr. And I won't do it (this would require much more of my time...).

Given the current complexity of knitr, its wide non-\LaTeX targe audience (mardown seems dominant these days) and the narrowness of this potential feature, I can understand Yihui Xie's decision...

So, unless you have new ideas about this old thorn in my flank, I close this issue.

harmsk added a commit to harmsk/patchSynctex that referenced this issue Oct 12, 2016
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 a pull request may close this issue.

2 participants