Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typo in reading body from arg #260

Closed
wants to merge 1 commit into from

Conversation

rakoo
Copy link
Contributor

@rakoo rakoo commented Jun 17, 2024

When given a body argument like this:

mcom foo@bar -body "this is my body"

the body isn't taken into account. This is because we try to cat from a file called "this is my body", instead of doing a process substitution for a file that fake-contains "this is my body". I believe it is a typo.

@leahneukirchen
Copy link
Owner

No, see the documentation:

The flag -body file can be used to
     prepopulate the draft for mcom and mrep with the contents of file.

You can use a here-string if you want to pass a string as a body (if your shell supports it).

@rakoo
Copy link
Contributor Author

rakoo commented Jun 18, 2024

Oh ok, so the argument to mcom should be a file, not a string

The problem I have is probably in mmailto then, which passes the content of the body arg as a string. I'll do the necessary changes over 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.

2 participants