Skip to content

readd MariaDB goolf/goalf easyconfigs removed in #217 + style fixes#293

Merged
boegel merged 21 commits intoeasybuilders:developfrom
boegel:mysql
Sep 25, 2015
Merged

readd MariaDB goolf/goalf easyconfigs removed in #217 + style fixes#293
boegel merged 21 commits intoeasybuilders:developfrom
boegel:mysql

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented May 29, 2013

GCC compiler toolchains that include a recent binutils (goolf-2.x.y?) are required before this can be merged, see easybuilders/easybuild-framework#644.

@boegel boegel modified the milestones: v1.x, 2.x Jun 24, 2015
@rjeschmi
Copy link
Copy Markdown
Contributor

I think this should be closed, and perhaps consider removing mariadb from EB for now.

binutils should be fine on centos6 and above, but the versions here are very old.

Can we tag things for the merge sprint?

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 22, 2015

@rjeschmi: I can try and look into revamping this by updating the MariaDB version here, and using foss/2015b

(I tagged this for tomorrow's sprint)

@boegel boegel added this to the merge sprint 2.0 milestone Sep 22, 2015
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.

delete this

@wpoely86
Copy link
Copy Markdown
Member

@boegel you do this or you want a me to patch it?

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 23, 2015

@wpoely86: if you're up for patching it, by all means

please bump the MariaDB version and usefoss/2015b while you're at it

wpoely86 and others added 5 commits September 23, 2015 12:38
* develop: (2813 commits)
  fix GMP dep in Python 2.7.10 goolf/1.4.10 easyconfig
  fix remark wrt sources
  fix remark
  fix/extend sanity check
  {lang}[goolf/1.4.10] Python 2.7.10
  dcw bumped to 1.1.2 and libxml2 python pound deleted
  bump to 3.7.0 + bump dep versions
  Adding moduleclass
  add missing M4 easyconfig
  add missing 'easyblock' line
  A minor change in sources
  minor style fix
  add missing 'easyblock' lines, fix style issues
  add missing patch file for ANTLR
  add more missing easyblock lines
  add missing 'easyblock' line
  Minor changes in configopts
  add missing easyconfig file
  install qhull.pc via CMake rather than using postinstallcmds
  collapse two patches into one
  ...
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.

@wpoely86: no, leave the existing easyconfig, just add a new one for 10.0.21 with foss/2015b

can you look into a test report too?

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.

why? they are old, untested and never merged?

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.

no, you're changing an easyconfig that's now in develop (and which works fine): https://github.com/hpcugent/easybuild-easyconfigs/tree/develop/easybuild/easyconfigs/m/MariaDB

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.

okay, but it's weird. I never moved this file. I don't understand why git thinks it's a rename.

@wpoely86
Copy link
Copy Markdown
Member

Test report by @wpoely86
FAILED
Build succeeded for 2 out of 3 (2 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/85e48aae94ac4bacacd5 for a full test report.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why not just drop back to GNU 4.9.3-2.25 for this. It will be helpful for other toolchains, like PGI.

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.

@wpoely86: remove this easyconfig, serves no purpose anymore?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Also here, I prefer dropping down to GNU, thoughts?

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.

+1, makes sense to me

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.

@wpoely86: remove this easyconfig, serves no purpose anymore?

@wpoely86
Copy link
Copy Markdown
Member

Test report by @wpoely86
FAILED
Build succeeded for 7 out of 8 (6 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/d704ac0c280484078ad8 for a full test report.

@wpoely86
Copy link
Copy Markdown
Member

Test report by @wpoely86
SUCCESS
Build succeeded for 5 out of 5 (1 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/97e855f83cd4e67a95ba for a full test report.

@wpoely86
Copy link
Copy Markdown
Member

lgtm
@rjeschmi @boegel please re-review.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 23, 2015

Jenkins: test this please

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 23, 2015

Jenkins: ok to test

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.

@wpoely86: remove this easyconfig, serves no purpose anymore?

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.

Can you do that yourself? It's your branch 😉

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.

ah, yes, forgot about that part ;)

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.

specify why you do this? what's the benefit?

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.

Policy. We want EB controlled versions?

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.

OK, sure

@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4541/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.

@wpoely86
Copy link
Copy Markdown
Member

@boegel you forgot the bison-foss easyconfig

@hpcugentbot
Copy link
Copy Markdown

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

boegel commented Sep 25, 2015

Test report by @boegel
SUCCESS
Build succeeded for 4 out of 4 (4 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/d85cd2735da143d39f50 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 25, 2015

Test report by @boegel
SUCCESS
Build succeeded for 4 out of 4 (4 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/aa0bf74b09d344b4703b for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 25, 2015

Hooray! \o/

Oldest PR we have now, ready to go, thanks for the efforts on this @wpoely86, and @rjeschmi for the reviewing.

boegel added a commit that referenced this pull request Sep 25, 2015
readd MariaDB goolf/goalf easyconfigs removed in #217 + style fixes
@boegel boegel merged commit 93eced7 into easybuilders:develop Sep 25, 2015
@boegel boegel deleted the mysql branch September 25, 2015 14:35
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