Skip to content

fix source_urls for arpack-ng v3.1.3 and v3.1.5, ChIP-Seq v1.5-1, Ghostscript v9.10, v9.14 and v9.16, Git v1.7.12, v1.8.2 and v1.8.3.1, lynx v2.8.7 and MEME v4.8.0#4050

Merged
boegel merged 5 commits intoeasybuilders:developfrom
migueldiascosta:20170119162040_new_pr_arpack-ng313
Feb 2, 2017
Merged

fix source_urls for arpack-ng v3.1.3 and v3.1.5, ChIP-Seq v1.5-1, Ghostscript v9.10, v9.14 and v9.16, Git v1.7.12, v1.8.2 and v1.8.3.1, lynx v2.8.7 and MEME v4.8.0#4050
boegel merged 5 commits intoeasybuilders:developfrom
migueldiascosta:20170119162040_new_pr_arpack-ng313

Conversation

@migueldiascosta
Copy link
Copy Markdown
Member

(created using eb --new-pr)

@migueldiascosta migueldiascosta changed the title fix source_urls for arpack-ng v3.1.3 and v3.1.5, ChIP-Seq v1.5-1, Ghostscript v9.10, Git v1.7.12, v1.8.2 and v1.8.3.1, lynx v2.8.7 and MEME v4.8.0 fix source_urls for arpack-ng v3.1.3 and v3.1.5, ChIP-Seq v1.5-1, Ghostscript v9.10, v9.14 and v9.16, Git v1.7.12, v1.8.2 and v1.8.3.1, lynx v2.8.7 and MEME v4.8.0 Jan 20, 2017
@boegel boegel added this to the 3.1.0 milestone Jan 20, 2017
toolchain = {'name': 'goolf', 'version': '1.4.10'}

source_urls = ['http://downloads.sourceforge.net/project/chip-seq/chip-seq/%(version)s/']
source_urls = ['http://downloads.sourceforge.net/project/chip-seq/chip-seq/1.5.1/']
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@migueldiascosta please avoid hardcoding stuff like this, since we're bound to miss this later

source_urls = ['http://downloads.sourceforge.net/project/chip-seq/chip-seq/%s/' % '.'.join(version.split('-'))]

Any idea why we're using 1.5-1 as version? Is that a mistake?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@boegel they seem to have changed scheme from e.g. 1.4-2 to e.g. 1.5.3, but in 1.5.0 and 1.5.1 they used the new scheme in the folder name and the old scheme in the filename...

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 20, 2017

@migueldiascosta since some of the download locations changed drastically, shall we also include checksums to make sure we're still downloading the same thing?

@migueldiascosta
Copy link
Copy Markdown
Member Author

@boegel some definitely changed, at least repackaged, I'll add the checksums of the current files

source_urls = ["http://downloads.ghostscript.com/public/old-gs-releases/"]
sources = [SOURCELOWER_TAR_GZ]

checksums = ['024f040b1371348a2b7fc4d25b379d33']
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this looks wrong, I'm getting 586494befb443363338c1b6379f13973 for this download...

Both when I try to redownload it, and with the download I had before:

$ ls -l ghostscript-9.14.tar.gz 
-rw-rw-r-- 1 myuser myuser 35514406 Dec 10  2014 ghostscript-9.14.tar.gz
$ md5sum ghostscript-9.14.tar.gz
586494befb443363338c1b6379f13973  ghostscript-9.14.tar.gz

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@boegel you're right, will fix

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 2, 2017

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 2, 2017

Going in, thanks @migueldiascosta!

@boegel boegel merged commit 81c183e into easybuilders:develop Feb 2, 2017
@migueldiascosta migueldiascosta deleted the 20170119162040_new_pr_arpack-ng313 branch March 2, 2017 03:24
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.

2 participants