Review Request 110907: Move kdeprintdialog.h and kcups* to kde4support

[Albert Astals Cid]

> On June 9, 2013, 10:25 a.m., Albert Astals Cid wrote:
> > Have we confirmed that Qt5 print dialog provides the options we have here? Otherwise it shouldn't go to kde4support, no?
> 
> Kevin Ottens wrote:
>     Yes it should, it shouldn't be in our API. Of course, someone should fix Qt5 or our QPA plugin if it doesn't provide the features we need.

Just to confirm, are you saying we are ok with marking things as deprecated without a replacement?


- Albert


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110907/#review33977
-----------------------------------------------------------


On June 9, 2013, 1:54 p.m., Anne-Marie Mahfouf wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110907/
> -----------------------------------------------------------
> 
> (Updated June 9, 2013, 1:54 p.m.)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Description
> -------
> 
> Move kdeprintdialog.h and kcups* to kde4support
> Fix compilation 
> Mark as deprecated
> Things I am unsure of: the change in khtml/khtmlview.cpp and in kdeui/dialogs/kshortcutseditor.cpp, some CMakeLists moves as well are to be checked.
> 
> 
> Diffs
> -----
> 
>   staging/kde4support/src/kdeui/kdeprintdialog.h PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/110907/diff/
> 
> 
> Testing
> -------
> 
> Build
> 
> 
> Thanks,
> 
> Anne-Marie Mahfouf
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130611/73f4a02c/attachment.html>