[Konsole-devel] Review Request 126808: [WIP] Port away from KDELibs4Support

[Dāvis Mosāns]

> On Jan. 19, 2016, 4:08 a.m., Dāvis Mosāns wrote:
> > src/main.cpp, line 167
> > <https://git.reviewboard.kde.org/r/126808/diff/1/?file=434618#file434618line167>
> >
> >     No idea about this, maybe it's not needed anymore at all?
> 
> Kai Uwe Broulik wrote:
>     I *think* QCommandLineParser / Q(Gui,Core)Application handle the Qt args automatically

It does handle their own argument parsing automatically, but here we don't parse them but check for their existence, only I'm not sure if this works at all...
See comment
```
    // The "unique process" model of konsole is incompatible with some or all
    // Qt/KDE options. When those incompatible options are given, konsole must
    // use new process
```


- Dāvis


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


On Feb. 23, 2016, 9:21 p.m., Dāvis Mosāns wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126808/
> -----------------------------------------------------------
> 
> (Updated Feb. 23, 2016, 9:21 p.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Repository: konsole
> 
> 
> Description
> -------
> 
> Port away from KUniqueApplication, and from KCmdLineArgs to
> QCommandLineParser. Remove KDELibs4Support dependency.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt c498aba89ed59333f2214f52e1b500ca9876233d 
>   src/Application.h b7389ef3d9a2d12a334fff00af3e2f9817d1289a 
>   src/Application.cpp 326c766fb5ff8669bcbce6f2d4d72f8764681400 
>   src/CMakeLists.txt 679807931585644000be0a38ccd67fe5158b3fa5 
>   src/MainWindow.h 500b39561116ce4ce72fa7d3a742c5a57c2b49ef 
>   src/MainWindow.cpp 822e45d57635dd7328eaec7db3eae41cdf19e61b 
>   src/SessionController.h 02f51d9db679eb32d6403ee1103b40842b0c668e 
>   src/SessionListModel.cpp fb28bb806d270b019978684bfeb1052a83afea03 
>   src/autotests/PartTest.cpp b3e7797bd5c52ca5449996099275888936ae9528 
>   src/main.cpp 5e1420e37363454cf5e7886a82381cb9eb4447d8 
>   src/settings/ProfileSettings.cpp d6dafbf17285192727679f92e91744e6c00b2c50 
> 
> Diff: https://git.reviewboard.kde.org/r/126808/diff/
> 
> 
> Testing
> -------
> 
> compiles/builds and basic things seems to be working
> 
> 
> Thanks,
> 
> Dāvis Mosāns
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20160223/f43f79dc/attachment-0001.html>