On Tue, Jan 10, 2023 at 11:46:08AM +0100, Jean-Marc Lasgouttes wrote:
> Le 10/01/2023 à 03:42, Scott Kostyshak a écrit :
> > The attached patch works around an AMS book unicode issue, as discussed 
> > here:
> > 
> >    
> > https://tex.stackexchange.com/questions/671116/how-to-find-and-fix-the-origin-of-invalid-unicode-character-generated-by-lyx
> > 
> > Disadvantage of the workaround is the added dependency, textcase.sty, which 
> > is not needed for AMS book documents without unicode. If this is a problem, 
> > I suppose we could add a new layout flag that, if activated, loads textcase 
> > if there is unicode.
> > 
> > Attached is an example .lyx file that does not compile without the patch 
> > and does compile with the patch.
> > 
> > Thoughts?
> 
> Since this is not needed with latest LaTeX,

The workaround is not needed? I think it is needed until AMS book is
updated (which is not clean if/when it will). For example, I have an
updated TeX Live 2022 and the workaround is needed.

> I'd suggest maybe a "Requires
> makeuppercase" that triggers code checking whether textcase.sty may be
> needed. (we can do that, right?)

I can look into that. You mean the goal is that on older LaTeX
distributions, the workaround may not be needed?

> Is this only an amsbook issue? What about book.cls?

I do not think book.cls needs the workaround. When I change the example
to Book (Standard), it compiles fine without the workaround.

> > By the way, should the "amsbook" dependency in \DeclareLaTeXClass be there 
> > regardless of this patch?
> 
> AFAIR, this is the default when no dependency is given (require
> layoutname.cls)

Ah makes sense.

Thanks,
Scott

Attachment: signature.asc
Description: PGP signature

-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel

Reply via email to