-
Notifications
You must be signed in to change notification settings - Fork 713
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
{vis}[intel/2016b] X11 v20160819 (REVIEW) #3440
{vis}[intel/2016b] X11 v20160819 (REVIEW) #3440
Conversation
That was fast! |
@boegel I would add some extra: Then we can build a proper Mesa (deps: x11, libdrm, LLVM, ...) on this. After Mesa Maybe also X-fonts: Thats all ;) |
@boegel Otherwise excellent work! Thx! |
# all extensions can be installed with configure/make/make install | ||
exts_defaultclass = 'ConfigureMake' | ||
|
||
preconfigopts = "if [ -f autogen.sh ]; then ./autogen.sh; fi && " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if [ ! -f configure ]; then ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
I agree with @hajgato. It's a good start but we need much more in the |
This is just a starting point, which should be enough to install Qt5 on top of this. I'm all for extending this bundle, but we need to agree that this is a good approach first (and I'm currently not 100% convinced myself that installing these packages as extensions is the best way). |
updated easyconfig to use enhanced |
Test report by @boegel |
allow_system_deps = [('Python', SYS_PYTHON_VERSION)] | ||
pyshortver = '.'.join(SYS_PYTHON_VERSION.split('.')[0:2]) | ||
|
||
preconfigopts = "if [ -f autogen.sh ]; then ./autogen.sh; fi && " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
! -f configure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, done
@boegel I also like it more! |
Test report by @wpoely86 |
Test report by @wpoely86 |
Test report by @boegel edit: was not using the updated |
Test report by @boegel |
Test report by @boegel |
Test report by @hajgato |
@boegel: on it. Otherwise the code looks good. |
Test report by @hajgato |
Test report by @hajgato |
Test report by @hajgato |
In #3463 I've used jhbuild to build the entire Xorg stack, about 240 packages (and excluding some problematic packages)...but it's the latest that's in the git repos of Xorg, not particular software releases |
Test report by @boegel |
Test report by @boegel |
retesting after minor update to Thanks for the feedback everyone! |
(created using
eb --new-pr
)requires
easybuilders/easybuild-easyblocks#979easybuilders/easybuild-easyblocks#980 and#3438