Bug 205524

Summary: biology/htslib: Update to 1.3
Product: Ports & Packages Reporter: cartwright
Component: Individual Port(s)Assignee: Jason Unovitch <junovitch>
Status: Closed FIXED    
Severity: Affects Some People CC: junovitch
Priority: --- Keywords: easy, patch, patch-ready
Version: LatestFlags: cartwright: maintainer-feedback+
Hardware: Any   
OS: Any   
Bug Depends on:    
Bug Blocks: 205525, 205526    
Attachments:
Description Flags
Patch to upgrade biology/htslib to 1.3
vlad-fbsd: maintainer-approval+
poudriere testport output for releng_10_2
none
poudriere testport output for releng_9_3 none

Description cartwright 2015-12-22 23:20:22 UTC
Created attachment 164511 [details]
Patch to upgrade biology/htslib to 1.3

Update biology/htslib to version 1.3.

Add libcurl support.

Add make test support.
Comment 1 cartwright 2015-12-22 23:21:16 UTC
Created attachment 164512 [details]
poudriere testport output for releng_10_2
Comment 2 cartwright 2015-12-22 23:21:43 UTC
Created attachment 164513 [details]
poudriere testport output for releng_9_3
Comment 3 VK 2015-12-23 00:16:58 UTC
Comment on attachment 164511 [details]
Patch to upgrade biology/htslib to 1.3

Thanks. Note that you have the ability to approve the patches for ports you're maintainer of.
Comment 4 Jason Unovitch freebsd_committer freebsd_triage 2015-12-24 22:58:36 UTC
Everywhere else in ports we use "CURL" for that option. Not as 'libcurl' or lower case.  Otherwise with that small change I think the patch looks good and I am reviewing the entire batch of patches to commit all at once since they depend on this bug.
Comment 5 commit-hook freebsd_committer freebsd_triage 2015-12-25 01:32:04 UTC
A commit references this bug:

Author: junovitch
Date: Fri Dec 25 01:31:11 UTC 2015
New revision: 404398
URL: https://svnweb.freebsd.org/changeset/ports/404398

Log:
  biology/{bcftools,htslib,samtools}: update 1.2.x -> 1.3

  - Update ports to 1.3 and set BUILD_DEPENDS of dependent ports to require
    version 1.3 of htslib.
  - Add CURL option to htslib
  - Add TEST_TARGET with perl and bash dependencies for testing
  - Tidy up spacing and pkg-message's

  PR:		205524
  PR:		205525
  PR:		205526
  Submitted by:	cartwright@asu.edu (maintainer)

Changes:
  head/biology/bcftools/Makefile
  head/biology/bcftools/distinfo
  head/biology/bcftools/files/patch-Makefile
  head/biology/bcftools/files/patch-plugins_fixploidy.mk
  head/biology/bcftools/files/patch-plugins_vcf2sex.mk
  head/biology/bcftools/files/patch-prob1.c
  head/biology/bcftools/files/patch-test_test.pl
  head/biology/bcftools/files/pkg-message.in
  head/biology/bcftools/pkg-plist
  head/biology/htslib/Makefile
  head/biology/htslib/distinfo
  head/biology/htslib/files/patch-Makefile
  head/biology/htslib/files/patch-configure.ac
  head/biology/htslib/files/patch-cram_cram__io.c
  head/biology/htslib/files/patch-hfile__libcurl.c
  head/biology/htslib/pkg-plist
  head/biology/samtools/Makefile
  head/biology/samtools/distinfo
  head/biology/samtools/files/ax_with_curses.m4
  head/biology/samtools/files/ax_with_htslib.m4
  head/biology/samtools/files/patch-Makefile
  head/biology/samtools/files/patch-configure.ac
  head/biology/samtools/files/patch-dict.c
  head/biology/samtools/files/patch-errmod.c
  head/biology/samtools/files/patch-test_test.pl
  head/biology/samtools/files/pkg-message.in
  head/biology/samtools/pkg-plist
Comment 6 Jason Unovitch freebsd_committer freebsd_triage 2015-12-25 01:33:30 UTC
Committed. Thanks!

Note, this was also missing:
TEST_DEPENDS=   bash:${PORTSDIR}/shells/bash

A `make test` failed on the missing bash dependency.  I added that in as well as fixing the naming for CURL option as mentioned in comment 4.