Review Request 114737: KInfocenter/OpenGL: reimplement the ReadPipe() function with QProcess

Commit Hook null at kde.org
Fri Mar 7 23:01:52 GMT 2014


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


This review has been submitted with commit 76d9995933f23acdef1dbf34a62649f149561e73 by Wolfgang Bauer to branch master.

- Commit Hook


On Jan. 26, 2014, 9:43 a.m., Wolfgang Bauer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114737/
> -----------------------------------------------------------
> 
> (Updated Jan. 26, 2014, 9:43 a.m.)
> 
> 
> Review request for kde-workspace and David Stephen Hubner.
> 
> 
> Repository: kde-workspace
> 
> 
> Description
> -------
> 
> This patch reimplements the ReadPipe() function by using QProcess instead of popen().
> This should make it more portable.
> 
> As a positive side-effect, this also removes those "sh: lspci: command not found." messages when run in Konsole and lspci is not in the user's path.
> 
> This was suggested on the kde-core-devel mailinglist in November:
> http://lists.kde.org/?l=kde-core-devel&m=138407113011843&w=2
> http://lists.kde.org/?l=kde-core-devel&m=138409755820003&w=2
> 
> 
> Diffs
> -----
> 
>   kinfocenter/Modules/opengl/opengl.cpp 8901957 
> 
> Diff: https://git.reviewboard.kde.org/r/114737/diff/
> 
> 
> Testing
> -------
> 
> Ran KInfocenter with lspci in /usr/bin/ (i.e. in the user's path) and /sbin/ (not in the user's path). The OpenGL module showed the 3D accelerator info correctly in both cases.
> With lspci removed completely it showed "unknown" as expected.
> 
> 
> Thanks,
> 
> Wolfgang Bauer
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140307/72402145/attachment.htm>


More information about the kde-core-devel mailing list