From: Roel Janssen <roel@gnu.org>
To: guix-devel@gnu.org
Subject: Re: [PATCH] Add vcflib.
Date: Wed, 20 Apr 2016 14:39:19 +0200 [thread overview]
Message-ID: <87h9ew3154.fsf@gnu.org> (raw)
In-Reply-To: <idjshzhqp7w.fsf@bimsb-sys02.mdc-berlin.net>
Dear reviewers,
Can I make the review process easier by posting the patches in separate
threads?
Thanks!
Kind regards,
Roel Janssen
Ricardo Wurmus writes:
> Roel Janssen <roel@gnu.org> writes:
>
>> Dear Guix,
>>
>> In an effort to package freebayes, I would first like to add vcflib and
>> its dependencies. Therefore, I have attached eight patches.
>>
>> I've attempted to decouple the dependencies from vcflib, which worked
>> for intervaltree and htslib. The following dependencies are actually
>> tightly coupled in the build process:
>> - tabixpp
>> - smithwaterman
>> - multichoose
>> - fsom
>> - filevercmp
>> - fastahack
>
> Thank you for your efforts!
>
>> To decouple these, we would need to include header files in the package
>> output that aren't part of the public interface. In addition to that,
>> we would need to patch the build system to not look for <object>.o, but
>> instead add the right directives to the linker.
>
> I see what you mean. This is ugly, but there isn’t much you can do
> about this. I’ve encountered many packages that do the same (some of
> which also involve tabix), and in the case of “libvcflib.a” it really
> does seem very hard to untangle dependencies further.
>
> “filevercmp” and “fsom” don’t install libraries at all and apparently
> are supposed to be included in other code.
>
>> I don't think that is the desirable approach, because that would cause
>> these packages to provide header files that should only be used
>> internally. Therefore, I use the source of these packages in vcflib,
>> and unpack them in the vcflib project root, to avoid confusion on
>> interfaces and fiddling with the build system.
>
> This seems acceptable to me. I haven’t yet looked at your patches, but
> the justification you give is valid; I looked at the Makefiles of vcflib
> and included projects and I think this is okay.
>
> ~~ Ricardo
next prev parent reply other threads:[~2016-04-20 12:39 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-22 15:24 [PATCH] Add vcflib Roel Janssen
2016-03-23 13:49 ` Ricardo Wurmus
2016-04-20 12:39 ` Roel Janssen [this message]
2016-04-20 12:46 ` Ricardo Wurmus
2016-04-22 7:24 ` Roel Janssen
2016-04-26 14:47 ` Ricardo Wurmus
2016-04-26 17:31 ` Roel Janssen
2017-01-16 21:59 ` Ricardo Wurmus
2016-04-20 12:49 ` Ricardo Wurmus
2016-04-26 21:50 ` Roel Janssen
2017-01-16 22:04 ` Ricardo Wurmus
2016-04-20 12:53 ` Ricardo Wurmus
2016-04-22 7:40 ` Roel Janssen
2016-04-26 14:47 ` Ricardo Wurmus
2016-04-26 17:37 ` Roel Janssen
2017-01-16 21:56 ` Ricardo Wurmus
2016-04-20 12:56 ` Ricardo Wurmus
2016-04-20 12:58 ` Ricardo Wurmus
2016-04-22 8:04 ` Roel Janssen
2016-04-22 11:27 ` Ricardo Wurmus
2016-04-25 9:20 ` Roel Janssen
2016-04-26 14:42 ` Ricardo Wurmus
2016-04-20 13:06 ` Ricardo Wurmus
2016-04-20 13:08 ` Ricardo Wurmus
2016-04-26 17:50 ` Roel Janssen
2016-04-26 17:51 ` Roel Janssen
2017-01-16 21:54 ` Ricardo Wurmus
2017-01-17 21:57 ` Roel Janssen
2017-01-18 8:24 ` Ricardo Wurmus
2016-04-20 13:18 ` Ricardo Wurmus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87h9ew3154.fsf@gnu.org \
--to=roel@gnu.org \
--cc=guix-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).