add easyblock for Hadoop that supports building native libs by boegel · Pull Request #563 · easybuilders/easybuild-easyblocks

boegel · 2015-02-23T15:14:02Z

@ehiggs: please review?

hpcugentbot · 2015-02-23T15:15:48Z

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/787/
Test PASSed.

hpcugentbot · 2015-02-23T15:16:33Z

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/788/
Test PASSed.

wpoely86 · 2015-02-23T15:17:00Z

doesn't this do actually the same as the install_step() from Tarball?

no, Tarball copies from a different location

hmm, I guess I could support specifying the location to copy from in Tarball.install_step

seems like a good idea?

hpcugentbot · 2015-02-23T15:29:33Z

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/789/
Test PASSed.

hpcugentbot · 2015-02-23T15:31:06Z

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/790/
Test PASSed.

wpoely86 · 2015-02-23T15:45:13Z

why not an easyblock option?

I don't think that makes a lot of sense? usually, no 'source' is involved, and it's just a command to run

I meant an easyconfig parameter but it's fine as it is for me.

hpcugentbot · 2015-02-23T15:59:42Z

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/791/
Test PASSed.

boegel · 2015-02-23T16:10:41Z

OK, going in, thanks for the review @wpoely86!

add easyblock for Hadoop that supports building native libs

boegel added 2 commits February 23, 2015 16:12

add easyblock for Hadoop that supports building native libs

cca302d

copy from start_dir rather than builddir

d7ce912

wpoely86 reviewed Feb 23, 2015
View reviewed changes

don't reimplement Tarball.install_step, but enhance/use it

20bf1a1

don't define $HADOOP_CONF_DIR

3838ff9

wpoely86 reviewed Feb 23, 2015
View reviewed changes

boegel mentioned this pull request Feb 23, 2015

{tools}[dummy] Hadoop w/ native libs (REVIEW) easybuilders/easybuild-easyconfigs#1418

Merged

use log.error instead of log.exception

8451407

boegel added a commit that referenced this pull request Feb 23, 2015

Merge pull request #563 from boegel/Hadoop

eec91a3

add easyblock for Hadoop that supports building native libs

boegel merged commit eec91a3 into easybuilders:develop Feb 23, 2015

boegel deleted the Hadoop branch February 23, 2015 16:11

boegel mentioned this pull request Jan 23, 2016

implement easyblock for Cloudera Hadoop distribution #45

Closed

Conversation

boegel commented Feb 23, 2015

Uh oh!

hpcugentbot commented Feb 23, 2015

Uh oh!

hpcugentbot commented Feb 23, 2015

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hpcugentbot commented Feb 23, 2015

Uh oh!

hpcugentbot commented Feb 23, 2015

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hpcugentbot commented Feb 23, 2015

Uh oh!

boegel commented Feb 23, 2015

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants