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

mtr should be installed setuid so it doesn't require sudo #35085

Closed
5 tasks done
dwt opened this issue Dec 13, 2018 · 1 comment
Closed
5 tasks done

mtr should be installed setuid so it doesn't require sudo #35085

dwt opened this issue Dec 13, 2018 · 1 comment
Labels
outdated PR was locked due to age

Comments

@dwt
Copy link
Contributor

dwt commented Dec 13, 2018

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

  • are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
  • have a problem with brew install (or upgrade, reinstall) a single, official formula (not cask)? If it's a general brew problem please file this issue at Homebrew/brew: https://github.com/Homebrew/brew/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
  • [x ] ran brew update and can still reproduce the problem?
  • ran brew doctor, fixed all issues and can still reproduce the problem?
  • ran brew gist-logs <formula> (where <formula> is the name of the formula that failed) and included the output link?
  • if brew gist-logs didn't work: ran brew config and brew doctor and included their output with your issue?

To help us debug your issue please explain:

  • What you were trying to do (and why)

Trying to use mtr without sudo

  • What happened (include command output)
% mtr 1.1 
Failure to open IPv4 sockets: Operation not permitted
Failure to open IPv6 sockets: Operation not permitted
mtr: Failure to start mtr-packet: Invalid argument
  • What you expected to happen

mtr should just work

  • Step-by-step reproduction instructions (by running brew install commands)
% brew install mtr
% mtr 1.1 

This was already discussed here but then just closed on moving repos. Basically on all linux distros mtr is installed as setuid root, and I am arguing (as did the people in Homebrew/legacy-homebrew#166) that this should also be the default for home-brew. If you disagree, please at least consider adding this as an option to the formula.

@fxcoudert
Copy link
Member

This has been declined multiple times already: Homebrew's policy is to not install setuid binaries. See #14428 Homebrew/legacy-homebrew#50207 Homebrew/legacy-homebrew#43862 Homebrew/legacy-homebrew#40992

@lock lock bot added the outdated PR was locked due to age label Jan 13, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jan 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

No branches or pull requests

2 participants