<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<p>Greg,<br>
</p>
<span style="white-space: pre-wrap">
</span>
<blockquote type="cite" cite="mid:rmiwmkk60sl.fsf@s1.lexort.com">
<pre class="moz-quote-pre" wrap="">
2) The tests all fail because they have #! lines with /bin/bash. </pre>
</blockquote>
Should be addressed per <a class="moz-txt-link-freetext" href="https://github.com/OSGeo/shapelib/pull/150">https://github.com/OSGeo/shapelib/pull/150</a><br>
<blockquote type="cite" cite="mid:rmiwmkk60sl.fsf@s1.lexort.com">
<pre class="moz-quote-pre" wrap="">
could document a dependency on bash, and then find bash with configure
and substitute it, that all seems quite avoidable as I don't see
anything in the test scripts that really need non-POSIX sh features.
With the bash problem worked around, most tests pass, except
FAIL: tests/shpproj.sh
======================
../test-driver: ./tests/shpproj.sh: not found
FAIL tests/shpproj.sh (exit status: 127)
and I don't see that in the tarball, so I'm guessing that isn't a
problem only I am seeing.
shapelib doesn't depend on proj, so perhaps there needs to be
documentation of tests dependencies?</pre>
</blockquote>
I don't reproduce that error (shpproj.sh used to test a shpproj.c
utility that was removed during the PROJ6 transition, so this is a
remains of a more complete tests, but it has the virtue of testing a
few of the contrib/ utilities that have no tests otherwise)<br>
<blockquote type="cite" cite="mid:rmiwmkk60sl.fsf@s1.lexort.com">
<pre class="moz-quote-pre" wrap="">
4) I undid the kludge in 1, and I get a failure to build, because make
fires the rule for one of the programs first, and libshp.la isn't built
yet. Running again with gmake instead, it builds. So there's something
wrong, but I suspect it's wiser to work around this by documenting that
GNU Make is required.</pre>
</blockquote>
The automake/autoconf build system doesn't not receive much love
this days (but should not be worse or better that it used to be).
Perhaps we should remove it at some point.<br>
<blockquote type="cite" cite="mid:rmiwmkk60sl.fsf@s1.lexort.com">
<pre class="moz-quote-pre" wrap="">
5) There are a surprisingly large number of programs in bin. The pkgsrc
package didn't used to install them, but they look like that likely
won't conflict with other pacakges.. Do you think users of shapelib
expect all of these to be installed?</pre>
</blockquote>
<p>No opinion here. I hardly use myself any of the shapelib utils. I
see Debian ships all of them
(<a class="moz-txt-link-freetext" href="https://packages.debian.org/sid/amd64/shapelib/filelist">https://packages.debian.org/sid/amd64/shapelib/filelist</a>), but
some of them come from the contrib/ area (which doesn't
necessarily means they are worse qualities than other non-contrib
utilities). I came much later to the party, so I don't have much
historical background to provide.</p>
<p>Nothing of the above seems to be 1.6.1 specific, so I don't think
it deserves a new RC</p>
<p>Even<br>
</p>
<span style="white-space: pre-wrap">
</span>
<pre class="moz-signature" cols="72">--
<a class="moz-txt-link-freetext" href="http://www.spatialys.com">http://www.spatialys.com</a>
My software is free, but my time generally not.</pre>
</body>
</html>