[med-svn] [Git][med-team/unicycler][master] 4 commits: Add pilon dependency once at least pilon-non-free is available

Andreas Tille gitlab at salsa.debian.org
Tue Jun 12 14:15:59 BST 2018


Andreas Tille pushed to branch master at Debian Med / unicycler


Commits:
3a359b70 by Andreas Tille at 2018-06-12T14:58:12+02:00
Add pilon dependency once at least pilon-non-free is available

- - - - -
473f7a84 by Andreas Tille at 2018-06-12T14:58:48+02:00
Better cleaning

- - - - -
f15d5f82 by Andreas Tille at 2018-06-12T14:59:07+02:00
Install script needed in test cases

- - - - -
a9fd6e9d by Andreas Tille at 2018-06-12T15:15:39+02:00
Try hard to get bowtie2 found but failed

- - - - -


4 changed files:

- debian/control
- + debian/patches/bowtie.patch
- debian/patches/series
- debian/rules


Changes:

=====================================
debian/control
=====================================
--- a/debian/control
+++ b/debian/control
@@ -11,6 +11,7 @@ Build-Depends: debhelper (>= 11~),
                bowtie2,
                freebayes,
                ncbi-blast+,
+               pilon,
                racon,
                samtools,
                spades,
@@ -28,6 +29,7 @@ Depends: ${python3:Depends},
          bowtie2,
          freebayes,
          ncbi-blast+,
+         pilon,
          racon,
          samtools,
          spades


=====================================
debian/patches/bowtie.patch
=====================================
--- /dev/null
+++ b/debian/patches/bowtie.patch
@@ -0,0 +1,38 @@
+Description: No idea how the test suite can be convinced to find the tools
+  Asked upstream for help in
+      https://github.com/rrwick/Unicycler/issues/114
+  This patch does not really help
+
+--- a/unicycler/unicycler.py
++++ b/unicycler/unicycler.py
+@@ -411,10 +411,10 @@ def get_arguments():
+                               help='Do not use Pilon to polish the final assembly (default: Pilon '
+                                    'is used)'
+                                    if show_all_args else argparse.SUPPRESS)
+-    polish_group.add_argument('--bowtie2_path', type=str, default='bowtie2',
++    polish_group.add_argument('--bowtie2_path', type=str, default='/usr/bin/bowtie2',
+                               help='Path to the bowtie2 executable'
+                                    if show_all_args else argparse.SUPPRESS)
+-    polish_group.add_argument('--bowtie2_build_path', type=str, default='bowtie2-build',
++    polish_group.add_argument('--bowtie2_build_path', type=str, default='/usr/bin/bowtie2-build',
+                               help='Path to the bowtie2_build executable'
+                                    if show_all_args else argparse.SUPPRESS)
+     polish_group.add_argument('--samtools_path', type=str, default='samtools',
+--- a/test/test_dependencies.py
++++ b/test/test_dependencies.py
+@@ -59,13 +59,13 @@ class TestDependencies(unittest.TestCase
+         self.assertEqual(ret_code, 1)
+ 
+     def test_bowtie2_build_not_found(self):
+-        stdout, stderr, ret_code = self.run_unicycler(['--bowtie2_build_path', 'not_a_real_path'])
++        stdout, stderr, ret_code = self.run_unicycler(['--bowtie2_build_path', '/usr/bin'])
+         self.assertTrue(bool(re.search(r'bowtie2-build\s+not found', stdout)))
+         self.assertTrue('could not find bowtie2-build' in stderr)
+         self.assertEqual(ret_code, 1)
+ 
+     def test_bowtie2_not_found(self):
+-        stdout, stderr, ret_code = self.run_unicycler(['--bowtie2_path', 'not_a_real_path'])
++        stdout, stderr, ret_code = self.run_unicycler(['--bowtie2_path', '/usr/bin'])
+         self.assertTrue(bool(re.search(r'bowtie2\s+not found', stdout)))
+         self.assertTrue('could not find bowtie2' in stderr)
+         self.assertEqual(ret_code, 1)


=====================================
debian/patches/series
=====================================
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1 +1,2 @@
 spades.patch
+# bowtie.patch 


=====================================
debian/rules
=====================================
--- a/debian/rules
+++ b/debian/rules
@@ -13,6 +13,8 @@ export DH_BUILD_MAINT_OPTIONS=nocheck
 override_dh_auto_clean:
 	echo "Avoid:  command 'UnicyclerClean' has no such option 'all'"
 	find . -name "*.o" -delete
+	find . -name "*.so" -delete
+	find . -name __pycache__ -type d | xargs rm -rf
 
 override_dh_auto_build:
 	dh_auto_build
@@ -22,5 +24,6 @@ override_dh_auto_build:
 override_dh_auto_test:
 	BUILDPATH=$(shell pybuild --print build_dir --interpreter python3) ; \
 	ln -s $(CURDIR)/sample_data $${BUILDPATH} ; \
+	ln -s $(CURDIR)/unicycler-runner.py $${BUILDPATH} ; \
 	cp -a unicycler/*.so $${BUILDPATH}/$(PYBUILD_NAME) ; \
 	PYTHONPATH=$${BUILDPATH}/$(PYBUILD_NAME) dh_auto_test



View it on GitLab: https://salsa.debian.org/med-team/unicycler/compare/12489935ee409dc417a7885cb73ff67af22a6be6...a9fd6e9d4851abb7cc898cda7e22cd3dbdbd1230

-- 
View it on GitLab: https://salsa.debian.org/med-team/unicycler/compare/12489935ee409dc417a7885cb73ff67af22a6be6...a9fd6e9d4851abb7cc898cda7e22cd3dbdbd1230
You're receiving this email because of your account on salsa.debian.org.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/debian-med-commit/attachments/20180612/77c14260/attachment-0001.html>


More information about the debian-med-commit mailing list