-
Notifications
You must be signed in to change notification settings - Fork 82
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
[latexindent.exe] manipulateSentences cannot be 1 in versions after 3.22 #514
Comments
What version of perl are you using? What's your operating system? |
I am using the standalone latexindent.exe on Windows 11. This issue is general for any tex files if |
I can confirm this is a bug, demonstrated in github actions log at https://github.com/cmhughes/latexindent.pl/actions/runs/7863164391/job/21453471349 I'll look into it. |
Similarly, cat test.tex |latexindent --modifylinebreaks --yaml="modifyLineBreaks:oneSentencePerLine:manipulateSentences:1" for latexindent 3.23.6 yields Variable length lookbehind not implemented in regex m/(?x) # ignore spaces in the below
(?: #
\.\) .../ at /usr/local/texlive/2023/texmf-dist/scripts/latexindent/LatexIndent/Sentence.pm line 160, <> line 34. with exit code 128 |
Is this just a question of a recent perl version (> 5.26 in this case)? As far as I can see, for example, texlive does not ship a perl executable, so that this depends on a recent version on the system. |
In version 3.23.8 the error persists, but I suspect this is rather due to an insufficiently recent Perl version? |
There's a mention that is developed on 5.38 but it is not clear if that's the minimal requirement (Opensuse ships with 5.26, maybe cause of this failure). |
Yes, this is expected, V3.23.8 wasn't designed to fix this issue. I'm hoping to get to this as my next latexindent priority. |
Okay, thank you very much for the clarification. It was somewhat in a limbo since a commit addressed this issue, though it's still open |
Apologies for the confusion, I can see how/why that happened. |
Thanks for this. I believe that, as of 55306bc I've fixed this. The problem was that I had specified an old version of Perl (5.32) in the routine that creates You can see an output of the tests in github actions at https://github.com/cmhughes/latexindent.pl/actions/runs/8582320265/job/23520314375 (this log may expire at some point). This will be part of the next release, please leave this issue open until I've made the release. Thanks for reporting. |
Released at https://github.com/cmhughes/latexindent.pl/releases/tag/V3.23.9, uploaded to ctan |
Please provide the following when posting an issue:
original .tex code
yaml settings
The file is indented correctly. But there are logs in the terminal as follows
If I disable
manipulateSentences: 0
, there is not log. If I remove the modification added in #447, there is also no log.The text was updated successfully, but these errors were encountered: