Matthieu, is it possible to pass command string like "tla inventory" to
process handlers?

Currently they takes three arguments like:

    (defun tla--default-error-function (output error status)
      "Default function called when a tla process ends with a non-zero
    status."
      (tla--show-error-buffer error)
      (error "tla failed: %s" status))

My proposal is:

    (defun tla--default-error-function (output error status cmd)
      "Default function called when a tla process ends with a non-zero
    status."
      (tla--show-error-buffer error)
      (error "\"%s\" failed: %s" cmd status))

Only showing error code is too few information.

Furtherer more, passing default-directory to the handlers also helpful to debug;

    (defun tla--default-error-function (output error status cmd pwd)
      "Default function called when a tla process ends with a non-zero
    status."
      (tla--show-error-buffer error)
      (error "\"%s\" failed: %s at %s" cmd status pwd))

Regards,
Masatake YAMATO

Reply via email to