<div dir="ltr">I'll take a look at those along with an apparent pkgconfig bug.</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature">—Mark<br>_______________________<br>Mark E. Anderson <<a href="mailto:emer@emer.net" target="_blank">emer@emer.net</a>><br></div></div>
<br><div class="gmail_quote">On Tue, Mar 13, 2018 at 11:58 PM, Ryan Schmidt <span dir="ltr"><<a href="mailto:ryandesign@macports.org" target="_blank">ryandesign@macports.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
On Mar 13, 2018, at 21:32, Mark Anderson wrote:<br>
<br>
> Perry E. Metzger (pmetzger) pushed a commit to branch master<br>
> in repository macports-ports.<br>
><br>
><br>
> <a href="https://github.com/macports/macports-ports/commit/1b655b8f6dae92268712ca6ff6900ab60008f53e" rel="noreferrer" target="_blank">https://github.com/macports/<wbr>macports-ports/commit/<wbr>1b655b8f6dae92268712ca6ff6900a<wbr>b60008f53e</a><br>
><br>
> The following commit(s) were added to refs/heads/master by this push:<br>
><br>
>      new 1b655b8  Update Tesseract to 03.05.01<br>
><br>
> 1b655b8 is described below<br>
><br>
><br>
> commit 1b655b8f6dae92268712ca6ff6900a<wbr>b60008f53e<br>
><br>
> Author: Mark Anderson<br>
> AuthorDate: Tue Mar 13 20:43:00 2018 -0400<br>
><br>
><br>
>     Update Tesseract to 03.05.01<br>
><br>
><br>
><br>
>     * Update Tesseract to 03.05.01<br>
><br>
>     * Enable building with CMake<br>
><br>
><br>
><br>
>     Fixes: <a href="https://trac.macports.org/ticket/54006" rel="noreferrer" target="_blank">https://trac.macports.org/<wbr>ticket/54006</a><br>
><br>
>     Closes: <a href="https://trac.macports.org/ticket/50128" rel="noreferrer" target="_blank">https://trac.macports.org/<wbr>ticket/50128</a><br>
><br>
>     Closes: <a href="https://trac.macports.org/ticket/50118" rel="noreferrer" target="_blank">https://trac.macports.org/<wbr>ticket/50118</a><br>
><br>
> ---<br>
>  textproc/tesseract/Portfile | 10 +++++-----<br>
>  1 file changed, 5 insertions(+), 5 deletions(-)<br>
<br>
It fails to build on the buildbot:<br>
<br>
CMake Error at /opt/local/share/cmake-3.10/<wbr>Modules/<wbr>FindPackageHandleStandardArgs.<wbr>cmake:137 (message):<br>
  Could NOT find PkgConfig (missing: PKG_CONFIG_EXECUTABLE)<br>
<br>
<br>
On my system, this warning is printed at destroot and should be fixed:<br>
<br>
--->  Staging tesseract into destroot<br>
Warning: violation by /opt/local/cmake<br>
Warning: tesseract violates the layout of the ports-filesystems!<br>
Warning: Please fix or indicate this misbehavior (if it is intended), it will be an error in future releases!<br>
<br>
The cmake-1.1 portgroup defines this variable which is presumably where it thinks those files should be installed:<br>
<br>
# standard place to install extra CMake modules<br>
set cmake_share_module_dir ${prefix}/share/cmake/Modules<br>
<br>
However, it looks like most ports actually install their cmake files into /opt/local/lib/cmake. I don't know if maybe the value in the portgroup is outdated.<br>
<br>
<br>
<br>
> @@ -3,8 +3,9 @@<br>
>  PortSystem          1.0<br>
><br>
>  PortGroup           github 1.0<br>
> +PortGroup           cmake 1.1<br>
<br>
This affects all subports too, and they don't need cmake. So this portgroup inclusion should be moved to inside the `if {${name} eq ${subport}}` block below.<br>
<br>
<br>
> @@ -23,8 +24,8 @@ long_description    The Tesseract OCR engine was one of the top  3  engines in \<br>
><br>
>  github.master_sites ${github.homepage}/archive/${<wbr>git.branch}<br>
<br>
The next time this port's version is updated, please remove this master_sites override and use the portgroup's defaults. (But don't change it now, because that would change the port from archive to tarball downloads, and the checksums wouldn't match.)<br>
<br>
<br>
> @@ -32,14 +33,13 @@ if {${name} eq ${subport}} {<br>
>                              port:leptonica \<br>
>                              port:jpeg<br>
><br>
> -    configure.env-append    LIBLEPT_HEADERSDIR=${prefix}/<wbr>include/leptonica<br>
> +    cmake.out_of_source     yes<br>
<br>
This is the default for the cmake-1.1 portgroup and doesn't need to be specified.<br>
<br>
</blockquote></div><br></div>