Review Request 128209: restore system environment for subprocesses

[Kevin Funk]

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


Ship it!




Ship It!

- Kevin Funk


On June 16, 2016, 3:10 p.m., Sven Brauch wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128209/
> -----------------------------------------------------------
> 
> (Updated June 16, 2016, 3:10 p.m.)
> 
> 
> Review request for KDevelop and Kevin Funk.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> When running in some kind of container (esp. AppImage), we need to restore the original system environment for all external processes. This patch does that, using the information about the original environment which was set by the startup shell script.
> 
> 
> Diffs
> -----
> 
>   outputview/outputexecutejob.cpp 2f184dc 
>   outputview/outputjob.h 5c9524c 
>   outputview/outputjob.cpp 6a6d1d6 
>   plugins/executescript/scriptappjob.cpp 20d8acd 
>   plugins/externalscript/externalscriptjob.cpp 5b48c2b 
>   util/commandexecutor.cpp 578a6bf 
>   util/environmentgrouplist.h 9e26a1b 
>   util/environmentgrouplist.cpp c762296 
>   vcs/dvcs/dvcsjob.cpp a45d87c 
> 
> Diff: https://git.reviewboard.kde.org/r/128209/diff/
> 
> 
> Testing
> -------
> 
> Can run cmake from KDevelop AppImage now without library errors.
> 
> 
> Thanks,
> 
> Sven Brauch
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20160616/04e3fdf3/attachment.html>