[Kde-games-devel] Review Request 126124: Port to KF5/QT5

[Albert Astals Cid]

> On des. 27, 2015, 12:32 p.m., Laurent Montel wrote:
> > Seems ok for me.
> > Ship it!
> > Thanks

There's two issues here:

A) Manoharan does not have a developer account
B) knights is part of extragear

A) is easily solvable, Laurent or me or someone else can commit the code but committing the code without making a release at some point doesn't give us much.

Because B) a release won't automatically happen, so we need someone to do a release.

Manoharan do you feel like you would want to become the maintainer of knights? After all noone else has done much work on it since 2012. By that we would give you commit access and you should also commit to doing a release.

What do you think?


- Albert


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


On des. 27, 2015, 9:56 a.m., Manoharan Sundaramoorthy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126124/
> -----------------------------------------------------------
> 
> (Updated des. 27, 2015, 9:56 a.m.)
> 
> 
> Review request for KDE Games, Knights and Miha Čančula.
> 
> 
> Repository: knights
> 
> 
> Description
> -------
> 
> Knights: Port to KF5/QT5
> 
> 
> Diffs
> -----
> 
>   src/proto/localprotocol.cpp 711ec5c 
>   src/proto/ficsprotocol.cpp 2c9b07d 
>   src/proto/ficsprotocol.h 3d3e395 
>   src/proto/ficsdialog.ui 8eea2e7 
>   src/proto/ficsdialog.cpp d89a07c 
>   src/proto/ficsdialog.h 93d5564 
>   src/offerwidget.cpp 0993ae2 
>   src/org.kde.knights.desktop PRE-CREATION 
>   src/knightsview.cpp 5451eb9 
>   src/knights.desktop 400cf57 
>   src/knightsview.h 40df529 
>   src/knights.cpp cb688a7 
>   src/historywidget.cpp 9f7f3fd 
>   src/gamemanager.h fe7f883 
>   src/gamedialog.cpp 2d0a124 
>   src/externalcontrol.h 40dbf55 
>   src/enginesettings.ui eb59af2 
>   src/enginesettings.cpp 4802ce0 
>   src/core/pos.cpp 8623c27 
>   src/customdifficultydialog.ui f9f7b29 
>   src/core/item.cpp 30fe74e 
>   src/clock.cpp 1e13366 
>   src/clockwidget.h 5686a95 
>   src/board.cpp 775cfd8 
>   icons/hi64-app-knights.png df75196 
>   src/CMakeLists.txt 6d77801 
>   icons/hi32-app-knights.png 120c7b2 
>   doc/CMakeLists.txt 5aad7c2 
>   icons/16-apps-knights.png PRE-CREATION 
>   src/popup.ui d4665af 
>   src/promotiondialog.ui 810e67d 
>   src/proto/chatwidget.h 4afcdf6 
>   src/proto/chatwidget.cpp fa209f9 
>   src/proto/chatwidget.ui 82ff8b7 
>   src/proto/computerprotocol.h 619071f 
>   src/proto/computerprotocol.cpp a0b032f 
>   src/main.cpp 504fcc5 
>   src/historywidget.ui aacc0be 
>   src/gamemanager.cpp c9d9566 
>   src/historywidget.h 0d9f823 
>   src/gamedialog.ui 436887d 
>   src/externalcontrol.cpp 1467491 
>   src/gamedialog.h 5518290 
>   src/enginesettings.h 2942ad1 
>   src/core/move.h 50e32b0 
>   src/core/move.cpp 4f7ca0e 
>   src/core/piece.cpp 4564ae8 
>   src/core/pos.h 343a8b5 
>   src/clockwidget.cpp 2a4e744 
>   src/clock.h 340c885 
>   src/board.h d464bd1 
>   icons/hi48-app-knights.png 26684f0 
>   CMakeLists.txt ed25a68 
>   icons/CMakeLists.txt 7affd27 
>   icons/hi16-app-knights.png ee4a3d9 
>   icons/32-apps-knights.png PRE-CREATION 
>   icons/48-apps-knights.png PRE-CREATION 
>   icons/64-apps-knights.png PRE-CREATION 
>   src/proto/protocol.cpp e489d33 
>   src/proto/seekgraph.h cba3b5f 
>   src/proto/seekgraph.cpp 3c83591 
>   src/proto/textprotocol.h 8216f24 
>   src/proto/textprotocol.cpp d6f5290 
>   src/proto/uciprotocol.cpp 69d7efb 
>   src/proto/xboardprotocol.h 280d18a 
>   src/proto/xboardprotocol.cpp 01fb4ae 
>   src/rules/chessrules.cpp 3e993e4 
>   src/rules/rules.h 032cde4 
> 
> Diff: https://git.reviewboard.kde.org/r/126124/diff/
> 
> 
> Testing
> -------
> 
> 1. Tested Playing against gnuchess
> 2. Two human players
> 3. Playing with clock
> 
> 
> Thanks,
> 
> Manoharan Sundaramoorthy
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20151228/ea2a3608/attachment-0001.html>