Bug 345342

Summary: workspace 5.2 fails to be build with kjobwidgets git master (2015-03-19)
Product: [Plasma] plasmashell Reporter: Harald Sitter <sitter>
Component: generalAssignee: David Edmundson <kde>
Status: RESOLVED FIXED    
Severity: grave CC: bhush94, nowrep, plasma-bugs
Priority: NOR    
Version: master   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Harald Sitter 2015-03-19 16:55:13 UTC
jobviewadaptor.cpp: In member function 'void JobViewV2Adaptor::setError(uint)':
jobviewadaptor.cpp:57:15: error: 'class JobView' has no member named 'setError'
     parent()->setError(errorCode);
               ^

problem: kjobwidgets master has a new function in the XML setError(uint), the plasma-workspace build creates an adaptor for it using the JobView class from kuiserver. The JobView class however has no setError(uint) in workspace < 5.3 and thus fails to build.


Reproducible: Always
Comment 1 David Rosca 2015-03-19 16:59:17 UTC
This will be fixed once https://git.reviewboard.kde.org/r/122970/ is submitted.
Comment 2 David Edmundson 2015-03-22 17:12:42 UTC
So this is fixed then?
Comment 3 David Rosca 2015-03-22 20:06:18 UTC
Git commit abafc8a6e72e4bb8d84cac2863d8513ed50aff00 by David Rosca.
Committed on 22/03/2015 at 20:03.
Pushed by drosca into branch 'master'.

Move added setError method to org.kde.JobView.xml

Reuse org.kde.JobView.xml file that is no longer used anywhere.
It now contains the current org.kde.JobViewV2 interface spec.

org.kde.JobViewV2.xml is still available, but only for backwards
compatibility.

This fixes source compatibility break from
812d0b440e7f08e3acdd1c9bb95779bd871b75b3
REVIEW: 122970

M  +0    -1    src/CMakeLists.txt
M  +0    -1    src/kuiserverjobtracker.cpp
M  +0    -1    src/kuiserverjobtracker_p.h
M  +9    -2    src/org.kde.JobView.xml
M  +0    -4    src/org.kde.JobViewV2.xml

http://commits.kde.org/kjobwidgets/abafc8a6e72e4bb8d84cac2863d8513ed50aff00