Skip to content

Prevent non-critical error to stop Guile v1.8.8 build#7446

Merged
boegel merged 2 commits intoeasybuilders:developfrom
vanzod:20190118101152_new_pr_Guile188
Jan 18, 2019
Merged

Prevent non-critical error to stop Guile v1.8.8 build#7446
boegel merged 2 commits intoeasybuilders:developfrom
vanzod:20190118101152_new_pr_Guile188

Conversation

@vanzod
Copy link
Copy Markdown
Member

@vanzod vanzod commented Jan 18, 2019

(created using eb --new-pr)

@vanzod
Copy link
Copy Markdown
Member Author

vanzod commented Jan 18, 2019

Test report by @vanzod
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
cermis - Linux debian 9.6, Intel(R) Xeon(R) CPU E5620 @ 2.40GHz, Python 2.7.13
See https://gist.github.com/0d1ad8c17145ac466c032b7866091d33 for a full test report.

@vanzod vanzod added the bug fix label Jan 18, 2019
@vanzod vanzod added this to the 3.8.1 milestone Jan 18, 2019
@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 18, 2019

@vanzod Can you use configopts += "..." to avoid making the line too long?

Also, why is this problem only popping up for you?!

@vanzod
Copy link
Copy Markdown
Member Author

vanzod commented Jan 18, 2019

@boegel I have no idea why I am the only one reporting this issue. However I see no harm in disabling this specific error to prevent it from stopping the build.

@vanzod
Copy link
Copy Markdown
Member Author

vanzod commented Jan 18, 2019

Test report by @vanzod
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
cermis - Linux debian 9.6, Intel(R) Xeon(R) CPU E5620 @ 2.40GHz, Python 2.7.13
See https://gist.github.com/976c570689c0a7677a55018e5e382833 for a full test report.

@easybuilders easybuilders deleted a comment from boegelbot Jan 18, 2019
boegel
boegel approved these changes Jan 18, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 18, 2019

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node3102.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/4ea404e948ce4ae0d19e22c931d50030 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 18, 2019

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2012.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/395b13393b0afcebaddc0fca8e61c37b for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 18, 2019

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 18, 2019

Going in, thanks @vanzod!

@boegel boegel merged commit 47bcdc7 into easybuilders:develop Jan 18, 2019
@boegel boegel mentioned this pull request Jan 18, 2019
Merged
@vanzod vanzod mentioned this pull request Jan 18, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

3.8.1

Development

Successfully merging this pull request may close these issues.

2 participants

@vanzod @boegel