Skip to content

remove include/GL/wglext.h from Mesa sanity check#4354

Merged
boegel merged 2 commits intoeasybuilders:developfrom
damianam:fix_mesa_sanity_check
Mar 17, 2017
Merged

remove include/GL/wglext.h from Mesa sanity check#4354
boegel merged 2 commits intoeasybuilders:developfrom
damianam:fix_mesa_sanity_check

Conversation

@damianam
Copy link
Copy Markdown
Member

Fixes #4352

@boegel boegel added this to the 3.1.2 milestone Mar 16, 2017
'include/GL/glxext.h', 'include/GL/glx_mangle.h',
'include/GL/wmesa.h'],
'include/GL/gl.h', 'include/GL/glxext.h',
'include/GL/glx_mangle.h', ^'include/GL/wmesa.h'],
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

^ causes syntax error

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Argh! Sorry, it should be fixed now.

'include/GL/glxext.h', 'include/GL/glx_mangle.h',
'include/GL/wmesa.h'],
'include/GL/gl.h', 'include/GL/glxext.h',
'include/GL/glx_mangle.h', ^'include/GL/wmesa.h'],
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

^ causes syntax error

@boegel boegel changed the title Fixed wrong sanity check in Mesa remove include/GL/wglext.h from Mesa sanity check Mar 17, 2017
@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 17, 2017

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 17, 2017

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 17, 2017

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 17, 2017

Going in, thanks @damianam!

@boegel boegel merged commit f70613c into easybuilders:develop Mar 17, 2017
@damianam damianam deleted the fix_mesa_sanity_check branch July 12, 2017 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants