Re: Review Request 124184: Remove Qt3 dependencies from meanwhile protocol

Pali Rohár <[email protected]>
Newsgroups gmane.comp.kde.devel.kopete
Message-ID <[email protected]>

> On jún 26, 2015, 6:56 dopoludnia, Laurent Montel wrote:
> > protocols/meanwhile/CMakeLists.txt, line 4
> > <https://git.reviewboard.kde.org/r/124184/diff/1/?file=381418#file381418line4>
> >
> >     Why this definition ?
> 
> R.Harish  Navnit wrote:
>     It was suggested by Pali on the mailing list. It enables warnings for all deprecated functions and all qt3/kde3 functions, so that we get an idea of everything that needs to be ported to KDE4 at compile time.

It was suggestion for you when porting code :-) not something which should be in Kopete tree enabled by default.


- Pali


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124184/#review81762
-----------------------------------------------------------


On jún 26, 2015, 6:28 dopoludnia, R.Harish  Navnit wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124184/
> -----------------------------------------------------------
> 
> (Updated jún 26, 2015, 6:28 dopoludnia)
> 
> 
> Review request for Kopete, Laurent Montel and Pali Rohár.
> 
> 
> Repository: kopete
> 
> 
> Description
> -------
> 
> Replace the Q3GroupBox widgets with QGroupBox instead.
> 
> 
> Diffs
> -----
> 
>   protocols/meanwhile/CMakeLists.txt 5b3e7d468bda985add7a83c1b3f9a9939c8ef81f 
>   protocols/meanwhile/ui/meanwhileeditaccount.ui 5e61aed483fb2501a26aa2452247e0a61782ed8d 
> 
> Diff: https://git.reviewboard.kde.org/r/124184/diff/
> 
> 
> Testing
> -------
> 
> Build succeeded.
> 
> 
> Thanks,
> 
> R.Harish  Navnit
> 
>

_______________________________________________
kopete-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kopete-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.