Skip to content

Add error buffer for LP subprocesses#13

Open
meliache wants to merge 1 commit intohaji-ali:masterfrom
meliache:feature/add-error-buffer-for-lp-processes
Open

Add error buffer for LP subprocesses#13
meliache wants to merge 1 commit intohaji-ali:masterfrom
meliache:feature/add-error-buffer-for-lp-processes

Conversation

@meliache
Copy link

Otherwise, any error output is hidden from the user, which makes debugging difficult.

I didn't use the a2ps print menu yet, but its process call is

(make-process
 :name "a2ps-print"
 :buffer buffer
 :stderr buffer
 :connection-type 'pipe
 :command cmd)

so it also shows you the stderr in a buffer. For lp the normal stdout might not
be so useful, so it's fine not having and stdout buffer there, but I would prefer having the stderr in a separate buffer.

What I haven't implemented yet but would be nice is to also give a warning in case of an error with a hint to check out the error buffer for more information. Like what magit does if a commit or push fails.

Otherwise, any error output is hidden from the user, which makes debugging
difficult.

I didn't use the a2ps print menu yet, but its process call is

``` emacs-lisp

(make-process
 :name "a2ps-print"
 :buffer buffer
 :stderr buffer
 :connection-type 'pipe
 :command cmd)
```

so it also shows you the stderr in a buffer. For lp the normal stdout might not
be so useful, so it's fine not having and stdout buffer there.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant