[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