<div dir="ltr"><div><div>sundials-config was removed in sundials 2.6, so I'd just remove the references to it. There's a patch <a href="https://github.com/aragilar/debian-packaging-sundials/blob/master/debian/patches/Add-pkgconfig-files.patch">https://github.com/aragilar/debian-packaging-sundials/blob/master/debian/patches/Add-pkgconfig-files.patch</a> which adds pkg-config support, as otherwise you have to know what the correct paths are.<br><br>Questions about the current packaging: Why is 0001-Added-missing-linkages-to-the-DSOs.patch needed, shouldn't the necessary linking already be set up? Also, what's the need to override dh_install for the docs (see <a href="https://anonscm.debian.org/cgit/debian-science/packages/sundials.git/tree/debian/rules#n29">https://anonscm.debian.org/cgit/debian-science/packages/sundials.git/tree/debian/rules#n29</a>), shouldn't it just work?<br><br></div>Do we want to rename parallel to mpi in the packaging, to reduce confusion (as there is now two other parallel methods)?<br></div>Also, do we want to keep the current "tests", given 1) unrelieable, and 2) only cover the C serial version (the Fortran and parallel versions appear to be commented out, presumably because they keep failing)? Wouldn't be better to write a simple set of tests which solve (for example) the simple harmonic oscillator, and which run against the different features sundials has (c.f. <a href="https://github.com/bmcage/odes/tree/master/scikits/odes/tests">https://github.com/bmcage/odes/tree/master/scikits/odes/tests</a>)? I can do that when I get back home.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 6 February 2017 at 09:51, Dima Kogan <span dir="ltr"><<a href="mailto:dima@secretsauce.net" target="_blank">dima@secretsauce.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">James Clarke <<a href="mailto:jrtc27@debian.org">jrtc27@debian.org</a>> writes:<br>
<br>
>> dh_install: Cannot find (any matches for) "bin/sundials-config" (tried in "." and "debian/tmp")<br>
<br>
</span>bin/sundials-config isn't something that upstream builds anymore, I<br>
don't think. Removing this from the appropriate install makes it go much<br>
further.<br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Don't send me files in proprietary formats (.doc(x), .xls, .ppt etc.). It isn't good enough for <a href="http://opendotdotdot.blogspot.com/2010/04/rms-and-tim-berners-lee-separated-at.html" target="_blank">Tim Berners-Lee</a>, and it isn't good enough for me either. For more information visit <a href="http://www.gnu.org/philosophy/no-word-attachments.html" target="_blank">http://www.gnu.org/philosophy/no-word-attachments.html</a>.<br><br>Truly great madness cannot be achieved without significant intelligence.<br> - Henrik Tikkanen<br><br>If you're not messing with your sanity, you're not having fun.<br> - James Tocknell<br><br>In theory, there is no difference between theory and practice; In practice, there is.</div>
</div>