jeffdaily / parasail

Pairwise Sequence Alignment Library
Other
243 stars 34 forks source link

Fix tarball #77

Closed SoapZA closed 4 years ago

SoapZA commented 4 years ago

Hi @jeffdaily your latest tarballs are missing a header. I've added it in Makefile.am and changed Travis to use make distcheck, which catches such issues (it tries to build the package out-of-source using the produced tarball). Could we cut a new release after merging my fixes?

jeffdaily commented 4 years ago

Don't stress about the failed CI. Looks like I broke develop branch.

SoapZA commented 4 years ago

also, Travis seems to be dead?

jeffdaily commented 4 years ago

Travis build fired off but it isn't reporting here. https://travis-ci.org/github/jeffdaily/parasail/builds/674973519

jeffdaily commented 4 years ago

Looks like I broke the travis credentials recently. Will look into that, too.

SoapZA commented 4 years ago

@jeffdaily did you have time to bump the version and cut a new release?

jeffdaily commented 4 years ago

Didn't have time yesterday to tag and release. Hopefully by end of today.

jeffdaily commented 4 years ago

Tagged v2.4.2.
https://github.com/jeffdaily/parasail/releases/tag/v2.4.2

SoapZA commented 4 years ago

Thanks Jeff!

SoapZA commented 4 years ago

@jeffdaily you seem to be missing the bootstrapped tarballs

jeffdaily commented 4 years ago

yup. been too long since I last deployed. trying to fix this. auth failed when trying to push assets from travis to github.

jeffdaily commented 4 years ago

pardon the delay. had auth issues for both travis and appveyor release deployment. retagged with updated auth tokens for each. new release (same version number 2.4.2). Just waiting for assets to finish building.

jeffdaily commented 4 years ago

done