-
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
Make failing for Tk-8.6.4-foss-2015b-no-X11.eb #2261
Comments
The intention is indeed that this Tk build doesn't require X11 headers, which is why we specify This is a known issue (to me), and I thought there was an open issue for this already, but I couldn't find it... I think @verdurin ran into this too? |
@nathanhaigh: it looks like including I'm puzzled why the X11 headers are required at all, when Tk is being configured using One option there is to contact http://wiki.tcl.tk/82 (i.e. CL linked from the other page), and ask him why this happens. |
@boegel I was in touch with some Tk folks on irc. No one really seemed to know what I tried a couple of things:
Although I got Tk installed by using my OS's |
Just done some investigation of builds using There is no difference (via |
It does feel like we need to fix the base python to remove Tk support and somehow remove this circular unresolvable dep. For the libX11 PR it still depends on Python which sends us into this hole. |
I resolved this by removing the Tk dep from Python. I don't think it is useful without X11 anyway and we can probably build it as a separate extension if it is needed |
@rjeschmi ignore my comment ;) its too morning for me... |
We now have libX11 easyconfigs that do not include a Python dependency anymore, so we can potentially solve this by adding libX11 as a (build) dep for Tk. I'm not sure that's an ideal situation though, it should still be possible to build Tk without X11 support? Any idea if that makes sense at all @nathanhaigh? cc @verdurin |
Just a note that this still happens with |
Just bumped into this one again, on a cloud-based cluster. Would be nice to fix it for |
@verdurin The only proper way to fix this, without relying on the OS, is to include libX11 as a dependency for Tk, which only works for the more recent toolchains where we got rid of the Python (build) dependency... |
I'm trying this with
whereas I've just built that module:
I've applied the framework patch in framework PR 1795 and it still fails. |
@boegel - it's very much based on that one:
|
To build Tk, you need an X11 stack. All current python easyconfig need X11 to build. Follow up in easybuilders#2261
Hi, I was able to successfully install Tk-8.6.4-foss-2015b-no-X11.eb only after installing an additional: libX11/1.6.3-foss-2015b and libX11/1.6.3-foss-2015b module. In Tk module as a dependency i manually added to load: libX11/1.6.3-foss-2015b and libX11/1.6.3-foss-2015b modules. |
@verdurin This can be closed now we include |
Yes it can. |
Trying to install this (a Python dependency) and I get errors from
make
like this:I assumed that since this is for
no-X11
then then I should get an error-free build. Am I missing something? For someone who know this stuff better than me, the following link may help: http://wiki.tcl.tk/1843.Does it require a dependency adding for supplying these headers or should the ones supplied by Tk in the
xlib/X11/
subdirectory.The text was updated successfully, but these errors were encountered: