mpath: call to execWithRedirect needs searchPath=1 (#579654).
On 04/12/2010 01:12 PM, Peter Jones wrote:
> On 04/12/2010 12:39 PM, David Lehman wrote:
>> On Mon, 2010-04-12 at 10:02 +0200, Ales Kozumplik wrote:
>>> On 04/12/2010 09:57 AM, Ales Kozumplik wrote:
>>>> storage/devices.py | 3 ++-
>>>> 1 files changed, 2 insertions(+), 1 deletions(-)
>>>> diff --git a/storage/devices.py b/storage/devices.py
>>>> index 387c4b4..03d143a 100644
>>>> --- a/storage/devices.py
>>>> +++ b/storage/devices.py
>>>> @@ -3106,7 +3106,8 @@ class MultipathDevice(DMDevice):
>>>> rc = iutil.execWithRedirect("multipath",
>>>> stdout = "/dev/tty5",
>>>> - stderr = "/dev/tty5")
>>>> + stderr = "/dev/tty5",
>>>> + searchPath = 1)
>>>> if rc:
>>>> raise MPathError("multipath activation failed for '%s'" %
>>> This is also missing from f13-branch and should go there if we support
>>> multipath for Fedora.
>> searchPath was removed from iutil.execWithRedirect on master and
>> f13-branch. (commit 7e9ddee96)
> ... there's something odd going on here. We took "searchPath" out because it
> wasn't used, but execWithRedirect() doesn't currently seem to be searching PATH,
> though at the time I think it did. I'm really not sure how this changed,
> but it really needs to be always using PATH.
By which I mean that execWithRedirect() should always search the path, and no
argument should be necessary.
All parts should go together without forcing. You must remember that
the parts you are reassembling were disassembled by you. Therefore,
if you can't get them together again, there must be a reason. By all
means, do not use a hammer.
-- IBM maintenance manual, 1925
Anaconda-devel-list mailing list