D28444: Add ECMCargo module

Ilya Bizyaev noreply at phabricator.kde.org
Mon Apr 6 19:40:16 BST 2020


IlyaBizyaev added a comment.


  I don't have enough CMake knowledge to give code comments, but here are some things I can notice in case they help in review:
  
  1. The function specifically mentions Cargo workspace, which is basically a Rust alternative to `TEMPLATE = subdirs` in qmake. It doesn't seem though that the current implementation has code specific for this project type, and it can probably work just as well for normal executable and library projects. Am I missing something? If it can't, how would developers use this ECM module if they don't need the workspace? Can it be generalized?
  2. In the implementation that I mentioned, the resulting target gets a name that we can use directly, whereas in your usage example you have to manually use an artifact variable to actually be able to link. You also manually specify the shared library extension; will this require a platform-dependent wrapper in end user's CMakeLists to be able to use this on e.g. Windows?
  
  Thank you for your work :)

REPOSITORY
  R240 Extra CMake Modules

REVISION DETAIL
  https://phabricator.kde.org/D28444

To: cblack, #frameworks, #build_system
Cc: apol, IlyaBizyaev, kde-frameworks-devel, kde-buildsystem, LeGast00n, cblack, GB_2, bencreasy, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200406/4d75644d/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list