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

Various fixes: add sanity check to GEOS and change homepage netcdf4-python #2285

Merged
merged 5 commits into from
Jan 8, 2016

Conversation

wpoely86
Copy link
Member

@wpoely86 wpoely86 commented Jan 8, 2016

As requested in #2221

@hpcugentbot
Copy link
Contributor

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/5501/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@boegel boegel added this to the v2.6.0 milestone Jan 8, 2016
sources = [SOURCELOWER_TAR_BZ2]

toolchain = {'name': 'goalf', 'version': '1.1.0-no-OFED'}

sanity_check_paths = {
'files': ['bin/geos-config', 'lib/libgeos.so', 'lib/libgeos.a', 'include/geos.h'],
'dirs': ["."]
Copy link
Member

Choose a reason for hiding this comment

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

change this to empty list please, . will always be a non-empty dir

@boegel
Copy link
Member

boegel commented Jan 8, 2016

and make the title a bit more descriptive, otherwise it'll be a nuisance when composing the release notes

@wpoely86 wpoely86 changed the title Various fixes Various fixes: add sanity check to GEOS and change homepage netcdf4-python Jan 8, 2016
@wpoely86
Copy link
Member Author

wpoely86 commented Jan 8, 2016

Test report by @wpoely86
FAILED
Build succeeded for 10 out of 12 (5 easyconfigs in this PR)
Linux SL 6.7, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/d1075f8acecbbece34ac for a full test report.

@@ -4,11 +4,16 @@ name = 'GEOS'
version = '3.3.5'

homepage = 'http://trac.osgeo.org/geos'
description = """GEOS (Geometry Engine - Open Source) is a C++ port of the Java Topology Suite (JTS)"""
Copy link
Member

Choose a reason for hiding this comment

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

missing """ in front, syntax error

@wpoely86
Copy link
Member Author

wpoely86 commented Jan 8, 2016

Test report by @wpoely86
FAILED
Build succeeded for 24 out of 59 (5 easyconfigs in this PR)
Linux centos linux 7.1.1503, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/9168bc7d87226ed264c6 for a full test report.

@boegel
Copy link
Member

boegel commented Jan 8, 2016

Test report by @boegel
SUCCESS
Build succeeded for 5 out of 5 (5 easyconfigs in this PR)
Linux SL 6.7, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/2c017b6abfd7eadb5aa5 for a full test report.

@hpcugentbot
Copy link
Contributor

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/5504/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@hpcugentbot
Copy link
Contributor

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/5508/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@boegel
Copy link
Member

boegel commented Jan 8, 2016

going in, thanks @wpoely86!

boegel added a commit that referenced this pull request Jan 8, 2016
Various fixes: add sanity check to GEOS and change homepage netcdf4-python
@boegel boegel merged commit bc2b98f into easybuilders:develop Jan 8, 2016
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.

3 participants