Summary: | x11-libs/qt-core-4.5.0 compile error on PPC64 | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Christian Schmitt <chrschmitt> |
Component: | New packages | Assignee: | Qt Bug Alias <qt> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | ppc64 |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | PPC64 | ||
OS: | Linux | ||
URL: | http://www.qtsoftware.com/developer/task-tracker/index_html?method=entry&id=245091 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 261632, 263049 | ||
Attachments: |
updated patch
fixed qt-core ebuild |
Description
Christian Schmitt
2009-02-17 19:40:51 UTC
Still the same with 4.5.0 final. Any suggestions? Ok, I found a patch for the issue on the qt bugtracker: http://www.qtsoftware.com/developer/task-tracker/index_html?method=entry&id=245091 I already tested it on PPC64 and it works. Could we add it to the tree? Yes, as long as this patch works on ppc64 arches I dont see any reason why we shouldnt add it. Unfortunately I cant test it here cause I dont have such an arch, so lets wait for another confirmation before I add it to the tree I will attach the ebuild and the patch for convenience Thats weird This patch seems to already be applied on final qt-4.5.0 This is the related section on my pc ( without applying the patch ) ----------------------------------- inline bool QBasicAtomicInt::testAndSetRelaxed(int expectedValue, int newValue) { register int result; asm volatile("lwarx %[result]," _Q_VALUE "\n" "xor. %[result], %[result], %[expectedValue]\n" "bne $+12\n" "stwcx. %[newValue]," _Q_VALUE "\n"<<<-------------we care about this line "bne- $-16\n" : [result] "=&r" (result), ------------------------------------------------------------------ So are you sure that final qt-4.5.0 still fails?If so, please attach here a valid patch Thanks The patch from the tracker is valid and portage applies it cleanly. I am absolutely sure that it is still needed, as my last attempt without the patch did not work. The only thing not correct in the patch is the line number. I'll attach a corrected version. Created attachment 184292 [details, diff]
updated patch
Created attachment 184294 [details]
fixed qt-core ebuild
Ok, I am attaching the ebuild now. I will wait a little bit longer for further testing and then I will commit it to the tree
This problem is fixed, but the next one is right behind the corner. See bug 261632 *** Bug 261632 has been marked as a duplicate of this bug. *** (In reply to comment #8) > This problem is fixed closing this then. |