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

Let purebred-mailcap build the ProcessConfig #8

Closed
romanofski opened this issue Oct 3, 2022 · 3 comments
Closed

Let purebred-mailcap build the ProcessConfig #8

romanofski opened this issue Oct 3, 2022 · 3 comments

Comments

@romanofski
Copy link
Member

The mailcap provides templates on how shell commands are executed. @frasertweedale suggested one function, e..g:

prepareProcess :: MailcapEntry -> Message s B.Bytestring -> ProcessConfig i o e

provided by the library returning a process config. That config can then be executed by purebred.

@romanofski
Copy link
Member Author

See #9 as the alternate approach

@frasertweedale
Copy link
Member

I think I want to reject this approach already. Making / cleaning up temporary files involves a lot of policy decisions that i think it's better to leave to the application.

This does not entirely preclude building a ProcessConfig, though. I just means that purebred-mailcap should not be responsible for creating temporary files, etc.

@frasertweedale
Copy link
Member

Closing in favour of #9.

@frasertweedale frasertweedale closed this as not planned Won't fix, can't repro, duplicate, stale Oct 5, 2022
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

No branches or pull requests

2 participants