bug#35780: texlive-latex-polyglossia package empty

bug#35780: texlive-latex-polyglossia package empty

[Josh Holland]

Steps to reproduce:
$ guix build texlive-latex-polyglossia
$ tree /gnu/store/d9nndb33yd22pay1yyg76nb0i56qdmkq-texlive-latex-polyglossia-49435

Expected results:
Something containing the file polyglossia.sty

Actual results:
Only an empty directory share/texmf-dist/tex/latex/polyglossia exists.

bug#35780: [PATCH] attempt to fix polyglossia

[Josh Holland]

---
I had a quick look into fixing this myself, and made a little bit
of progress.  Not being that familiar with how things work yet,
I'm going on what I can guess by looking at gnu/packages/tex.scm,
guix/build-system/texlive.scm and guix/build/texlive-build-system.scm
so I may well be misinterpreting some things.

The source for polyglossia comes in a single file, polyglossia.dtx.
The README embedded within it says to extract the files by running
xetex or luatex on it.  I tried to convince the build system to do
this by this patch, which may or may not have been the right approach.
It does at least get a new error:

Generating file(s) ../README 
! I can't write on file `../README.tex'.

It appears to be trying to write its README file to the directory above
the build directory, which is failing. Is there a way to make this
directory writeable, or to stop the build system from trying to generate
the README there?

 gnu/packages/tex.scm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/tex.scm b/gnu/packages/tex.scm
index c3ce80c8f5..0894148501 100644
--- a/gnu/packages/tex.scm
+++ b/gnu/packages/tex.scm
@@ -3249,7 +3249,8 @@ array environments; verbatim handling; and syntax diagrams.")
                (base32
                 "03ma58z3ypsbp7zgkzb1ylpn2ygr27cxzkf042ns0rif4g8s491f"))))
     (build-system texlive-build-system)
-    (arguments '(#:tex-directory "latex/polyglossia"))
+    (arguments '(#:tex-directory "latex/polyglossia"
+                 #:build-targets '("polyglossia.dtx")))
     (home-page "https://www.ctan.org/pkg/polyglossia")
     (synopsis "Alternative to babel for XeLaTeX and LuaLaTeX")
     (description
-- 
2.21.0

bug#35780: [PATCH] attempt to fix polyglossia

[Ricardo Wurmus]

Hi Josh,

> I had a quick look into fixing this myself, and made a little bit
> of progress.  Not being that familiar with how things work yet,
> I'm going on what I can guess by looking at gnu/packages/tex.scm,
> guix/build-system/texlive.scm and guix/build/texlive-build-system.scm
> so I may well be misinterpreting some things.
>
> The source for polyglossia comes in a single file, polyglossia.dtx.
> The README embedded within it says to extract the files by running
> xetex or luatex on it.  I tried to convince the build system to do
> this by this patch, which may or may not have been the right approach.

Thanks for giving it a try.  The build system works on “ins” files by
default.  I guess it should try “dtx” files as well.

> It does at least get a new error:
>
> Generating file(s) ../README
> ! I can't write on file `../README.tex'.
>
> It appears to be trying to write its README file to the directory above
> the build directory, which is failing. Is there a way to make this
> directory writeable, or to stop the build system from trying to generate
> the README there?

We can patch the dtx file in a build phase and replace “../README” with
just “README”.  We can’t write to the parent of the build directory, and
extracting everything a level deeper seems wrong.

What do you think?

--
Ricardo

bug#35780: [PATCH] attempt to fix polyglossia

[Josh Holland]

Hi,

(CCing guix-devel because I'm confused about this and more eyes can't
hurt)

On Sat, May 18, 2019 at 05:50:13AM +0200, Ricardo Wurmus wrote:
> Thanks for giving it a try.  The build system works on “ins” files by
> default.  I guess it should try “dtx” files as well.

Should this be a patch to the build-system?  Although maybe it's not
that simple: https://ctan.org/texarchive/macros/xetex/latex/polyglossia
lists four files, incuding an "ins" file.

> We can patch the dtx file in a build phase and replace “../README” with
> just “README”.

I tried patching the README path as Ricardo suggested.  This got
somewhere, complaining of missing fonts, so I added texlive-fonts-cm
to native-inputs.  With both these changes, the build now fails while
trying to build example-thai.tex, and the errors appear to indicate
trying to build a LaTeX file with plain TeX:

Processing file polyglossia.dtx (example-thai.tex) -> example-thai.tex
File polyglossia.dtx ended by \endinput.
Lines  processed: 11224
Comments removed: 1600
Comments  passed: 215
Codelines passed: 9209

! Undefined control sequence.
l.209 \documentclass
                  [11pt]{ltxdoc}
! Undefined control sequence.
l.210 \usepackage
               {color}
! Undefined control sequence.
l.211 \usepackage
               {xspace,fancyvrb}
! Undefined control sequence.
l.212 \usepackage
               [neverdecrease]{paralist}
! Undefined control sequence.
l.213 \definecolor
                {myblue}{rgb}{0.02,0.04,0.48}
! Undefined control sequence.
l.214 \definecolor
                {lightblue}{rgb}{0.61,.8,.8}
! Undefined control sequence.
l.215 \definecolor
                {myred}{rgb}{0.65,0.04,0.07}
! Undefined control sequence.

(and more...)

I'm not that familiar with the intricacies of properly building TeXLive
from source, but the difference between the file list on CTAN and what
comes from the Subversion repo via texlive-ref smells fishy to me.  If I
manually run any permutation of  `{xe,lua}tex polyglossia.{ins,dtx}`
it mostly seems to build OK, but to my surprise no polyglossia.sty
is produced!  There's no actual build instructions in the README or the
polyglossia.pdf, so I'm rather lost here.

Thanks,

-- 
Josh Holland

bug#35780: texlive-latex-polyglossia package empty

[Maxim Cournoyer]

Hello,

The texlive-polyglossia is no longer empty, and works, at least with
XeLaTeX.  You can see it in action building the doc of our
'python-ipython-documentation' package.

Closing!

Thanks,

Maxim