Follow-up Comment #5, bug #13832 (project mc):

Before committing this patch I'd like to see it extended in either of the
following ways first:

a) Test for mount options of $MC_TMPDIR and concatenate "/bin/sh " only when
necessary.
b) Extend shell_execute(), do_execute() to pass the error generated by
my_system on failure. Execute the command plainly and in case of an error
caused by the mount options execute it once again with a concatenated shell.
c) Extend to pass error as in b). Test on init and set a parameter
(need_shell) so we don't have to do this test on every invokation of
shell_execute(). In case of failure because the mount option changed during
the session reset the parameter need_shell to new value and retry.

I would prefer the last option.


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?func=detailitem&item_id=13832>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/

_______________________________________________
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel

Reply via email to