MikkelSchubert / paleomix

Pipelines and tools for the processing of ancient and modern HTS data.
https://paleomix.readthedocs.io/en/stable/
MIT License
43 stars 19 forks source link

Failure in build time test #13

Closed tillea closed 6 years ago

tillea commented 6 years ago

Hi, I tried to upgrade the Debian package of paleomix to the latest release 1.2.12 and noticed that there is a build time issue. I skipped the test test_requirementobjversioncommand_not_executable and in the comment of the according patch I droped the error that results when keeping the test. Am I missing something or is this test just wrong? Kind regards, Andreas.

MikkelSchubert commented 6 years ago

Hi Andreas, Thank you very much for the report.

I'm pushing a fix for this test. The problem is that it relies on the existence of the 'README.md' file, which used to exist in both the root and the tests/ directory, until one of the two files was renamed. Depending on how you ran the tests, it would use one or the other, so the patch simply makes sure that the naming is consistent.

Cheers