Skip to content

Contrib whoosh#2725

Merged
boegel merged 4 commits intoeasybuilders:developfrom
tloizou:contrib_Whoosh
Mar 16, 2016
Merged

Contrib whoosh#2725
boegel merged 4 commits intoeasybuilders:developfrom
tloizou:contrib_Whoosh

Conversation

@tloizou
Copy link
Copy Markdown
Contributor

@tloizou tloizou commented Mar 16, 2016

No description provided.

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Mar 16, 2016

Automatic reply from Jenkins: Can I test this?

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 16, 2016

Jenkins: ok to test

@boegel boegel added this to the v2.8.0 milestone Mar 16, 2016
boegel
boegel reviewed Mar 16, 2016
View reviewed changes
Comment thread easybuild/easyconfigs/a/astropy/astropy-1.0.6-goolf-1.4.10-Python-2.7.3.eb Outdated

dependencies = [
(python, pyver),
('setuptools', '18.4', versionsuffix),
Copy link
Copy Markdown
Member

@boegel boegel Mar 16, 2016

Choose a reason for hiding this comment

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

@tloizou easyconfig for this is missing, any idea if it's available somewhere else, e.g. a PR that isn't merged yet?

Copy link
Copy Markdown
Contributor Author

@tloizou tloizou Mar 16, 2016

Choose a reason for hiding this comment

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

@boegel: sorry this files shouldn't be here. I will correct this. Please see PR #2724 for astropy which includes the setuptools config.

Copy link
Copy Markdown
Member

@boegel boegel Mar 16, 2016

Choose a reason for hiding this comment

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

OK, thanks!

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Mar 16, 2016

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7137/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 16, 2016

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

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Mar 16, 2016

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7141/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
Copy Markdown
Member

boegel commented Mar 16, 2016

lgtm, going in, thanks @tloizou!

boegel added a commit that referenced this pull request Mar 16, 2016
@boegel
Merge pull request #2725 from tloizou/contrib_Whoosh
cdc1056 
Contrib whoosh
@boegel boegel merged commit cdc1056 into easybuilders:develop Mar 16, 2016
@boegel boegel mentioned this pull request Mar 16, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v2.8.0

Development

Successfully merging this pull request may close these issues.

3 participants

@tloizou @hpcugentbot @boegel