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

Review request for Extra Cmake Modules.


Repository: extra-cmake-modules


Description
-------

Multiple ecm_create_qm_loader() with different catalog names would
overwrite each other's generated files, causing the wrong catalog to be
loaded at runtime for some targets.

This puts the catalog name into the generated filename. Since the
catalog name is the only difference between the generated files, this is
sufficient to fix the runtime behaviour.


Diffs
-----

  modules/ECMPoQmTools.cmake 12bcf6b67c57dd456f078820f86c1e8365cb7c06 
  tests/ECMPoQmToolsTest/CMakeLists.txt 
76d801411b87987ac05de8dd99a0f37266ef6909 
  tests/ECMPoQmToolsTest/check.cmake.in 
ab434d2e425ff57f323a196acc60e0e52c0abb10 
  tests/ECMPoQmToolsTest/check_conf.cmake.in 
9ab02e72c4f5aa740c04116e824a54c61fe8bda8 
  tests/ECMPoQmToolsTest/tr_test-po/de/catalog2.po PRE-CREATION 
  tests/ECMPoQmToolsTest/tr_test-po/en/catalog2.po PRE-CREATION 

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


Testing
-------

Extended unit tests, which pass.


Thanks,

Alex Merry

_______________________________________________
Kde-buildsystem mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-buildsystem

Reply via email to