Summary: | Support of $VARIABLE in environment settings | ||
---|---|---|---|
Product: | [Applications] kdevelop | Reporter: | Denis Kuplyakov <dener.kup> |
Component: | Build tools: Make | Assignee: | kdevelop-bugs-null |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | CC: | kfunk |
Priority: | NOR | Keywords: | junior-jobs |
Version: | 4.7.1 | ||
Target Milestone: | --- | ||
Platform: | openSUSE | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/kdevplatform/81aa2d3f4cc39a7d0cb5e5fbae02ecedafe2e22c | Version Fixed In: | 5.0.0 |
Sentry Crash Report: |
Description
Denis Kuplyakov
2015-07-13 16:10:31 UTC
Definitely useful, I agree. This should be relatively trivial to implement, wanna give it a try, Denis? (In reply to Kevin Funk from comment #1) > Definitely useful, I agree. > > This should be relatively trivial to implement, wanna give it a try, Denis? Sometimes it is hard to dive in the code you have never seen before. Maybe if I have enough time I will try it, but I think if it is trivial for somebody experienced with KDevelop, he should take this. Git commit 81aa2d3f4cc39a7d0cb5e5fbae02ecedafe2e22c by Kevin Funk. Committed on 10/01/2016 at 23:46. Pushed by kfunk into branch '5.0'. Support of $VARIABLE in environment settings FIXED-IN: 5.0.0 Summary: Added the possibility to use $VARIABLE in environment settings. After some discussion it was concluded, that we won't support defining a variable with other user defined variable. Only supported scenario is defining a variable with a system variable. Test Plan: Tested with the added unit test and in practice. Reviewers: kfunk Subscribers: kdevelop-devel Projects: #kdevelop Differential Revision: https://phabricator.kde.org/D770 M +5 -2 outputview/outputexecutejob.cpp M +20 -0 util/environmentgrouplist.cpp M +3 -0 util/environmentgrouplist.h M +3 -0 util/tests/CMakeLists.txt A +81 -0 util/tests/test_environment.cpp [License: LGPL (v2+)] A +35 -0 util/tests/test_environment.h [License: LGPL (v2+)] http://commits.kde.org/kdevplatform/81aa2d3f4cc39a7d0cb5e5fbae02ecedafe2e22c |