Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 9 Jul 2005 08:11:02 -0400
From:      John Baldwin <jhb@FreeBSD.org>
To:        soc-tyler <soc-tyler@FreeBSD.org>
Cc:        Perforce Change Reviews <perforce@FreeBSD.org>
Subject:   Re: PERFORCE change 79785 for review
Message-ID:  <200507090811.03859.jhb@FreeBSD.org>
In-Reply-To: <200507082347.j68NlN7L001942@repoman.freebsd.org>
References:  <200507082347.j68NlN7L001942@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Friday 08 July 2005 07:47 pm, soc-tyler wrote:
> http://perforce.freebsd.org/chv.cgi?CH=79785
>
> Change 79785 by soc-tyler@soc-tyler_launchd on 2005/07/08 23:46:51
>
> 	Remove -lexpat from Makefile, we're not going with XML for now.
> 	Comment out all the CoreFoundation related functions and begin to
> 	implement replacement functions for them (using libutil, etc)
>
> Affected files ...
>
> .. //depot/projects/soc2005/launchd/launchctl/Makefile#4 edit
> .. //depot/projects/soc2005/launchd/launchctl/launchctl.c#4 edit
>
> Differences ...
>
> ==== //depot/projects/soc2005/launchd/launchctl/Makefile#4 (text+ko) ====
>
> @@ -9,7 +9,7 @@
>  LAUNCHD=  ${.CURDIR}/../
>
>  CFLAGS+= -g -Wall -W -Wshadow -Wpadded -I${LAUNCHD}/includes
> -LDADD+=	-lreadline
> +LDADD+=	-lreadline -lutil
>
>  .PATH: ${LAUNCHD}
>
>

Doesn't look like -lexpat was ever in the list. :)  Note that there is a 
version of libexpat in the base system for system utilities to use that is 
called libbsdxml.  libgeom uses it to parse GEOM's XML tree.  So, if the 
reason you decided to not do XML was to avoid importing libexpat into the 
tree, you don't have to worry about that, it's already there.

-- 
John Baldwin <jhb@FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve"  =  http://www.FreeBSD.org



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200507090811.03859.jhb>