Skip to content

rename S.A.G.E. to SAGE, can't have directories with trailing dot in Windows#4368

Merged
boegel merged 4 commits intoeasybuilders:developfrom
rjeschmi:sage-rename
Mar 18, 2017
Merged

rename S.A.G.E. to SAGE, can't have directories with trailing dot in Windows#4368
boegel merged 4 commits intoeasybuilders:developfrom
rjeschmi:sage-rename

Conversation

@rjeschmi
Copy link
Copy Markdown
Contributor

I'll also change the software name, but this generally works.

@rjeschmi
Copy link
Copy Markdown
Contributor Author

Test report by @rjeschmi
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
ottbioinfo.openstacklocal - Linux centos linux 7.3.1611, Intel Xeon E312xx (Sandy Bridge), Python 2.7.5
See https://gist.github.com/3f63b417bb6bebe9fe53e02a3d6d5a13 for a full test report.

@boegel boegel changed the title cannot have directories with trailing dot in windows rename S.A.G.E. to SAGE, can't have directories with trailing dot in Windows Mar 18, 2017
@boegel boegel added this to the 3.1.2 milestone Mar 18, 2017
source_urls = ['https://storage.googleapis.com/google-code-archive-downloads/v2/code.google.com/opensage/']
sources = ['%(name)sv%(version)s_Linux_x86_64.tar.gz']
source_urls = ['https://github.com/elstonsage/sageCore/releases/download/v6.3']
sources = ['S.A.G.E.v%(version)s_Linux_x86_64.tar.gz']
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.

@rjeschmi let's include a checksum too, since we're changing the source_urls?

I have c94a0b9af656212251ce452f7b2f99ec for this one, both for old & new source_urls.

toolchain = {'name': 'dummy', 'version': 'dummy'}

source_urls = ['http://darwin.cwru.edu/sage/files/']
sources = ['S.A.G.E.v%(version)s_Linux_x86_64.tar.gz']
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.

please also add checksum: da3b135389df1ab9a21288cb041ac0c3

@rjeschmi
Copy link
Copy Markdown
Contributor Author

Test report by @rjeschmi
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
ottbioinfo.openstacklocal - Linux centos linux 7.3.1611, Intel Xeon E312xx (Sandy Bridge), Python 2.7.5
See https://gist.github.com/ee20d296c9b9af224851d61ad9268ef3 for a full test report.

boegel added a commit to boegel/easybuild-easyconfigs that referenced this pull request Mar 18, 2017

source_urls = ['https://storage.googleapis.com/google-code-archive-downloads/v2/code.google.com/opensage/']
sources = ['%(name)sv%(version)s_Linux_x86_64.tar.gz']
source_urls = ['https://github.com/elstonsage/sageCore/releases/download/v6.3']
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.

@rjeschmi one more detail: avoid hardcoding the version, use %(version)s:

source_urls = ['https://github.com/elstonsage/sageCore/releases/download/v%(version)s']

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 18, 2017

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 18, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2101.delcatty.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/47da9d77c2623fc27b88c3c4ed3eb6ac for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 18, 2017

Going in, thanks @rjeschmi!

@boegel boegel merged commit bb02b7d into easybuilders:develop Mar 18, 2017
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