Re: [PATCH] Added exists to the minion module command to check if a command exists on the system.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 1/13/08, Michael DeHaan <mdehaan@xxxxxxxxxx> wrote:
> Steve Milner wrote:
> >  func/minion/modules/command.py |   15 ++++++++++++++-
> >  1 files changed, 14 insertions(+), 1 deletions(-)
>
> Applied.
>
> The run() method does not invoke the shell, so still requires full paths
> to commands, even though run() doesn't.
>
> This is ok though as which will still do the right thing.     Maybe
> os.path.exists()?
> Probably does not matter.
>
> Thanks!
>
>

If anyone requests it I'll change it over ... os.access(path, os.X_OK)
would make sense to me.

-- 
"An organisation that treats its programmers as morons will soon have
programmers that are willing and able to act like morons only."
    -Bjarne Stroustrup

_______________________________________________
Func-list mailing list
Func-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/func-list

[Index of Archives]     [Fedora Users]     [Linux Networking]     [Fedora Legacy List]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux