Re: TLC for the cmakemanager - feedback required

On Mittwoch, 12. Februar 2020 23:11:01 CET Milian Wolff wrote:
> 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.

Anyone? Otherwise my plan is to merge this tomorrow morning.

Cheers

-- 
Milian Wolff
[email protected]
http://milianw.de

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEezawi1aUvUGg3A1+8zYW/HGdOX8FAl5MQvcACgkQ8zYW/HGd
OX+szg//VBjUdNGYRM6rD1mylvVD9fQmRr2J6RG9bzEN5kgrZno49nHT62hUKYtM
c6RHzFcXmvu7oOSw9dzvZDd+Bg5xer/s8kXFyR+wnNh0HGrb7AyPCLUO8qGVNT8M
dqeUcAqS6Fn1oefR+9CIY7/ef4qHaHatwApvL3ZNUmvAqltrSXTyiG84E37r6oSC
PxS4H2fZdBOdXqVSDDliT4IZJDSx6U13kpqyiUzmPVh+YyOKawaa4SDf6CEogTHs
D9WNoI3v8hYLJz0SBtC8yEpBy+oy2yXcmpHX3dLQnRazcTUgfKhY0Tf0JJ1XvRLs
O2emG6UjiFj+2v9SdJLXokhXcKrSlKa+3jWbE3zLgBrBB7EZ+VW1uXagd/WqLT6u
uHIn8NxWsLowP2E1djqW5geCxWAbYOHC12bpfrY19AWPu1WNHknovu0bCy5ObiW+
hexIdnwruaQm0HWznfzDlDlWaaS1wYCQDD+/d8DvArGzR+X6qvyPIgbcGmk/+oHo
xCBz00TKgXQXBw65DEldm2birrlYE+vR1Z75hxJp/DfPsUAEIfpfFRZtoS6EJR2U
N0pYHYmZEhI89ueFVDrJXIWO0E932NPvH/d6Njl98sgKbMsKuJJNGG6w71uqMsQv
75NK5wHkAwlhps/s12YGFe+PIRdAOtqE6yIbM7WnMCSMNK4++Go=
=qoF9
-----END PGP SIGNATURE-----