Hide Forgot
Created attachment 379427 [details] disable make check for fedora-arm disable make check for fedora-arm -%ifnarch ppc sparcv9 +%ifnarch ppc sparcv9 armv5te Can you commit this into F-12 and devel ?
Rather than continuing to paper over the test failures, I'd rather see the tests fixed for these other architectures.
This bug appears to have been reported against 'rawhide' during the Fedora 13 development cycle. Changing version to '13'. More information and reason for this action is here: http://fedoraproject.org/wiki/BugZappers/HouseKeeping
Created attachment 402360 [details] Patch to resolve test failures The test failure is in tst-ikstack. The failure occurs on architectures where doubles are 64bit aligned, but pointers are smaller, hence ppc, sparc and arm have the problems. The issue is that the ikstack and ikschunk structures are not necessarily aligned to 64bits, and in fact ikschunk, with 32bit pointers, is 20 bytes long, and hence not aligned. The patch forces alignment of the two structures, and more importantly makes the sizes of the structs multiples of the size of a double. I have made a couple of other small changes: i) The ALIGN macro for a pointer that was already aligned was adding DEFAULT_ALIGNMENT bytes unnecessarily. The amendment also means that the parameter to the macro is only processed once. ii) The changing of char data[4] to char data[0] in the ikschunk struct avoids a wasted allocation of 4 bytes. The patch presumably needs to go upstream, but there has been no activity there for 8 months now. Could the patch please be applied in Fedora pending an update from upstream.
Sorry, I forgot to mention that I have done scratch builds on the primary Koji (both Intel and PPC), and also the secondaries for sparc and arm. The builds, including the tests, were all successful. I'm currently using F-12 on my Arm system, and so I would be grateful if the patch could be applied back that far.
Many thanks for applying the patch. Could it possibly also be applied for F-12, so that I can build it on the ARM Koji?
iksemel-1.4-2.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/iksemel-1.4-2.fc13
iksemel-1.4-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/iksemel-1.4-2.fc12
iksemel-1.4-2.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/iksemel-1.4-2.fc11
please remember to send patches to upstream!
iksemel-1.4-2.el5 has been submitted as an update for Fedora EPEL 5. http://admin.fedoraproject.org/updates/iksemel-1.4-2.el5
(In reply to comment #9) > please remember to send patches to upstream! I'll send them upstream, but upstream development seems pretty dead.
http://groups.google.com/group/iksemel/browse_thread/thread/64754924f179754b
iksemel-1.4-2.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update iksemel'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/iksemel-1.4-2.fc11
iksemel-1.4-2.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update iksemel'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/iksemel-1.4-2.fc12
iksemel-1.4-2.el5 has been pushed to the Fedora EPEL 5 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update iksemel'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/iksemel-1.4-2.el5
iksemel-1.4-2.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update iksemel'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/iksemel-1.4-2.fc13
iksemel-1.4-2.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
iksemel-1.4-2.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
iksemel-1.4-2.el5 has been pushed to the Fedora EPEL 5 stable repository. If problems still persist, please make note of it in this bug report.
iksemel-1.4-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.