{devel} [foss/2018a] libutempter v1.1.6 by SimonPinches · Pull Request #6426 · easybuilders/easybuild-easyconfigs

SimonPinches · 2018-06-05T08:32:05Z

Some issues downloading gzipped tarball from Git repo. I did it by hand outside EasyBuild so far...

verdurin · 2018-06-06T22:30:37Z

@SimonPinches Please fix the style errors reported by Travis.

boegel · 2018-06-07T06:41:57Z

$ eb --from-pr 6426 --check-style
== temporary log file in case of crash /tmp/eb-hhreSU/easybuild-DqqNmZ.log

*** WARNING: Using easyconfigs from unstable (pending/failed tests or merge conflict) PR #6426 ***

Running style check on 1 easyconfig(s)...
/tmp/eb-hhreSU/files_pr6426/l/libutempter/libutempter-1.1.6.2-GCC-6.4.0-2.28.eb:12:94: E226 missing whitespace around arithmetic operator
/tmp/eb-hhreSU/files_pr6426/l/libutempter/libutempter-1.1.6.2-GCC-6.4.0-2.28.eb:12:107: E226 missing whitespace around arithmetic operator
/tmp/eb-hhreSU/files_pr6426/l/libutempter/libutempter-1.1.6.2-GCC-6.4.0-2.28.eb:12:114: E226 missing whitespace around arithmetic operator
/tmp/eb-hhreSU/files_pr6426/l/libutempter/libutempter-1.1.6.2-GCC-6.4.0-2.28.eb:12:121: E501 line too long (137 > 120 characters)
/tmp/eb-hhreSU/files_pr6426/l/libutempter/libutempter-1.1.6.2-GCC-6.4.0-2.28.eb:12:127: E226 missing whitespace around arithmetic operator
[FAIL] /tmp/eb-hhreSU/files_pr6426/l/libutempter/libutempter-1.1.6.2-GCC-6.4.0-2.28.eb
ERROR: One or more style checks FAILED!

verdurin · 2018-06-18T18:47:13Z

Test report by @verdurin
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in this PR)
easybuild.novalocal - Linux centos linux 7.5.1804, Intel Xeon E312xx (Sandy Bridge), Python 2.7.5
See https://gist.github.com/5d40770f1c70ba01296fe0fa37d5af7e for a full test report.

verdurin · 2018-06-18T18:49:49Z

As hinted above, test build failed owing to download error.

boegel · 2018-06-19T06:43:52Z

+
+source_urls = [
+    'http://git.altlinux.org/people/ldv/packages/?p=libutempter.git;a=snapshot;h=' +
+    version[0:5] + '-alt' + version[6:7] + ';sf=tgz']


@SimonPinches If this doesn't work and there's no alternative for downloading the sources, we'll need to escape with a manual download for now (but then that should be clarified with a comment that mentions the full URL).

@boegel I've added the full download link in a comment. Perhaps you can help make the auto-download work but so far I've failed :-(

boegel · 2018-09-15T11:30:41Z

+# Download link: http://git.altlinux.org/people/ldv/packages/?p=libutempter.git;a=snapshot;h=1.1.6-alt2;sf=tgz
+# source_urls = [
+#     'http://git.altlinux.org/people/ldv/packages/?p=libutempter.git;a=snapshot;h='; +
+#     version[0:5] + '-alt' + version[6:7] + ';sf=tgz']


@SimonPinches I think we'll need to go with manual download for this one, so just including a comment like:

# download via http://git.altlinux.org/people/ldv/packages/?p=libutempter.git;a=snapshot;h=1.1.6-alt2;sf=tgz # and rename libutempter-1.1.6-alt2-3c6e555.tar.gz to libutempter-1.1.6.2.tar.gz sources = [{ 'filename': SOURCE_TAR_GZ, 'extract_cmd': "tar xfvz %s --strip 1", }]

@boegel Done (with an extra space before 'filename').

….0-2.28]

boegel · 2018-09-18T08:50:49Z

+    'extract_cmd': "tar xfvz %s --strip 1",
+}]
+
+checksums = ['b7e55b666c575d8f2d294c82655b6af21f0c7339625ba6707e5edc2883a2b09a']


@SimonPinches Hmm, I'm getting a different checksum? de1183edd2c1ec80f93a57bc8847f9dd5d47774b78cef6c437583e3c3dae1226

@boegel @SimonPinches Just saw a different checksum:
8b683c355d3376b95a27180f9d710c9b06261fd68b9587c0b13fe443f267d4c5

@boegel @verdurin If I repeat the download and rename, then I get a different checksum again: 198092c3044c5d51bc142411b0a55b75d57207b80d99f1ee785fa4760dfb47cd
Since the original filename seems to already contain a hash, I'm surprised it's continually changing...
How should we handle this?

Ugh, that's pretty annoying... Maybe they are fingerprinting downloads or something?

Any differences between two consecutive downloads that have different checksums with unpack & diff -ru dir1 dir2?

@boegel @verdurin The extracted tarballs are the same, it's the timestamps of the tar files that are then gzipped that causes the different hashes. If I first unzip all the various tgz files then the sha256sum of the tarballs all agree. Anyway to first unzip and then test the checksum?

No, the checksums are computed on the source files as specified in sources.

But, we can make unzipping part of the instructions (including a comment to explain why), and list SOURCE_TAR in sources?

verdurin · 2018-09-23T08:39:14Z

Test report by @verdurin
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in this PR)
easybuild.novalocal - Linux centos linux 7.5.1804, Intel Xeon E312xx (Sandy Bridge), Python 2.7.5
See https://gist.github.com/dce627b1a88c18ad34fabdc7c7711f0e for a full test report.

verdurin · 2018-09-23T08:42:16Z

Test report by @verdurin
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in this PR)
easybuild.novalocal - Linux centos linux 7.5.1804, Intel Xeon E312xx (Sandy Bridge), Python 2.7.5
See https://gist.github.com/be38779ac5b3c014947fb6083692ebbf for a full test report.

boegel · 2018-10-16T15:33:02Z

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
gligar02.gligar.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/38eb3230ce5487413c5603d5e9bf1c3a for a full test report.

boegel · 2019-05-03T15:39:03Z

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node3112.skitty.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 2.7.5
See https://gist.github.com/095f6f8972e6d1e6bc6b00b35e5fe247 for a full test report.

boegel · 2019-05-03T15:41:43Z

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2606.swalot.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/25d9e662ba79fcc16e354aa7c99d40e2 for a full test report.

boegel · 2019-05-03T15:42:23Z

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2016.delcatty.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/12803f3c49189b42d44146b7e558f4fe for a full test report.

boegel · 2019-05-03T15:43:51Z

Going in, thanks @SimonPinches!

SimonPinches force-pushed the libutempter branch 3 times, most recently from 39b8e0a to 130d922 Compare June 5, 2018 09:18

SimonPinches force-pushed the libutempter branch from 130d922 to 8d4b4de Compare June 8, 2018 06:53

First eb file for libutempter v1.1.6

153aa4b

SimonPinches force-pushed the libutempter branch from 8d4b4de to 153aa4b Compare June 8, 2018 07:01

easybuilders deleted a comment from boegelbot Jun 11, 2018

boegel added the new label Jun 11, 2018

Style improvements for libutempter v1.1.6.2 for GCC/6.4.0-2.28

264bc6c

boegel reviewed Jun 19, 2018

View reviewed changes

SimonPinches force-pushed the libutempter branch from 674d150 to ad62efe Compare September 10, 2018 16:35

Add manual download link for libutempter v1.1.6.2 [GCC/6.4.0-2.28]

5df37af

SimonPinches force-pushed the libutempter branch from ad62efe to 5df37af Compare September 11, 2018 06:25

easybuilders deleted a comment from boegelbot Sep 15, 2018

boegel requested changes Sep 15, 2018

View reviewed changes

Update manual download instructions for libutempter v1.1.6.2 [GCC/6.4…

7d9ac80

….0-2.28]

boegel requested changes Sep 18, 2018

View reviewed changes

Unzip libutempter compressed tarball to get consistent checksum

30bf9fb

SimonPinches changed the title ~~{devel} [foss/2018a] libutempter v1.1.6 {WIP}~~ {devel} [foss/2018a] libutempter v1.1.6 Oct 16, 2018

boegel requested changes Oct 16, 2018

View reviewed changes

Comment thread easybuild/easyconfigs/l/libutempter/libutempter-1.1.6.2-GCC-6.4.0-2.28.eb Outdated

Update comments in libutempter-1.1.6.2-GCC-6.4.0-2.28.eb

8c59a63

boegel approved these changes May 3, 2019

View reviewed changes

boegel added this to the next release (3.9.1) milestone May 3, 2019

boegel merged commit e61d095 into easybuilders:develop May 3, 2019

Conversation

SimonPinches commented Jun 5, 2018

Uh oh!

verdurin commented Jun 6, 2018

Uh oh!

boegel commented Jun 7, 2018

Uh oh!

verdurin commented Jun 18, 2018

Uh oh!

verdurin commented Jun 18, 2018

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!

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!

verdurin commented Sep 23, 2018

Uh oh!

verdurin commented Sep 23, 2018

Uh oh!

Uh oh!

boegel commented Oct 16, 2018

Uh oh!

boegel commented May 3, 2019

Uh oh!

boegel commented May 3, 2019

Uh oh!

boegel commented May 3, 2019

Uh oh!

boegel commented May 3, 2019

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants