[Kde-imaging] [Bug 286018] Pano Tool and ExpoBlend tool do not work on OS X [patch]

[brad]

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





--- Comment #6 from brad <bkn ithryn net>  2011-11-30 13:05:41 ---
On Nov 30, 2011, at 7:31 AM, Ananta Palani wrote:

> https://bugs.kde.org/show_bug.cgi?id=286018
> 
> 
> 
> 
> 
> --- Comment #5 from Ananta Palani <anantapalani gmail com>  2011-11-30 12:31:18 ---
> (In reply to comment #4)

> can't you just set the PATH environment variable to contain the path
> to the Hugin executables?
> 
i had troubles with setting the PATH in the QProcess. I don't recall exactly
what the problems were; this is way i decided to edit the makefile. When i have
a revisited patch that applies cleanly to the new codebase and address your
other complaints, i'll take another look at this problem of PATH vs. edit
Makefile.


> Likewise, why can't you do the same thing once you have found one of the
> executables, instead of asking where all the others are, store the path from
> the first and use that to find all other executables. 
> 
Is this the same point (#3) you made in your previous post?

> Before you suggest a new patch, please update to latest source since Benjamin
> also made some code changes.
> 
i am working on a new submission of the patch to apply cleaner to the recently
changed files and to address the problems you're seeing. i guess i wasn't clear
about that in my last post. 

Note to Benjamin: this patch needs some work, please wait until i have
resubmitted that patch to address these problems before you spend time on
reviewing it.

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