Page MenuHomeFreeBSD

[new port] biology/canu: Single molecule sequence assembler
ClosedPublic

Authored by jwb on Dec 21 2017, 4:53 PM.
Tags
None
Referenced Files
F168489233: D13574.id36898.diff
Fri, Aug 28, 3:57 PM
F168477833: D13574.id36907.diff
Fri, Aug 28, 2:32 PM
F168471848: D13574.id36906.diff
Fri, Aug 28, 1:51 PM
F168469879: D13574.id36868.diff
Fri, Aug 28, 1:39 PM
Unknown Object (File)
Thu, Aug 27, 3:15 PM
Unknown Object (File)
Thu, Aug 27, 2:38 PM
Unknown Object (File)
Thu, Aug 27, 12:55 PM
Unknown Object (File)
Thu, Aug 27, 8:07 AM
Subscribers

Details

Summary

[new port] biology/canu: Single molecule sequence assembler
Approved by: jrm (mentor) or wen (mentor)
Differential to be added to log

Upstream has already patched head to conform to filesystem hierarchy standards, so the post-build hacks will go away in 1.7.

Test Plan

portlint -C and stage-qa clean
Poudriere tested on {10.3,11.1}-{amd64,i386}

User-tested by researchers

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jrm added inline comments.
biology/canu/Makefile
4 ↗(On Diff #36868)

missing tab?

This revision is now accepted and ready to land.Dec 22 2017, 12:13 AM

Added that to my checklist

This revision now requires review to proceed.Dec 22 2017, 1:27 AM
This revision is now accepted and ready to land.Dec 22 2017, 1:29 AM
This revision was automatically updated to reflect the committed changes.
head/biology/canu/Makefile
14–15

Why += ?

head/biology/canu/pkg-plist
54–55 ↗(On Diff #36898)

installing only static libraries seems like a bad idea.

Good catch on the +=, that was an oversight.

The upstream build system does not support shared libs and it's a nightmare
to patch. The developers are receptive to suggestion, though. I plan to
open an issue asking for shared libs in a future release.

This revision is now accepted and ready to land.Dec 22 2017, 1:47 PM
This revision was automatically updated to reflect the committed changes.