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

hardwrap is incompatible with xelatex #12

Closed
godbyk opened this issue Oct 21, 2010 · 6 comments
Closed

hardwrap is incompatible with xelatex #12

godbyk opened this issue Oct 21, 2010 · 6 comments
Labels

Comments

@godbyk
Copy link
Collaborator

godbyk commented Oct 21, 2010

When attempting to compile a document with xelatex, I get the following error:

! Undefined control sequence.
l.48   \ifnum\pdfshellescape
                            >0\relax

We should probably account for this possibility.

@godbyk
Copy link
Collaborator Author

godbyk commented Oct 21, 2010

Per your XeTeX reference, it looks like \shellescape should be used if we're running XeTeX.

@wspr
Copy link
Owner

wspr commented Oct 22, 2010

How strict do you need to be about requiring supplementary packages? We could use the pdftexcmds to provide a wrapper for these, or we could do the more complex

\@ifpackageloaded{pdftexcmds}{\RequirePackage{pdftexcmds}}{%
  % emulate pdftexcmds's wrapper for \pdfshellescape
}

(If we're going to those efforts, we should also add code for the existence of \shellescape and \pdfshellescape in the first place, although I think the latter has been around for long enough we'dn't need to for it.)

What do you think?

@godbyk
Copy link
Collaborator Author

godbyk commented Oct 22, 2010

If pdftexcmds is a fairly standard package (that is, it's supplied in all the major distributions) and it's contemporary to the versions of the distributions our package will appear in, then I see no reason not to use it.

I'm typically all for using existing packages when possible. The only downsides I've seen are that there are sometimes compatibility issues between packages and I've run into problems with requiring packages that are in newer distributions. (Apparently many people install LaTeX once and never update their packages.)

It looks like the pdftexcmds package requires Lua, though. Am I understanding that correctly? If so, it might be best to either check for the existence of the \shellescape primitive or use ifxetex.

Thoughts?

@wspr
Copy link
Owner

wspr commented Oct 22, 2010

If pdftexcmds is a fairly standard package (that is, it's supplied in all the major distributions) and it's contemporary to the versions of the distributions our package will appear in, then I see no reason not to use it.

Hmmm, it's probably in that class of packages for which users would need to upgrade their packages to some extent (Heiko's stuff can be relatively bleeding-edge and interdependent.)

It looks like the pdftexcmds package requires Lua, though. Am I understanding that correctly?

Not quite -- it uses Lua to emulate certain pdftex primitives when running under LuaTeX, but when running under XeTeX it simply provides a consistent interface to the commands that are shared between the engines.

Perhaps we can start with using the package and then add in compatibility for older distribution if/when we feel it's necessary.

@godbyk
Copy link
Collaborator Author

godbyk commented Oct 22, 2010

That sounds like a good idea, then. We'll take the easy route first and write more code later if we actually need to.

@wspr
Copy link
Owner

wspr commented Oct 23, 2010

fix xetex and luatex compilation

I thought XeTeX used input pipes as well, but I
was wrong; simply bypass the kpsewhich query when
not using pdfTeX

Closed by 9b745c6

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

No branches or pull requests

2 participants