Add `shell-command` keybinding from emacs

[VisenDev]

I find myself missing the M-! binding from emacs to run a shell command. A minimal implementation of this in lem would look like this.

(lem:define-command shell-command (cmd) ((:string "Command to run: "))
    (lem-process:run-process
        (uiop:split-string cmd)
           :output-callback (lambda (str) 
                              (lem:message str))))                                        

(lem:define-key lem:*global-keymap* "M-!" 'shell-command) 

I've tried using run-shell, but that doesn't seem to work

0: *shell pid:81579* 
warning: No TTY for interactive shell (tcgetpgrp failed)
setpgid: Inappropriate ioctl for device
 

I also setting lem-shell-mode:*default-shell-command*, to no avail.

[VisenDev]

I have since discovered (purely by accident) that lem has implemented this under the name pipe-command and bound it to C-x @

It might be useful to put together a list of notable changes from emacs in order to help avoid problems like this

[vindarel]

I didn't know it either.

I'm in favor of renaming it to something with "shell" in the name. And, maybe not the simplest "shell-command", but what about adding an information on what it's doing, like "shell-command-to-popup", so we could have "shell-command-to-buffer", "shell-command-to-file" and whatnot.

bonus point if we don't remove "pipe-command" but explicitely mark it "deprecated".

doc section added: https://lem-project.github.io/usage/usage/#shell-commands

[VisenDev]

I'm in favor of renaming it to something with "shell" in the name. And, maybe not the simplest "shell-command", but what about adding an information on what it's doing, like "shell-command-to-popup", so we could have "shell-command-to-buffer", "shell-command-to-file" and whatnot.

Yeah, calling it "pipe-command" made me think it was going to pipe the buffer contents thru the spawned process or something like that

Should the C-x @ binding be changed as well?