By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 2897 - Patches for 3 koffice build failures with clang
Summary: Patches for 3 koffice build failures with clang
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: non-core programs (show other bugs)
Version: R14.0.x [Trinity]
Hardware: All Linux
: P5 normal
Assignee: Timothy Pearson
URL:
Depends on:
Blocks: R14.0.5
  Show dependency tree / graph
 
Reported: 2018-05-06 09:52 CDT by Ray Vine
Modified: 2018-05-21 18:13 CDT (History)
2 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ray Vine 2018-05-06 09:52:29 CDT 
Error 1:
In file included from ../../../../kspread/plugins/scripting/scripting.cc:30:
In file included from ../../../../lib/kross/main/manager.h:31:
../../../../lib/kross/main/../api/object.h:145:9: error: cannot use 'throw' with exceptions disabled
        throw TDESharedPtr<Exception>( new Exception(TQString("Object \"%1\" invalid.").arg(object ? object->getClassName() : "")) );
        ^

Patch:
Additional flag as per chalk/plugins/viewplugins/scripting/Makefile.am

--- kspread/plugins/scripting/Makefile.am
+++ kspread/plugins/scripting/Makefile.am
@@ -23,0 +24,2 @@
+
+KDE_CXXFLAGS = $(USE_EXCEPTIONS)

-------------

Error 2:
../../../kexi/migration/keximigratetest.cpp:41:17: error: call to constructor of 'TDEApplication' is ambiguous
        TDEApplication app(argc, argv, "Kexi Migrate Test");
                       ^   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Patch:
Based on 0004-fix-an-FTBFS-when-build-with-clang.patch @ http://trinity-devel.pearsoncomputing.net/?0::11397

--- kexi/migration/keximigratetest.cpp
+++ kexi/migration/keximigratetest.cpp
@@ -41 +41 @@
-	TDEApplication app(argc, argv, "Kexi Migrate Test");
+	TDEApplication app(argc, argv, TQCString("Kexi Migrate Test"));

-------------

Error 3:
In file included from ../../../../filters/kword/mswrite/mswriteimport.cc:43:
../../../../filters/kword/mswrite/mswriteimport.h:26:7: error: forward declaration of class cannot have a nested name specifier
class MSWrite::InternalParser;
      ^~~~~~~~~

Patch:
--- filters/kword/mswrite/mswriteimport.h
+++ filters/kword/mswrite/mswriteimport.h
@@ -26 +26 @@
-class MSWrite::InternalParser;
+namespace MSWrite { class InternalParser; }
Comment 1 Slávek Banko 2018-05-21 18:13:05 CDT 
Pushed to GIT in hash c4956ecd (master) and de36358e (r14.0.x).