Jan Kiszka wrote:
Philippe Gerum wrote:
Jan Kiszka wrote:
this patches cleans up the include/rtdm folder by moving internal
headers to ksrc/skins/rtdm, leaving only rtdm.h, rtdm_driver.h, the two
profile headers, and syscall.h there. The latter is still only needed
for building Xenomai itself, thus it will not be installed. Successfully
built for 2.4 and 2.6. Additionally, I compiled latest RTnet SVN against
it without problems (x86, PPC is currently being fixed by Wolfgang).
Please apply/move the involved headers and run bootstrap.
This one broke the build here:
kernel/xenomai/skins/rtdm/core.c:36:18: core.h: No such file or directory
kernel/xenomai/skins/rtdm/core.c:37:20: device.h: No such file or directory
Updated your repos? As you forgot to apply the moving, I also got
problems and applied my local versions of core.h, device.h, and proc.h,
also removing the three from their previous location.
The problem is that the patch does not recreate those files in
ksrc/skin/rtdm, but only deletes them from include/rtdm.
Additionally, it's a really very bad idea to start including things like
"core.h" removing the rtdm/ prefix, while other include files like
nucleus/core.h exist. For the sake of readability, I'm going to revert
this particular change at least.
That's why I use #include "core.h", which -for me- clearly states: "Pick
the local one!"
Nope, it "states pick the first one you find outside of the system dirs
depending on the order of the directories listed by -I in your
Makefile", which is quite different, and leads to all sort of
braindamage issues if you happen to screw up your CFLAGS in your Makefile.
You will now have to tweak the Makefiles again...
That's ok, I will do that. The prefix rule is enforced everywhere for
internal headers so that nobody gets mad trying to find why the wrong
one is included, and the best way not to depend on "-I" pecularities is
to avoid playing with them in the first place. Additionally, this leaves
no doubt about which header you wanted to include, regardless of the
sanity of your Makefile.