Review Request: Custom buildsystem: avoid error if first command arg contains equals sign

[Andreas Pakulat]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106236/#review18134
-----------------------------------------------------------

Ship it!


Once you checked that it compiles please commit.

- Andreas Pakulat


On Aug. 27, 2012, 7:13 p.m., Nicolas Alvarez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106236/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2012, 7:13 p.m.)
> 
> 
> Review request for KDevelop and Andreas Pakulat.
> 
> 
> Description
> -------
> 
> Avoid error if the first argument to the build command contains an equals sign.
> 
> splitArgs with AbortOnMeta will return a FoundMeta error if the first word contains an equals sign, because it thinks it's a shell variable assignment, as in "LANG=C make" or similar. We were passing the list of arguments of the build command to KShell::splitArgs, without including the command itself. Thus, if the first argument (*after* the command name!) had an equals sign, like in command="make" arguments="V=1", KDevelop was saying "the given arguments would need a real shell", which isn't true.
> 
> This patch fixes the problem by passing the command name to splitArgs too (and then removing the command name from the returned split list).
> 
> 
> Diffs
> -----
> 
>   projectmanagers/custom-buildsystem/custombuildjob.cpp 83af2561c149fb8c807c8c4d58371f0b727e18f5 
> 
> Diff: http://git.reviewboard.kde.org/r/106236/diff/
> 
> 
> Testing
> -------
> 
> I didn't even test if it compiles. I just wanted to post the patch and run to class :)
> 
> 
> Thanks,
> 
> Nicolas Alvarez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120827/61d269a8/attachment.html>