[Kde-imaging] [Bug 283650] panorama tool can't find cpfind version on OS X

[brad]

https://bugs.kde.org/show_bug.cgi?id=283650





--- Comment #8 from brad <bkn ithryn net>  2011-10-13 14:57:13 ---
On Wednesday, October 12, 2011 05:47:02 PM benjamin.girault at gmail.com wrote:
> https://bugs.kde.org/show_bug.cgi?id=283650
> 
> 
> 
> 
> 
> --- Comment #7 from  <benjamin girault gmail com>  2011-10-12 17:47:01 ---
> Using RegExp is an overkill here because we are looking for a specific
> substring. 
>
in general regular expressions tend to be less maintainable too (i.e. very 
ugly) -- so for the longevity of the code a simpler solution is probably the 
best solution.

> Using grep is too, because it means starting an additional
> program, and allocating a stack for it, etc...
>
Agreed. At the time i suggested this, i thought you would only need to shell-
out once, but after trying a few things and reading about QProcess, you would 
need to shell-out twice. So this is definitely not the best solution either. 

> That being said, I am thinking about this issue, and trying to find a nice
> way to handle this.
>
Thank you. It'll be much appreciated.

Cheers, 
 -- brad

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.