<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">
Hi Roland
<div class=""><br class="">
<div>
<blockquote type="cite" class="">
<div class="">On 19 Feb 2015, at 22:05, Roland Fehrenbacher <<a href="mailto:rf@q-leap.de" class="">rf@q-leap.de</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<blockquote type="cite" class="">
<blockquote type="cite" class="">
<blockquote type="cite" class="">
<blockquote type="cite" class="">
<blockquote type="cite" class="">"T" == Torbjørn Rognes <<a href="mailto:torognes@ifi.uio.no" class="">torognes@ifi.uio.no</a>> writes:<br class="">
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<br class="">
Hi Torbjørn,<br class="">
<br class="">
   T> Hi Andreas<br class="">
<blockquote type="cite" class="">
<blockquote type="cite" class="">On 18 Feb 2015, at 23:29, Andreas Tille <<a href="mailto:andreas@an3as.eu" class="">andreas@an3as.eu</a>> wrote:<br class="">
<blockquote type="cite" class=""><br class="">
Thank you. We will consider all patches and potentially<br class="">
integrate all of them into a future version of vsearch. As far<br class="">
as I can see now they all seem to be fine. I will inform you<br class="">
when they have been integrated.<br class="">
</blockquote>
<br class="">
Thanks, that would be very helpful.<br class="">
</blockquote>
</blockquote>
<br class="">
   T> We have already integrated the patches related to the manual and<br class="">
   T> the code. What remains are those patches that apply to the<br class="">
   T> Makefile(s). We will handle these together with the other changes<br class="">
   T> that we want to make to the Makefile relating to reading<br class="">
   T> compressed files using the zlib or bzip2 libraries.<br class="">
<br class="">
please write your Makefile such that it will simply use the existing<br class="">
zlib or bzip2 libraries on the system. Bundling the zlib or bzip2 code<br class="">
with your own code makes life more difficult from the packaging point of<br class="">
view (we'll have to create a modified source package without those<br class="">
libs).<br class="">
</div>
</blockquote>
<div><br class="">
</div>
<div>Hm. We were planning to do exactly what you do not recommend. But we might reconsider. We planned to include the source of zlib and bzip2 in the src directory and link their object files directly into vsearch. They seem quite stable.</div>
<div><br class="">
</div>
<div>We have already included the code from Google CityHash (<a href="https://code.google.com/p/cityhash/" class="">https://code.google.com/p/cityhash/</a>).</div>
<div><br class="">
</div>
<div>We think it would make life easier for those who compile and use vsearch, because then we can make sure that both of these libraries are part of vsearch and that they use the correct version of the libraries. In my experience the availability and version
 of these libraries vary a lot from system to system, and I had problems compiling vsearch without updating at least one of them.</div>
<div><br class="">
</div>
<div>Why do you have to remove these libs from the package if we include them?</div>
<div><br class="">
</div>
<div>- Torbjørn</div>
<div><br class="">
</div>
<br class="">
<blockquote type="cite" class="">
<div class=""><br class="">
Thanks,<br class="">
<br class="">
Roland<br class="">
<br class="">
</div>
</blockquote>
</div>
<br class="">
</div>
</body>
</html>