This request is superseded by
request 899041
(Show diff)
Overview
Request 894926 superseded
No description set
- Created by dirkmueller
- In state superseded
- Superseded by 899041
- Open review for factory-staging
Loading...
Request History
dirkmueller created request
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
licensedigger accepted review
ok
dimstar_suse set openSUSE:Factory:Staging:E as a staging project
Being evaluated by staging project "openSUSE:Factory:Staging:E"
dimstar_suse accepted review
Picked "openSUSE:Factory:Staging:E"
dimstar_suse added factory-staging as a reviewer
Being evaluated by group "factory-staging"
dimstar_suse accepted review
Unstaged from project "openSUSE:Factory:Staging:E"
dimstar accepted review
dimstar_suse superseded request
Fix submitted
Fails the build in ceph now:
Well well well. Someone had the audacity to add
-fno-rtti -fno-exceptions
and produce some really non-standard C++. Of course ceph (which, for all intents and purposes, uses C++ in standards-conforming mode) can't use it.Upstream thread for
-fno-rtti
: https://github.com/google/leveldb/issues/731They suggest to either enable
-fno-rtti
for other libs as well (ceph in the case) or to patch it out. Building ceph with-fno-rtti
is not a good idea, and it even does not work as ceph usestypeid
. Probably removing-fno-rtti
should not do any harm, as it should only increase the resulting file size.At least when tested it, both testsuites (ceph and leveldb) succeeded with a patched leveldb.
The patched version I spoke of is this one: SR#898809
breaks ceph
Unignored: returned to active backlog.