Sorry for the mess

[Danny Kukawka]

On Tuesday 23 May 2006 16:23, David Zeuthen wrote:
> On Tue, 2006-05-23 at 04:51 -0400, Joe Marcus Clarke wrote:
> > On Wed, 2006-05-17 at 18:35 -0400, David Zeuthen wrote:
> > > On Thu, 2006-05-18 at 00:03 +0200, Danny Kukawka wrote:
> > > >  SUBDIRS =  freebsd linux device-manager
> > > > +if HALD_COMPILE_LINUX2
> > > > +SUBDIRS = linux device-manager
> > > > +endif
> > > > +if HALD_COMPILE_SOLARIS
> > > > +SUBDIRS = solaris device-manager
> > > > +endif
> > > > +if HALD_COMPILE_FREEBSD
> > > > +SUBDIRS = freebsd device-manager
> > > > +endif
> > >
> > > This would probably break 'make dist' but I suppose the general idea is
> > > good and the tests needs to be in each $arch/Makefile.am.
> >
> > What about the attached patch?  It correctly puts all the subdirs in the
> > distfile, and will only install the tools for the right OS.
>
> Looks good to commit. Thanks!
>
>     David

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.

Danny

-------------- next part --------------
A non-text attachment was scrubbed...
Name: hal-fix-add_freebsd_detection_to_configure.diff
Type: text/x-diff
Size: 800 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/hal/attachments/20060523/f5dfa6a3/hal-fix-add_freebsd_detection_to_configure.bin