Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xview - XView widget libraries
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214751
------- Additional Comments From j.w.r.degoede@hhs.nl 2006-11-09 14:21 EST ------- Created an attachment (id=140809) --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=140809&action=vie...) Patch with some interesting fixes from Suse
I've spend some hours taking a look and I've come to the same conclusion as the Debian maintainer, this is very hard to fix for 64 bit. More then that fixing probably will also include fixing / changing all xview using clients!
Luckily all 64 bit platforms we support also have a 32 bit compatibility option, so I think we should just not build xview (and apps using it) for x86_64 / ppc64. It would be a good idea IMHO in cases like this to add xview + deps + packages using it to a list of packages to copy over to the x86_64 repo from the i386 repo, so that it will be readily available for those who want it.
My taking a look started with suse since they had x86_64 packages of xview in their repo, but appearantly these have had the famous suse QA done do them (iow none). I did find some other interesting patches in there, which I have bundled in a smaller one with possible real fixes and a larger one which fixes a load of warnings (but no where near all warnings). I also have a patch which fixes some 64 bit related warnings by adding the necessary prototypes, which isn't enough to get this running but IMHO still should be applied / send upstream (Debian claims to be upstream these days) as it is an improvement.