Skip to content
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

AlienImage (TKService) should not depend on X11 #36

Closed
dbarbier opened this issue Jun 7, 2011 · 1 comment
Closed

AlienImage (TKService) should not depend on X11 #36

dbarbier opened this issue Jun 7, 2011 · 1 comment

Comments

@dbarbier
Copy link
Collaborator

dbarbier commented Jun 7, 2011

AlienImage_X11XWDAlienData.cxx and AlienImage_X11XWDFileHeader.cxx include X11/Xlib.h because they need X macros.

Maybe we can remove this dependency by defining the few macros needed. A first step is to include X11/X.h instead of X11/Xlib.h.

@dbarbier
Copy link
Collaborator Author

Here are some additional information, this issue is quite complex and my description was pretty dumb.

In OCCT, OCAF and Data Exchange components depend on Visualization. Some people want to install OCCT on machines without an X server, so a proposal has been made on Debian list to split components:
http://lists.debian.org/debian-science/2008/01/msg00023.html
This proposal has been implemented almost as described.

In OCE, the situation is quite different, I did not want to introduce new components (OCAF-lite and DE-lite). And when cross-compiling for Mingw, TKService caused trouble because my cross-compiler does not have X11 stuff. So in order to compile the maximum stuff, an option has been added to compile TKService with or without X11 support (see commit 68d1d35).

Then OSX users also told that they want to get rid of X11 stuff whenever possible.

This explains the current mess, it would surely be a good idea to think about this issue from scratch and find a better plan.

dbarbier added a commit that referenced this issue Aug 18, 2011
…iled

This allows building OCAF and Data Exchange on computers without X11.

Closes issue #36.
@tpaviot tpaviot closed this as completed Aug 18, 2011
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

No branches or pull requests

2 participants