BEEF 0.1.1-r16 + rename libbeef to BEEF #1679

valtandor · 2015-06-07T23:01:43Z

New EasyConfigs for the BEEF library and binary.

hpcugentbot · 2015-06-07T23:01:45Z

Automatic reply from Jenkins: Can I test this?

valtandor · 2015-06-07T23:02:03Z

Yes.

boegel · 2015-06-08T07:03:18Z

Jenkins: ok to test

hpcugentbot · 2015-06-08T07:08:55Z

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3452/
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3452/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 · 2015-09-25T00:24:59Z

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4510/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 · 2015-09-25T07:16:32Z

@valtandor: I recently created an easyconfig for this too, which was merged a while ago, see #1827

There are a couple of differences: I went with libbeef as name (it was not 100% clear to me what the official name was), you didn't include source_urls, I'm passing CC="$CC" to make sure the right compiler is being used, and the homepage/description is different

It would be nice to sync up these easyconfigs in one way or another... Thoughts?

valtandor · 2015-10-06T00:06:04Z

@boegel: I have updated the home page and description, and gone with the name BEEF as it seems to be the official name of the whole package itself (the tarball name notwithstanding). Comments have been provided in lieu of source_urls where the source code was built from an SVN checkout. CC="$CC" has been included where appropriate. I hope this is OK.

hpcugentbot · 2015-10-06T00:19:20Z

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4634/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 · 2015-10-06T07:04:44Z

easybuild/easyconfigs/b/BEEF/BEEF-0.1.1-r16-iomkl-4.6.13.eb

+toolchain = {'name': 'iomkl', 'version': '4.6.13'}
+toolchainopts = {'openmp': False, 'usempi': False}
+
+# There is no source_url as this version is built from a tarred SVN checkout.


maybe include the SVN repo URL?
and also thesvn command one can use to check out this exact version?

you'll need to fix the easyconfig filename here too, the versions don't match up

boegel · 2015-10-06T07:05:32Z

@valtandor: I'm OK with going with BEEF, I wasn't quite sure when I went with libbeef either, and I don't feel strongly about it, thanks for syncing things up!

hpcugentbot · 2015-10-15T01:24:58Z

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4741/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 · 2015-10-15T07:29:12Z

easybuild/easyconfigs/b/BEEF/BEEF-0.1.1-r16-intel-2015a.eb

+#for url in source_urls:
+#    svn co -r 16 url
+#    tar --exclude-vcs -cjvf "beef-0.1.1-r16.tar.bz2" "beef"
+#sources = [SOURCELOWER_TAR_BZ2]


@valtandor: you probably meant to uncomment the source_urls, and leave sources as is?

I'm not entirely sure. Does Easybuild allow for checking out SVN sources directly?

No, not yet. This is in the works, but it has been dormant for a while, see easybuilders/easybuild-framework#1082.

So, for now, you need to create the tarball manually, and only specify sources (together with some comments on how to create the source tarball, like you did here).

OK, will do, when I have a moment and am not working on a file in another branch.

hpcugentbot · 2015-10-26T19:19:22Z

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4844/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 · 2015-10-27T07:55:59Z

easybuild/easyconfigs/b/BEEF/BEEF-0.1.1-intel-2015a.eb

+toolchainopts = {'openmp': False, 'usempi': False}
+
+source_urls = ['https://confluence.slac.stanford.edu/download/attachments/146704476']
+sources = "libbeef-{0}.tar.gz".format(version)


this should be a list, and please use %(version)s (avoid using Python code when you can in easyconfigs)

sources = ['libbeef-%(version)s.tar.gz']

hpcugentbot · 2015-10-27T19:21:20Z

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4856/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 · 2015-10-27T19:31:34Z

easybuild/easyconfigs/b/BEEF/BEEF-0.1.1-r16-intel-2015a.eb

+
+# To make a source tarball from the SVN repository:
+# 1. svn co -r 16 svn://suncatls1.slac.stanford.edu/beef
+# 2. tar --exclude-vcs -cjvf beef-0.1.1-r16.tar.bz2 beef


two issues I ran into when testing this:

my tar does not support --exclude-vcs; it does support using --exclude \.svn, maybe that's more generic?

you should tar beef/trunk, not beef right? if not, I run into the problem that configure is not found during the EB run

So, my suggestion:

2. tar --exclude \.svn -cjvf beef-0.1.1-r16.tar.bz2 beef/trunk

How about tar --exclude \.svn --transform 's,^beef/trunk,beef-0.1.1-r16,' -cjvf beef-0.1.1-r16.tar.bz2 beef/trunk?

Nice suggestion, but: tar: Option --transform is not supported :-)

I wouldn't worry too much about how the top-level dir is called, since it only matters for the build dir.

Righto. My local repo is right now in a different branch, but once it's finished with that branch I'll make these changes and upload them.

boegel · 2015-10-27T19:34:02Z

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

hpcugentbot · 2015-10-27T21:58:52Z

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4857/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 · 2015-10-27T22:08:37Z

Going in, thanks @valtandor!

BEEF 0.1.1-r16 + rename libbeef to BEEF

Benjamin Roberts added 4 commits April 17, 2015 14:53

New package BEEF for Bayesian Error Estimation Functionals

b8f6a49

Merge branch 'develop' into BEEF

7d361df

EasyConfig for BEEF using iomkl 4.6.13 appropriate for VASP

86e5188

Merge branch 'develop' into BEEF

ce5c610

boegel modified the milestone: v2.2 Jun 24, 2015

Merge branch 'develop' into BEEF

9ffdf8e

boegel added this to the v2.4.0 milestone Sep 25, 2015

Benjamin Roberts added 2 commits October 6, 2015 12:32

Merge branch 'develop' into BEEF

f10a06d

Merge libbeef and BEEF

7fb6d7d

boegel reviewed Oct 6, 2015
View reviewed changes

Added SVN source URL and so forth

c7d61b8

boegel reviewed Oct 15, 2015
View reviewed changes

Updated comments regarding source file preparation

6270ab4

boegel reviewed Oct 27, 2015
View reviewed changes

Turn sources into a list

781b554

boegel reviewed Oct 27, 2015
View reviewed changes

Make tar command more universal

daa008a

boegel changed the title ~~Beef~~ BEEF 0.1.1-r16 + rename libbeef to BEEF Oct 27, 2015

boegel added a commit that referenced this pull request Oct 27, 2015

Merge pull request #1679 from valtandor/BEEF

ca63e42

BEEF 0.1.1-r16 + rename libbeef to BEEF

boegel merged commit ca63e42 into easybuilders:develop Oct 27, 2015

valtandor deleted the BEEF branch October 27, 2015 22:23

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

BEEF 0.1.1-r16 + rename libbeef to BEEF #1679

BEEF 0.1.1-r16 + rename libbeef to BEEF #1679

valtandor commented Jun 7, 2015

hpcugentbot commented Jun 7, 2015

valtandor commented Jun 7, 2015

boegel commented Jun 8, 2015

hpcugentbot commented Jun 8, 2015

hpcugentbot commented Sep 25, 2015

boegel commented Sep 25, 2015

valtandor commented Oct 6, 2015

hpcugentbot commented Oct 6, 2015

boegel Oct 6, 2015

boegel commented Oct 6, 2015

hpcugentbot commented Oct 15, 2015

boegel Oct 15, 2015

valtandor Oct 21, 2015

boegel Oct 21, 2015

valtandor Oct 22, 2015

hpcugentbot commented Oct 26, 2015

boegel Oct 27, 2015

hpcugentbot commented Oct 27, 2015

boegel Oct 27, 2015

boegel Oct 27, 2015

valtandor Oct 27, 2015

boegel Oct 27, 2015

valtandor Oct 27, 2015

boegel commented Oct 27, 2015

hpcugentbot commented Oct 27, 2015

boegel commented Oct 27, 2015

BEEF 0.1.1-r16 + rename libbeef to BEEF #1679

BEEF 0.1.1-r16 + rename libbeef to BEEF #1679

Conversation

valtandor commented Jun 7, 2015

hpcugentbot commented Jun 7, 2015

valtandor commented Jun 7, 2015

boegel commented Jun 8, 2015

hpcugentbot commented Jun 8, 2015

hpcugentbot commented Sep 25, 2015

boegel commented Sep 25, 2015

valtandor commented Oct 6, 2015

hpcugentbot commented Oct 6, 2015

Choose a reason for hiding this comment

boegel commented Oct 6, 2015

hpcugentbot commented Oct 15, 2015

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hpcugentbot commented Oct 26, 2015

Choose a reason for hiding this comment

hpcugentbot commented Oct 27, 2015

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

boegel commented Oct 27, 2015

hpcugentbot commented Oct 27, 2015

boegel commented Oct 27, 2015