[Kde-games-devel] Review Request 121589: Rename binary to ksnakeduel

Ian Wadham iandw.au at gmail.com
Thu Feb 19 23:01:50 UTC 2015


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



CMakeLists.txt
<https://git.reviewboard.kde.org/r/121589/#comment52590>

    Should these files be renamed?



CMakeLists.txt
<https://git.reviewboard.kde.org/r/121589/#comment52591>

    Name still contains ktron.



CMakeLists.txt
<https://git.reviewboard.kde.org/r/121589/#comment52592>

    Still contains ktron.



ksnakeduel.knsrc
<https://git.reviewboard.kde.org/r/121589/#comment52588>

    There aeem to be still quite a few ktron's around. In this one, the link leads to a broken file.



ksnakeduelui.rc
<https://git.reviewboard.kde.org/r/121589/#comment52593>

    Uses ktron name.



main.cpp
<https://git.reviewboard.kde.org/r/121589/#comment52594>

    Contains name KTRON.


- Ian Wadham


On Feb. 19, 2015, 2:22 p.m., Jonathan Riddell wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121589/
> -----------------------------------------------------------
> 
> (Updated Feb. 19, 2015, 2:22 p.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Repository: ksnakeduel
> 
> 
> Description
> -------
> 
> For some reason ksnakeduel has three names. This gets rid of one of them, ktron and changes it to ksnakeduel. This was mostly done already but this changes the binary name too which was causing confusion to us little packagers.
> 
> I also think the kdesnake script and .desktop file should go away, you can set it in the UI.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 49616d7 
>   pics/hi32-app-kdesnake.png 27b133b 
>   pics/hi32-app-ksnakeduel.png PRE-CREATION 
>   pics/hi32-app-ktron.png f467683 
>   pics/hi48-app-kdesnake.png 556c9e1 
>   pics/hi48-app-ksnakeduel.png PRE-CREATION 
>   pics/hi48-app-ktron.png ac857f9 
>   pics/hi64-app-kdesnake.png e9d7ebe 
>   pics/hi64-app-ksnakeduel.png PRE-CREATION 
>   pics/hi64-app-ktron.png a79a401 
>   settings.kcfgc 8450b7d 
>   themes/CMakeLists.txt aad7fed 
>   doc/CMakeLists.txt 99fed14 
>   kdesnake.cmake 0a4e46d 
>   kdesnake.desktop 5e49105 
>   ksnakeduel.desktop PRE-CREATION 
>   ksnakeduel.kcfg PRE-CREATION 
>   ksnakeduel.knsrc PRE-CREATION 
>   ksnakeduelui.rc PRE-CREATION 
>   ktron.desktop 3b2e386 
>   ktron.kcfg ab764b9 
>   ktron.knsrc 9cc0835 
>   ktronui.rc baab667 
>   main.cpp 79fb05f 
>   pics/hi128-app-kdesnake.png 797a458 
>   pics/hi128-app-ksnakeduel.png PRE-CREATION 
>   pics/hi128-app-ktron.png 73da488 
>   pics/hi16-app-kdesnake.png de8f38a 
>   pics/hi16-app-ksnakeduel.png PRE-CREATION 
>   pics/hi16-app-ktron.png 0928dbf 
>   pics/hi22-app-kdesnake.png 1b7948b 
>   pics/hi22-app-ksnakeduel.png PRE-CREATION 
>   pics/hi22-app-ktron.png e2206df 
>   pics/hi256-app-kdesnake.png 270a652 
>   pics/hi256-app-ksnakeduel.png PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/121589/diff/
> 
> 
> Testing
> -------
> 
> compile, install and run
> 
> 
> Thanks,
> 
> Jonathan Riddell
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20150219/8351bc93/attachment.html>


More information about the kde-games-devel mailing list