Review Request 128599: Improve kpackage_install_package

Aleix Pol Gonzalez aleixpol at kde.org
Thu Aug 4 12:33:23 UTC 2016


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

(Updated Aug. 4, 2016, 12:33 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Plasma.


Changes
-------

Submitted with commit ad30347a4717a03de82fdd78a85b7641a01c9ef8 by Aleix Pol to branch master.


Repository: kpackage


Description
-------

* Make sure we don't just append to the regenerateindex.sh file but refresh it on every configure execution.
* Don't call generate every time when we configure. It's bound to break on systems that install in /usr.
* Use the KF5::kpackagetool5 imported target rather than trying to figure out where it went.


Diffs
-----

  CMakeLists.txt 312be78 
  KF5PackageMacros.cmake 81379c4 
  src/kpackagetool/CMakeLists.txt f0c40ee 

Diff: https://git.reviewboard.kde.org/r/128599/diff/


Testing
-------

Things now install properly.
The regenerateindex.sh files now make some sense.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160804/708bfd8e/attachment.html>


More information about the Plasma-devel mailing list