Debian Bug report logs - #320155
xview: amd64 support

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Kaare Hviid <ukh@id.cbs.dk>

To: submit@bugs.debian.org

Subject: xview: amd64 support

Date: Wed, 27 Jul 2005 13:26:00 +0200

[Message part 1 (text/plain, inline)]

Package: xview
Version: 3.2p1.4-19
Severity: wishlist
Tags: patch sid

Trying to get XView into shape on amd64 has taken me way too long.
Considering the current transition mess and the fact that the patch
has been sitting idle without any work for a couple of months, I'd
better post it now, rather than waiting for my sid boxes to stabilize
in order to do thorough testing.  So far, everything appears to work
fine on amd64 with this patch.  So here goes...

The main purpose of this patch is adding amd64 support for XView.  It
also attepmts to fix the following issues:

*	Support for building with gcc-4.0.  It should also allow
	building with gcc-3.4 and gcc-3.3.

*	This patch *might* support building and running on ia64 too,
	provided you add ia64 to the Architecture line.  I have yet
	to find someone willing to test this patch on their ia64
	systems though.

Random ramblings follow...

The main fix is in libxview/attr/attr.c, where an _XV_API_BROKEN_64BIT
#define will enable filtering of attributes that appear suspect, much
in the same vain as suggested in #125391.  It's not 100% fool-proof
though and some attribute arguments, notably XV_KEY_DATA and
PANEL_VALUE, have to be carefully casted to (Attr_attribute) due to the
design.  Also, some attributes have been wrongly defined (c.f. the
headers), why applications built on amd64 and ia64 with the previously
non-functioning xview libraries have to be rebuilt.  I have taken care
*not* to change the attribute values for any other platforms.

However, the bulk of this patch consists of:

*	Casting of XView attributes and ensuring that attribute lists
	are NULL-terminated rather than 0-terminated.  In the case of
	gcc on Debian GNU/Linux, NULL should be #defined to (void *)0,
	ensuring that termination will be correct.  Now, this isn't
	needed thanks to the filter in attr.c, but it should be of no
	harm, so I decided not to remove them.
*	A lot of ANSI C prototypes.  They actually helped me to track
	down a couple of 64-bit related bugs.  I decided to let them
	stay as well.

There are also some other minor 64-bit fixes - and a lot of spots where
I couldn't figure out the Right thing to do - these are marked FIXME.

Unfortunately, olvwm will not work properly on amd64 when built with
gcc 3.3 using -O2.  gcc 3.4 and 4.0 works, as does -O0 and and -O3
with gcc 3.3.  Since we're now moving on to gcc 4.0 as the default
compiler, I simply do not find it very interesting to dig for the cause
of the problem.  However, I recently found that olvwm segfaults, both
the current version and my 4.0-recompiled version using this patch on
alpha, at least when started via an Xnest server.  I lack the resources
to try it on a real X server though.  I get a misaligned trap, and that
may very well be what happens on amd64 too with gcc 3.3.  I simply don't
have the time and energy to hunt down this one now - and I think it's
more important I get this patch out of the door first.

Some of the external applications that depend on XView have 64-bit
problems of their own - but I'll leave them for now until if or when our
XView maintainer decides he wants to go through with this patch.

Thanks!

-ukh

[xview.diff.bz2 (application/octet-stream, attachment)]

Message #10 received at 320155@bugs.debian.org (full text, mbox, reply):

From: Kaare Hviid <ukh@id.cbs.dk>

To: 320155@bugs.debian.org

Subject: patch does NOT work on ia64

Date: Fri, 29 Jul 2005 11:06:59 +0200

Thanks to Dan Frazier, I now know that my amd64 patch does NOT
work on ia64.  Many thanks to Dan Frazier for prompt testing and
patiently answering my questions!
    Also, even though olvwm does work on amd64, there clearly are
issues that I don't understand.  Until I understand why it doesn't
work correctly on alpha or when compiled with gcc-3.3 -O2, I *suggest*
that amd64 is dropped from the Architecture line of olvwm in my patch.

-ukh

Message #15 received at 320155-done@bugs.debian.org (full text, mbox, reply):

From: Ralf Treinen <treinen@free.fr>

To: 320155-done@bugs.debian.org

Subject: xview: amd64 support added

Date: Sun, 24 Jul 2011 21:04:54 +0200

Version: 3.2p1.4-24

According to the packages changelog, amd64 support was added in version
3.2p1.4-24 of the xview package.

-Ralf.

Send a report that this bug log contains spam.