<div dir="ltr">Stupid question :S Applied it in the libdivecomputer directory with -p1<div>Libdivecomputer seems to build correctly. I'm building the entire Subsurface project now and will report back after that.</div><div><br></div><div>Benjamin</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 28 June 2017 at 23:02, Benjamin <span dir="ltr"><<a href="mailto:nystire@gmail.com" target="_blank">nystire@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Where exactly do I apply the patch file? In the Subsurface directory?</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On 28 June 2017 at 22:33, Linus Torvalds <span dir="ltr"><<a href="mailto:torvalds@linux-foundation.org" target="_blank">torvalds@linux-foundation.org</a><wbr>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Wed, Jun 28, 2017 at 9:42 AM, Benjamin <<a href="mailto:nystire@gmail.com" target="_blank">nystire@gmail.com</a>> wrote:<br>
> Good afternoon<br>
> I'm running macos 10.12.5.<br>
> I just synced to the latest sources, and when running the build script, I<br>
> get the following message:<br>
><br>
> Making install in src<br>
><br>
>   CCLD     <a href="http://libdivecomputer.la" rel="noreferrer" target="_blank">libdivecomputer.la</a><br>
><br>
> Undefined symbols for architecture x86_64:<br>
><br>
>   "_crc32", referenced from:<br>
<br>
</span>Interesting. The new Suunto protocol over bluetooth needs crc32() for<br>
stream integrity control, and I was expecting to get it from zlib -<br>
which we require to be there anyway due to other dependencies (eg<br>
libgit uses zlib too).<br>
<br>
An you must have zlib installed *somehow*, since the<br>
<br>
    #include <zlib.h><br>
<br>
in suunto_eonsteel.c worked.<br>
<br>
I wonder what the difference here between macos and Linux is - maybe<br>
it's a clang linker difference, or maybe I got crc32() some other<br>
way..<br>
<br>
Anyway, it definitely is my bug - I think I just added the "-lz" to<br>
the wrong place (I hate autotools, and I'm bad at it).<br>
<br>
I think the attached trivial patch to move the -lz to the right place<br>
will fix it. Can you please test?<br>
<span class="m_7054813236012306911HOEnZb"><font color="#888888"><br>
                 Linus<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>