Review Request 121758: Kalzium : Porting to KF5 CMake built

Jeremy Whiting jpwhiting at kde.org
Fri Jan 2 19:23:04 UTC 2015 


> On Jan. 2, 2015, 12:12 p.m., Etienne Rebetez wrote:
> > Hi. Thanks a lot for your effort in porting kalzium to the new framework.
> > 
> > The pictures in the data/icons folder are already part of kalzium and should not be part of the port.

Etienne, those files are renames, the git diff Anu did didn't catch that is all. in kf5/ecm ecm-add-icons is used and needs icons to have a standard naming scheme.


- Jeremy


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


On Dec. 30, 2014, 11:13 a.m., Anu Mittal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121758/
> -----------------------------------------------------------
> 
> (Updated Dec. 30, 2014, 11:13 a.m.)
> 
> 
> Review request for KDE Edu and Aleix Pol Gonzalez.
> 
> 
> Repository: kalzium
> 
> 
> Description
> -------
> 
> Kalzium CMakeLists.txt files updated to KF5
> 
> 
> Diffs
> -----
> 
>   compoundviewer/CMakeLists.txt 0971248 
>   compoundviewer/widgets/CMakeLists.txt 32f200a 
>   data/icons/CMakeLists.txt 779ced5 
>   data/icons/hi16-actions-calculate.png PRE-CREATION 
>   data/icons/hi16-actions-chemical.png PRE-CREATION 
>   data/icons/hi16-actions-elempic.png PRE-CREATION 
>   data/icons/hi16-actions-energies.png PRE-CREATION 
>   data/icons/hi16-actions-eqchem.png PRE-CREATION 
>   data/icons/hi16-actions-glossary.png PRE-CREATION 
>   data/icons/hi16-actions-isotopemap.png PRE-CREATION 
>   data/icons/hi16-actions-kalzium_molviewer.png PRE-CREATION 
>   data/icons/hi16-actions-kalzium_rs.png PRE-CREATION 
>   data/icons/hi16-actions-kalzium_tables.png PRE-CREATION 
>   data/icons/hi16-actions-legend.png PRE-CREATION 
>   data/icons/hi16-actions-misc.png PRE-CREATION 
>   data/icons/hi16-actions-numbers.png PRE-CREATION 
>   data/icons/hi16-actions-orbits.png PRE-CREATION 
>   data/icons/hi16-actions-overview.png PRE-CREATION 
>   data/icons/hi16-actions-plot.png PRE-CREATION 
>   data/icons/hi16-actions-sidebar.png PRE-CREATION 
>   data/icons/hi16-actions-spectrum.png PRE-CREATION 
>   data/icons/hi16-actions-statematter.png PRE-CREATION 
>   data/icons/hi16-actions-timeline.png PRE-CREATION 
>   data/icons/hi16-actions-tooltip.png PRE-CREATION 
>   data/icons/hi22-actions-calculate.png PRE-CREATION 
>   data/icons/hi22-actions-chemical.png PRE-CREATION 
>   data/icons/hi22-actions-elempic.png PRE-CREATION 
>   data/icons/hi22-actions-energies.png PRE-CREATION 
>   data/icons/hi22-actions-eqchem.png PRE-CREATION 
>   data/icons/hi22-actions-glossary.png PRE-CREATION 
>   data/icons/hi22-actions-isotopemap.png PRE-CREATION 
>   data/icons/hi22-actions-kalzium_molviewer.png PRE-CREATION 
>   data/icons/hi22-actions-kalzium_rs.png PRE-CREATION 
>   data/icons/hi22-actions-kalzium_tables.png PRE-CREATION 
>   data/icons/hi22-actions-legend.png PRE-CREATION 
>   data/icons/hi22-actions-numbers.png PRE-CREATION 
>   data/icons/hi22-actions-orbits.png PRE-CREATION 
>   data/icons/hi22-actions-overview.png PRE-CREATION 
>   CMakeLists.txt f4ab27b 
>   KalziumConfigureChecks.cmake e3197ac 
>   data/icons/hi22-actions-plot.png PRE-CREATION 
>   data/icons/hi22-actions-sidebar.png PRE-CREATION 
>   data/icons/hi22-actions-spectrum.png PRE-CREATION 
>   data/icons/hi22-actions-statematter.png PRE-CREATION 
>   data/icons/hi22-actions-timeline.png PRE-CREATION 
>   data/icons/hi22-actions-tooltip.png PRE-CREATION 
>   data/icons/hi32-actions-calculate.png PRE-CREATION 
>   data/icons/hi32-actions-chemical.png PRE-CREATION 
>   data/icons/hi32-actions-elempic.png PRE-CREATION 
>   data/icons/hi32-actions-energies.png PRE-CREATION 
>   data/icons/hi32-actions-eqchem.png PRE-CREATION 
>   data/icons/hi32-actions-glossary.png PRE-CREATION 
>   data/icons/hi32-actions-isotopemap.png PRE-CREATION 
>   data/icons/hi32-actions-kalzium_molviewer.png PRE-CREATION 
>   data/icons/hi32-actions-kalzium_rs.png PRE-CREATION 
>   data/icons/hi32-actions-kalzium_tables.png PRE-CREATION 
>   data/icons/hi32-actions-legend.png PRE-CREATION 
>   data/icons/hi32-actions-misc.png PRE-CREATION 
>   data/icons/hi32-actions-numbers.png PRE-CREATION 
>   data/icons/hi32-actions-orbits.png PRE-CREATION 
>   data/icons/hi32-actions-overview.png PRE-CREATION 
>   data/icons/hi32-actions-plot.png PRE-CREATION 
>   data/icons/hi32-actions-sidebar.png PRE-CREATION 
>   data/icons/hi32-actions-spectrum.png PRE-CREATION 
>   data/icons/hi32-actions-statematter.png PRE-CREATION 
>   data/icons/hi32-actions-timeline.png PRE-CREATION 
>   data/icons/hi32-actions-tooltip.png PRE-CREATION 
>   data/icons/hi48-actions-calculate.png PRE-CREATION 
>   data/icons/hi48-actions-chemical.png PRE-CREATION 
>   data/icons/hi48-actions-energies.png PRE-CREATION 
>   data/icons/hi48-actions-eqchem.png PRE-CREATION 
>   data/icons/hi48-actions-glossary.png PRE-CREATION 
>   data/icons/hi48-actions-isotopemap.png PRE-CREATION 
>   data/icons/hi48-actions-kalzium_molviewer.png PRE-CREATION 
>   data/icons/hi48-actions-kalzium_rs.png PRE-CREATION 
>   data/icons/hi48-actions-kalzium_tables.png PRE-CREATION 
>   data/icons/hi48-actions-legend.png PRE-CREATION 
>   data/icons/hi48-actions-misc.png PRE-CREATION 
>   data/icons/hi48-actions-numbers.png PRE-CREATION 
>   data/icons/hi48-actions-orbits.png PRE-CREATION 
>   data/icons/hi48-actions-overview.png PRE-CREATION 
>   data/icons/hi48-actions-plot.png PRE-CREATION 
>   data/icons/hi48-actions-sidebar.png PRE-CREATION 
>   data/icons/hi48-actions-spectrum.png PRE-CREATION 
>   data/icons/hi48-actions-statematter.png PRE-CREATION 
>   data/icons/hi48-actions-timeline.png PRE-CREATION 
>   data/icons/hi48-actions-tooltip.png PRE-CREATION 
>   data/icons/hi64-actions-kalzium_molviewer.png PRE-CREATION 
>   data/icons/hi64-actions-kalzium_rs.png PRE-CREATION 
>   data/icons/hi64-actions-kalzium_tables.png PRE-CREATION 
>   data/icons/hi64-actions-numbers.png PRE-CREATION 
>   data/icons/hisc-actions-calculate.svgz PRE-CREATION 
>   data/icons/hisc-actions-chemical.svgz PRE-CREATION 
>   data/icons/hisc-actions-elempic.svgz PRE-CREATION 
>   data/icons/hisc-actions-energies.svgz PRE-CREATION 
>   data/icons/hisc-actions-eqchem.svgz PRE-CREATION 
>   data/icons/hisc-actions-glossary.svgz PRE-CREATION 
>   data/icons/hisc-actions-isotopemap.svgz PRE-CREATION 
>   data/icons/hisc-actions-kalzium_molviewer.svgz PRE-CREATION 
>   data/icons/hisc-actions-kalzium_rs.svgz PRE-CREATION 
>   data/icons/hisc-actions-kalzium_tables.svgz PRE-CREATION 
>   data/icons/hisc-actions-misc.svgz PRE-CREATION 
>   data/icons/hisc-actions-numbers.svgz PRE-CREATION 
>   data/icons/hisc-actions-orbits.svgz PRE-CREATION 
>   data/icons/hisc-actions-overview.svgz PRE-CREATION 
>   data/icons/hisc-actions-plot.svgz PRE-CREATION 
>   data/icons/hisc-actions-sidebar.svgz PRE-CREATION 
>   data/icons/hisc-actions-spectrum.svgz PRE-CREATION 
>   data/icons/hisc-actions-statematter.svgz PRE-CREATION 
>   data/icons/hisc-actions-timeline.svgz PRE-CREATION 
>   data/icons/hisc-actions-tooltip.svgz PRE-CREATION 
>   doc/CMakeLists.txt f04a92b 
>   libscience/CMakeLists.txt ac66132 
>   libscience/tests/CMakeLists.txt 21c4f85 
>   plasmoid/applet/bodr/CMakeLists.txt e3ed33e 
>   plasmoid/applet/concentrationPlasmoid/CMakeLists.txt 32ea556 
>   plasmoid/applet/didyouknow/CMakeLists.txt f6ca9e3 
>   plasmoid/applet/gasPlasmoid/CMakeLists.txt 7d199bb 
>   plasmoid/applet/nuclearPlasmoid/CMakeLists.txt 4acfab6 
>   plasmoid/applet/psePlasmoid/CMakeLists.txt 58da62f 
>   plasmoid/engine/CMakeLists.txt 534a56d 
>   src/CMakeLists.txt 3cefc58 
> 
> Diff: https://git.reviewboard.kde.org/r/121758/diff/
> 
> 
> Testing
> -------
> 
> cmake.. gives
> 
> Configuration done.
> Generation done.
> 
> 
> Thanks,
> 
> Anu Mittal
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20150102/cd5e4af5/attachment-0001.html>

More information about the kde-edu mailing list