[MacPorts] #51818: New release of LaTeXML
MacPorts
noreply at macports.org
Sun Sep 11 18:26:51 CEST 2016
#51818: New release of LaTeXML
-----------------------------+---------------------------------
Reporter: bruce.miller@… | Owner: macports-tickets@…
Type: update | Status: new
Priority: Normal | Milestone:
Component: ports | Version:
Resolution: | Keywords: maintainer haspatch
Port: LaTeXML |
-----------------------------+---------------------------------
Changes (by raimue@…):
* cc: raimue@… (added)
* keywords: => maintainer haspatch
Comment:
The files are not removed as they were not properly installed and
registered by MacPorts. The destroot phase is putting them into
`${prefix}` instead of `${destroot}${prefix}`. The `post-destroot` block
should be the following (similar for the +mactex variant):
{{{
post-destroot {
xinstall -d ${destroot}${texlive_texmfdist}/tex/latex/latexml
foreach sty [glob ${worksrcpath}/lib/LaTeXML/texmf/*.sty] {
file copy -force ${sty}
${destroot}${texlive_texmfdist}/tex/latex/latexml/
}
}
}}}
Why install the files both to `texmf-local` and `texmf-texlive`? Could we
not just set `TEXMF=${prefix}/share/tex/texmf-texlive`?
{{{
pre-configure {
configure.args-append TEXMF=
}
}}}
Why is this in a `pre-configure` block? This should be at the main level.
All of `latexml.*` are normal variables, do not declare this as options,
for example use `set latexml.found_tex no` and override them the same way
later.
The `post-activate` block at the end of the file should be replaced with
notes, as it was before this patch.
--
Ticket URL: <https://trac.macports.org/ticket/51818#comment:8>
MacPorts <https://www.macports.org/>
Ports system for macOS
More information about the macports-tickets
mailing list