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

update readme for PR #130 #141

Closed
wants to merge 3 commits into from
Closed

Conversation

bdeshi
Copy link
Member

@bdeshi bdeshi commented Apr 26, 2020

updates readme with install information for PR #130

== tasks ==

  • some_command | sudo some_command - this form is not very user-friendly.
    the sudo prompt might pop up in the middle of output.
    otoh, separating the pipes will increase the number of commands, which is also not desirable.
    possible solutions:
    • sudo command | sudo command - run each subcmd with sudo.
    • sudo sh -c 'command | command' - run in sudoed subshell.
    • sudo command <<EOF\n command\nEOF - too complex.
  • update dependency list

@bdeshi bdeshi marked this pull request as ready for review April 30, 2020 17:03
@bdeshi bdeshi added the Doc Documentation label May 1, 2020
@mominul
Copy link
Member

mominul commented Oct 2, 2020

I have merged these changes in #173

@mominul mominul closed this Oct 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Doc Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants