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

Update jlcode to v6.1 #29

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update jlcode to v6.1 #29

wants to merge 2 commits into from

Conversation

blegat
Copy link
Contributor

@blegat blegat commented Apr 26, 2022

@wg030 this does not seem to work out of the box with the new version of jlcode, what should be the recommended way to use jlcode v6.0 for the JuliaCon proceedings ?

jlcode

@wg030
Copy link

wg030 commented Apr 26, 2022

This seems to be related to wg030/jlcode#43
At least the output looks very similiar.

@wg030
Copy link

wg030 commented May 3, 2022

@blegat
I just updated the jlcode package and fixed all latest issues, especially the one that likely produced this output.

what should be the recommended way to use jlcode v6.0 for the JuliaCon proceedings ?

so for the JuliaCon procceedings I would recommend the following now:

  • update to the latest version v6.1
  • load the package with the package option charsperline=x choosing the largest integer value for x that fits to your template without causing overfull boxes (some quick testing might be necessary)
  • compile your code with luatex or xetex (pdftex is also possible, but might cause more problems as far as special unicode characters are concerned)
  • insert code ONLY with the commands \jlinputlisting, \begin{jllisting} or \jlinl as shown in the manual of the jlcode package!
  • DO NOT use \lstinputlisting, \begin{lstlisting} or \lstinline to insert code (the length of the codebox is not calculated with these commands)!
  • if something is not displayed in the correct color, try to fix it manually with one of the commands \addlitjlbase, \addlitjlstrnum, \addlitjlmacros or \addlitjlfunctions as shown in the manual of the jlcode package

@blegat
Copy link
Contributor Author

blegat commented May 20, 2022

@wg030 Thanks for the detailed recommendations, I updated to v6.1 and the ouptut with 60 chars is:
60chars
Looks good to merge on my side.

@blegat blegat marked this pull request as ready for review May 20, 2022 09:43
@wg030
Copy link

wg030 commented May 20, 2022

@wg030 Thanks for the detailed recommendations

You are very welcome!

Looks good to merge on my side.

Yes, indeed, that's a nice output!

@blegat blegat changed the title Update jlcode to v6.0 Update jlcode to v6.1 May 20, 2022
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