On 06/15/2018 09:11 PM, Matthew R. Trower wrote:
Hey Jon, I see that you've started working on this a bit.

I had a look at your commits (as I also have an interest in this space),
and I'm watching this hardcoded line grow:
---
INCLUDES = -I../../include -I/usr/local/include/X11 -I/usr/include/X11 
-I/usr/X11R6/include/X11 -I../../imports/x11/include/X11
---


Might you consider this insted?
---
INCLUDES = -I../../include -I/$(XPROJECTROOT)/include/X11 
-I../../imports/x11/include/X11
---


Patch attached.  Test it of course on your available systems, but it
should work well.  It's essentially what you're trying to accomplish
here.


No, it fails. This is a "bootstrap" Makefile, used to start the ball rolling. At this point, XPROJECTROOT does not even exist, so it ends up looking for includes in //include/X11, which of course fails...

I think there's actually a more correct way to do this in X11.tmpl and
friends, but I took the conservative route for the moment.  Some day, I
want to sit down, have a long hard look at that config/cf directory,
and clean up a lot of what we're doing.  It's my feeling that there's a
shiny, smart config system buried under years of grime, just begging to
be let out =)


Of that I have no doubt. :)
-jon


-mrt



------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot



_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel


--
Jon Trulson

"Fire all weapons and open a hailing frequency for my victory yodle."

                              - Zapp Brannigan

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel

Reply via email to