Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bio-SamTools and its dependencies. SAMTools wrong compiler corrected (wa... #992

Closed

Conversation

hajgato
Copy link
Collaborator

@hajgato hajgato commented Aug 6, 2014

Easyblock easybuilders/easybuild-easyblocks#450 needed.

Samtools was compiled with gcc, despite the ictce/intel toolchain. Most probably this is the situation in the case of SAMTools 0.1.18 versions as well.

@hpcugentbot
Copy link
Contributor

Automatic reply from Jenkins: Can I test this?

@@ -23,6 +23,8 @@ toolchainopts = {'optarch': True, 'pic': True}
sources = [SOURCELOWER_TAR_BZ2]
source_urls = [('http://sourceforge.net/projects/samtools/files/%(namelower)s/%(version)s', 'download')]

buildopts = 'CC="$CC" CFLAGS="$CFLAGS" '
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for noticing and reporting this.
@cedric wrote the first bits originally, yet most likely it was me who transcribed it for ictce.
So, whomever of you is relying on samtools-0.1.18/ictce, can you comment on current status?
(I will certainly look at it eventually, but perhaps not at the speed you neet it).

@boegel
Copy link
Member

boegel commented Aug 7, 2014

Jenkins: ok to test

@boegel
Copy link
Member

boegel commented Sep 10, 2014

Jenkins: test this please

@boegel
Copy link
Member

boegel commented Sep 10, 2014

Test report by @boegel
FAILED
Build succeeded for 6 out of 11
Linux SL 6.5, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/85d744ca536b2d0c9304 for a full test report.

@boegel
Copy link
Member

boegel commented Sep 10, 2014

@hajgato: the BioPerl tests are failing for me, see test report. Did they work for you?
@pescobar: care to try testing this too?

@hajgato
Copy link
Collaborator Author

hajgato commented Sep 11, 2014

@boegel @pescobar I will test this today.

@hajgato
Copy link
Collaborator Author

hajgato commented Sep 11, 2014

welded into #1080

@hajgato hajgato closed this Sep 11, 2014
@hajgato hajgato deleted the BioSAMTools_with_deps_mk2 branch October 9, 2015 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants