Skip to content

disable generating of man pages in recent libdrm easyconfigs to avoid installation failure if docbook-xsl is not installed#11182

Merged
lexming merged 2 commits intoeasybuilders:developfrom
boegel:20200829095520_new_pr_libdrm24100
Aug 29, 2020
Merged

disable generating of man pages in recent libdrm easyconfigs to avoid installation failure if docbook-xsl is not installed#11182
lexming merged 2 commits intoeasybuilders:developfrom
boegel:20200829095520_new_pr_libdrm24100

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Aug 29, 2020 •
edited
Loading

(created using eb --new-pr)

This avoids a failing installation when docbook-xsl is not installed in the OS:

I/O I/O error : error : Attempt to load network entity http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
Attempt to load network entity http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
I/O warning: warning: error : failed to load external entity "http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl"
failed to load external entity "http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl"
Attempt to load network entity http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
cannot parse http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
cannot parse http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
warning: failed to load external entity "http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl"
cannot parse http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
make[2]: *** [Makefile:626: drmHandleEvent.3] Error 4

@boegel
disable generating of man pages in recent libdrm easyconfigs to avoid…
5b21c47 
… installation failure if docbook-xsl is not installed
@boegel boegel added the bug fix label Aug 29, 2020
@boegel boegel added this to the next release (4.3.0) milestone Aug 29, 2020
@lexming
Copy link
Copy Markdown
Contributor

lexming commented Aug 29, 2020

Test report by @lexming
SUCCESS
Build succeeded for 5 out of 5 (5 easyconfigs in this PR)
node315.hydra.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/664b366ecfed2dd52d6356ce1b1d2717 for a full test report.

@lexming
Copy link
Copy Markdown
Contributor

lexming commented Aug 29, 2020

Test report by @lexming
SUCCESS
Build succeeded for 5 out of 5 (5 easyconfigs in this PR)
node116.hydra.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz, Python 2.7.5
See https://gist.github.com/847e8bf4e3ec338c9092edd364f0db0b for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 29, 2020

Test report by @boegel
SUCCESS
Build succeeded for 5 out of 5 (5 easyconfigs in this PR)
node3170.skitty.os - Linux centos linux 7.8.2003, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/4f73f7543d66a08f755639528fcb7c12 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 29, 2020

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

lexming
lexming approved these changes Aug 29, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@lexming lexming left a comment

Choose a reason for hiding this comment

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

LGTM

@lexming
Copy link
Copy Markdown
Contributor

lexming commented Aug 29, 2020

Going in, thanks @boegel !

@lexming lexming merged commit d787c64 into easybuilders:develop Aug 29, 2020
@boegel boegel deleted the 20200829095520_new_pr_libdrm24100 branch August 29, 2020 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lexming lexming lexming approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.3.0

Development

Successfully merging this pull request may close these issues.

2 participants

@boegel @lexming