Review Request 121525: Fixed icon installation warnings
Inge Wallin
inge at lysator.liu.se
Sun Dec 14 23:20:27 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121525/#review72014
-----------------------------------------------------------
Ship it!
Looks good. Please just fix the minor issue I point out below and then you can push.
God job!
step/CMakeLists.txt
<https://git.reviewboard.kde.org/r/121525/#comment50206>
These are hardly optional and should just use add_subdirectory()
- Inge Wallin
On Dec. 14, 2014, 11:15 p.m., Rahul Chowdhury wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121525/
> -----------------------------------------------------------
>
> (Updated Dec. 14, 2014, 11:15 p.m.)
>
>
> Review request for KDE Edu and Inge Wallin.
>
>
> Repository: step
>
>
> Description
> -------
>
> Modified the CMakeLists files and renamed some of the png files to fix the icon installation in proper places.
>
>
> Diffs
> -----
>
> step/icons/actions/hi22-action-step_object_Spring.png 2eec6ce70d89187246390bd9ab756a88500fd3a6
> step/icons/actions/hi22-action-step_object_Stick.png 4ba512927c1e9c753165b0360e2bd03559dda406
> step/icons/actions/hi22-action-step_object_Tracer.png fb5f86bb2c3badea9afbebb11ff8d1ddd45dc30b
> step/icons/actions/hi22-action-step_object_WeightForce.png d1a1ce882b1be7e2c83666459d23640a702fb356
> step/icons/actions/hi22-actions-pointer.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Anchor.png PRE-CREATION
> step/icons/actions/hi22-action-step_object_Graph.png e23c6497cdac456fdae3bd0eddd2df609ec7da97
> step/icons/actions/hi22-action-step_object_GravitationForce.png 9fefe34f2dffef71a916e4cde9270e417a58bdf3
> step/icons/actions/hi22-action-step_object_LinearMotor.png cd76d7170987e0684189816ad6be66d9f6507dd9
> step/icons/actions/hi22-action-step_object_Meter.png a8a09d891897287df30e4b06507412f93f31c643
> step/icons/actions/hi22-action-step_object_Note.png 9a8d87c9d5f5950c924e6818d2db2a36d3214700
> step/icons/actions/hi22-action-step_object_Particle.png 1744d794052e45523b7186c74dae5dbf32952572
> step/icons/actions/hi22-action-step_object_Pin.png 0d7dc40b3d4fdd0e0e8171e4e0edaf25b2da3857
> step/icons/actions/hi22-action-step_object_Polygon.png 8b428db511568602a4759c55723f7bce4e9358bd
> step/icons/actions/hi22-action-step_object_Rope.png dfb05676c1231d52960320242cc2613ce4314e1f
> step/icons/actions/hi22-action-step_object_SoftBody.png f60aaf89dd4d7357ba6f512144af60d61941b590
> step/icons/actions/hi22-action-step_object_Anchor.png 4fa715743ce9b75610f32e278d87755c7f139f52
> step/icons/actions/hi22-action-step_object_Box.png 04c4ec58c3f8581c6e8393cbf80cde0a6753774d
> step/icons/actions/hi22-action-step_object_ChargedParticle.png 1744d794052e45523b7186c74dae5dbf32952572
> step/icons/actions/hi22-action-step_object_CircularMotor.png 01ca0fa18fa040d168b99b14c1da610f51f639c3
> step/icons/actions/hi22-action-step_object_Controller.png 8a2940da80f8a08ff2c18da52a733ae7e0ab6482
> step/icons/actions/hi22-action-step_object_CoulombForce.png 7ddd4914a880a1387741c0a6f104425ceeb1162a
> step/icons/actions/hi22-action-step_object_Disk.png f2e432eef03724cc9a4b96d28afbe5d08e0418e4
> step/icons/actions/hi22-action-step_object_Gas.png 684cd154ae83285a9af787562e7e2afc25416057
> step/icons/actions/hi22-action-step_object_GasParticle.png 684cd154ae83285a9af787562e7e2afc25416057
> CMakeLists.txt 7a39ee9
> step/CMakeLists.txt 554c390
> step/icons/CMakeLists.txt f4c5595
> step/icons/actions/CMakeLists.txt 439bb24
> step/icons/actions/hi22-action-pointer.png b658c05aba529343ea3cbd5c5ee234f755f7f737
> step/icons/actions/hi22-actions-step_object_Box.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_ChargedParticle.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_CircularMotor.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Controller.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_CoulombForce.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Disk.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Gas.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_GasParticle.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Graph.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_GravitationForce.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_LinearMotor.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Meter.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Note.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Particle.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Pin.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Polygon.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Rope.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_SoftBody.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Spring.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Stick.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_Tracer.png PRE-CREATION
> step/icons/actions/hi22-actions-step_object_WeightForce.png PRE-CREATION
>
> Diff: https://git.reviewboard.kde.org/r/121525/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Rahul Chowdhury
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20141214/9891157c/attachment-0001.html>
More information about the kde-edu
mailing list