Sorry for the mess

David Zeuthen david at fubar.dk
Mon Jun 5 19:03:50 PDT 2006


On Tue, 2006-05-23 at 16:39 -0400, Joe Marcus Clarke wrote:
> > No, it's not good to commit. This would not work with the current CVS HEAD 
> > because there is no place where HALD_COMPILE_FREEBSD is defined. Need to add 
> > attached patch to configure.
> 
> Yeah, it works with my test code since I have the backend.  I will have
> to merge the backend soon.

I've now committed both your and Danny's patch. Feel free to commit your
FreeBSD backend at any time even if it's not finished since it's
probably better to have the code in CVS for more eyeballs etc. Same for
the Solaris backend.

Also, earlier today I've fixed some issues with the tools directory.
Basically the code needs to work when using ./run-hald.sh
(plus ./valgrind-hald.sh and ./debug-hald.sh) from the hald directory,
e.g. running with scripts and fdi files from the source tree. Mostly
because it's convenient. You should use it too! 

So I've changed

 1. Had a fd.o admin (thanks ajax!) do a chmod a+x on the files in 
    tools/linux and tools/freebsd

 2. Each script in tools got this patch

-    exec $0-$HALD_UNAME_S $@
+    exec ./$HALD_UNAME_S/hal-system-lcd-set-brightness-$HALD_UNAME_S $@

    which makes it a bit more copy-paste unfriendly but basically it
    works.

    David




More information about the hal mailing list