Re: [PATCH] to run on IA64

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Feb 19, 2007 at 05:50:51PM +0900, Atsushi SAKAI wrote:
> Hi, Dan
> 
> Thank you for your comments.
> 1) This patch is for rpmbuild failure on IA64.
>       see the attached log (it is stopped in conf.c compilation)
> 
> 2) About above fixes code.(endian issue)
>    Sorry for short discripting this issue.
>    My point is you are casting (unsigned int*) for unsigned char.
>    In little endian, this works fine?$B!!(conf.c).
>    But big endian case, this inputs "00" for character value.

Ahhhhhhhh. That makes much more sense to me now - explains why I never
saw any problem on x86_64/i686. Will fix this in CVS.

> conf.c: In function 'qemudParseInterfaceXML':
> conf.c:450: warning: cast increases required alignment of target type
> conf.c:451: warning: cast increases required alignment of target type
> conf.c:452: warning: cast increases required alignment of target type
> conf.c:453: warning: cast increases required alignment of target type
> conf.c:454: warning: cast increases required alignment of target type
> conf.c:455: warning: cast increases required alignment of target type
> make[2]: *** [libvirt_qemud-conf.o] Error 1
> make[2]: Leaving directory `/home/sakaia/rpmbuild/BUILD/libvirt-0.2.0/qemud'
> make[1]: *** [all-recursive] Error 1
> make[1]: Leaving directory `/home/sakaia/rpmbuild/BUILD/libvirt-0.2.0'
> make: *** [all] Error 2
> error: Bad exit status from /var/tmp/rpm-tmp.48812 (%build)
> 

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]