On Wed, May 20, 2009 at 07:40:43AM -0500, Schley Andrew Kutz wrote:
I will create a specific sub-dir and let you know.
okay thanks, any feedback on the two other issues ?
>>> On Thu, May 07, 2009 at 11:50:25PM -0500, Schley Andrew
Kutz wrote:
>>>> - Set environment variables:
>>>>
>>>> export LDFLAGS="-L/opt/local/lib"
>>>> export CPPFLAGS="-I/opt/local/include"
>>>> export MACOSX_DEPLOYMENT_TARGET=10.4
>>>> src/virsh.c:5665
[...]
>>>>
>>>> if (command_ret != 0 /* WEXITSTATUS (0) */) {
>
> That's bizarre ...
>
> WEXITSTATUS is defined in virsh.c:
>
> #ifndef WEXITSTATUS
> # define WEXITSTATUS(x) ((x) & 0xff)
> #endif
>
> it's used only once at the place you pointed out:
>
> if (command_ret != WEXITSTATUS (0)) {
>
> I think it was used for cygwin portability, but in that
> case I would have expected
>
> if (WEXITSTATUS(command_ret) != 0) {
>
> Why did this break on OS-X ?
Any idea about this ?
>>> That's great - we can easily fix these 2 bugs.
>>>
>>>> - Compile
>>>>
>>>> The MACOSX_DEPLOYMENT_TARGET variable is very important, otherwise
>>>> you
>>>> will get symbol errors when linking.
>
> What about detecting MACOSX_DEPLOYMENT_TARGET, because I assume it
> will change from one environment to another, do this in configure.in
> and
> export is in all Makefiles.am ? There must be a way to export the env
> variable from the generated Makefiles surely...
and this ?
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit
http://xmlsoft.org/
daniel(a)veillard.com | Rpmfind RPM search engine
http://rpmfind.net/
http://veillard.com/ | virtualization library
http://libvirt.org/