Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 216800 - rapidsvn breaks strict aliasing rules
Summary: rapidsvn breaks strict aliasing rules
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: Steve Arnold
Depends on:
Reported: 2008-04-08 00:24 UTC by Sergey Ilinykh
Modified: 2008-05-19 19:42 UTC (History)
2 users (show)

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

rapidsvn-sar.patch (rapidsvn-sar.patch,931 bytes, patch)
2008-04-08 00:25 UTC, Sergey Ilinykh
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey Ilinykh 2008-04-08 00:24:28 UTC
with new portage 2.1.5_rc2 rapidsvn fails to install because breakage of strict aliasing rules.

Reproducible: Always

Steps to Reproduce:
1. just try to install
Actual Results:  
stops on install

i've wrote patch
Comment 1 Sergey Ilinykh 2008-04-08 00:25:48 UTC
Created attachment 149066 [details, diff]

this patch fixes this
Comment 2 Steve Arnold archtester gentoo-dev 2008-04-09 01:39:07 UTC
Thanks for the patch; it tests out fine.  How'd you like to tackle the two remaining source files that spew the same errors?  Not the slew of wx header files, but the two rapidsvn source files:

hist_mgr.cpp and folder_browser.cpp  (I don't see any others than these two)

I believe the official Gentoo policy is that we don't fix these kinds of errors, since it's an upstream programming issue, but nobody will stop you from submitting another patch :)

You can also adjust your features to allow it to install as-is (I've tested it lightly with my local repo, and it seems to work fine).
Comment 3 Sergey Ilinykh 2008-04-14 12:51:42 UTC
ok. i've posted bug report to rapidsvn tracker.
but seems rapidsvn developers to lazy to review it.
Comment 4 Alexander Huemer 2008-05-18 11:27:51 UTC
the rapidsvn-sar.patch is listed in the ebuild of rapidsvn-0.9.6-r1, but it is missing in the portage tree. therefor emerge fails.
please add the patch to the tree.
Comment 5 Ryan Hill (RETIRED) gentoo-dev 2008-05-18 14:35:46 UTC
not sure if you wanted the patch included or if you forgot to remove the epatch line.  reopening.
Comment 6 Steve Arnold archtester gentoo-dev 2008-05-19 19:42:06 UTC
Yeah, the patch is in...  It got dropped on the floor somehow...