Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 43260 - AbiWord does not build on amd64, patches included
Summary: AbiWord does not build on amd64, patches included
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: High trivial (vote)
Assignee: AMD64 Project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-02-29 00:48 UTC by Adam Polkosnik
Modified: 2004-03-17 08:55 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
patch 1 (abi1.diff,1.87 KB, patch)
2004-02-29 00:50 UTC, Adam Polkosnik
Details | Diff
patch 2 (abi2.diff,834 bytes, patch)
2004-02-29 00:51 UTC, Adam Polkosnik
Details | Diff
patch 3 (abi3.diff,1.25 KB, patch)
2004-02-29 00:52 UTC, Adam Polkosnik
Details | Diff
patch 4 (abi4.diff,2.65 KB, patch)
2004-02-29 00:52 UTC, Adam Polkosnik
Details | Diff
patch 5 (last) (abi5.diff,1.40 KB, patch)
2004-02-29 00:52 UTC, Adam Polkosnik
Details | Diff
ALPHA fix (abiword-2.0.5.ebuild,2.50 KB, text/plain)
2004-03-16 08:26 UTC, Kris Kersey (RETIRED)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Polkosnik 2004-02-29 00:48:10 UTC
xap_Draw_Symbol.cpp:156: error: reinterpret_cast from `const void*' to `
   UT_uint32' loses precision...




Reproducible: Always
Steps to Reproduce:
1. emerge abiword
2.
3.

Actual Results:  
!!! ERROR: app-office/abiword-2.0.3-r1 failed.
!!! Function src_compile, Line 61, Exitcode 2
!!! (no error message)

Expected Results:  
"successful build"

The fix changes:
reinterpret_cast<UT_uint32> to reinterpret_cast<unsigned long>
and
reinterpret_cast<UT_sint32> to reinterpret_cast<signed long>
Comment 1 Adam Polkosnik 2004-02-29 00:50:41 UTC
Created attachment 26561 [details, diff]
patch 1
Comment 2 Adam Polkosnik 2004-02-29 00:51:13 UTC
Created attachment 26562 [details, diff]
patch 2
Comment 3 Adam Polkosnik 2004-02-29 00:52:00 UTC
Created attachment 26563 [details, diff]
patch 3
Comment 4 Adam Polkosnik 2004-02-29 00:52:25 UTC
Created attachment 26564 [details, diff]
patch 4
Comment 5 Adam Polkosnik 2004-02-29 00:52:53 UTC
Created attachment 26565 [details, diff]
patch 5 (last)
Comment 6 Adam Polkosnik 2004-02-29 01:06:38 UTC
AbiWord 2.0.3-r1
Comment 7 Kris Kersey (RETIRED) gentoo-dev 2004-03-02 08:48:19 UTC
Merged all the patches into one patch and added to 2.0.3-r1.  Marked ~amd64.
Comment 8 Kris Kersey (RETIRED) gentoo-dev 2004-03-02 08:49:44 UTC
Please send these upstream to abiword.  They need to make a more elegant cross-platform patch.
Comment 9 Hubert Figuiere 2004-03-04 23:21:17 UTC
We definitely need to address this issue, differently from what you do because it is quick and dirty :-/

Thanks for the patch and for handling the issue in Gentoo.

BTW, don't expect the patch to appear in 2.0.x tree.
Comment 10 Allen Ziegenfus 2004-03-16 08:14:43 UTC
alpha needs this patch too.  Could someone please add it to the 2.0.5 ebuild? I had to add this to get abiword to work: 

 if [ $ARCH = "alpha" ] ; then
        epatch ${FILESDIR}/${PN}-2.0.3-amd64.patch || die
    fi
Comment 11 foser (RETIRED) gentoo-dev 2004-03-16 08:24:54 UTC
alpha team check comment #10

i wonder why this would be needed now, while abi always worked on alpha (?)
Comment 12 Kris Kersey (RETIRED) gentoo-dev 2004-03-16 08:26:41 UTC
Created attachment 27476 [details]
ALPHA fix

Allen,

Try this ebuild out.  If it works, I will commit it.  From now on, you should
file something like this as a new bug report for the ALPHA team but I'll take
it this time. ;-)
Comment 13 Allen Ziegenfus 2004-03-16 17:39:59 UTC
Kris,

ooh. C'est bon. The amended ebuild is good. 

Do I signal the "alpha" team by just indicating the hardware type as alpha or is there some other field to set as well? I was debating about whether I should create a new bug or not. Sorry bout that -- I'm new to gentoo.
Comment 14 Kris Kersey (RETIRED) gentoo-dev 2004-03-17 08:55:19 UTC
Any word from the alpha team before I commit this?