Re: TLC for the cmakemanager - feedback required

Milian Wolff <[email protected]>
Newsgroups gmane.comp.kde.devel.kdevelop
Message-ID <2704926.e9J7NaK4W3@agathebauer>
On Sonntag, 9. Februar 2020 22:41:25 CET Milian Wolff wrote:
> Hey all,
> 
> I finally found some time to give the cmake manager plugin some tender
> loving care. I removed a bunch of dead code, fixed some unit tests and
> added support for the cmake-file-api [1].
> 
> You can find my work over at [2]. Be warned: I may potentially force push to
> that branch. That said, I would really welcome if others could have a look
> and give me some feedback.
> 
> My preliminary testing shows that it seems to work. The one big feature
> that's missing is watching the cmake files for changes and then
> automatically triggering a reconfiguration to update the code model data.
> I'll give that a shot in the next days and also test it on some bigger
> applications at work, before creating a merge request.
> 
> Cheers
> 
> [1]: https://bugs.kde.org/show_bug.cgi?id=417202
> [2]: https://invent.kde.org/mwolff/kdevelop/commits/cmake-file-api

The important features, namely code-model support and even project reloading 
when a cmake file is edited, is now implemented. I've created a merge request:

https://invent.kde.org/kde/kdevelop/merge_requests/106

There's even a nice new feature, namely target grouping by the FOLDER target 
property. This way, all the pesty CTest targets get automatically grouped. And 
it's all generic and users can create custom groups, if desired :)

Please test and give feedback, esp. on non-trivial CMake projects! If I don't 
hear anything back, I'll merge this sometime next week.

Cheers

-- 
Milian Wolff
[email protected]
http://milianw.de
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEezawi1aUvUGg3A1+8zYW/HGdOX8FAl5Ed/UACgkQ8zYW/HGd
OX8trA/9Gtvd+W1SYMGyDk+XqI8fbZIJKOrE5MeQH4oQbpiMS7D54R/ZCUo+MZJZ
iu2zSpXAtn3uK3/mkyZ5py+fcbozQHsP+Ub05MwQJZgKoPBAwUVLAjInRR0/qoji
qyHssgbXFcy73I/usNYTU2u+ZCtBwjZtYqgLGjOJxoOhadDXigkreBSsL9kMWJgs
PaGu2NEEGwQ0NIHZUHG2WKD1FJ6DGdHeKBeov8jj7foy/Vw2YSN4TSv35OeNmFmQ
2ZHBaigQQ50TXqP3qP3+lVIjch/GGeQIXJdamYi16ZIxTT7TAEX2UN5lz3wiAVSw
QmOh1oyawEuZq6dziMnbd3hj4b3tO+Hu31zPrtLsxXueQXBrRaY+VS2bFP4z1ZIM
uex/gF1Nh2FiZbeKS0lmyw49qqNLeO+LmsALI0FQbaEypIRqWZ6B5XdZBkf/xMDN
FHndTBCv5eScFeXGgKx7JjG8cAGicLLpTDRRFXM/RZDX+dsgUcOMk7wu0ZN/T6sw
S77KbmchwYZ5VtTzl7agDyTZ6D0SZJUZD6mC/r7ZCRckXPkvidqPgojTBQhIQQOB
pTsS9HjihW4L0093fzLmxHWJap11YA5EteOZ4q72PPOtU1hNQ8p0+kXWBD5mCpxz
kKg1Nwfjc+WAsUioGs1RT1nOYQlU/y35klAEmCCwkiIB25H3Y7U=
=9W2q
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.