-
-
Notifications
You must be signed in to change notification settings - Fork 12.4k
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
lsusb 1.1.4 #163135
lsusb 1.1.4 #163135
Conversation
c43d44d
to
690045c
Compare
Thanks for your contribution. The proposed fork has only 4 stars, so that does not really meet our notability criteria. I'm also worried that there are so many downloads for a seemingly abandoned project ...
|
Thank you for your comment, that is indeed the discussion from which I learnt about the proposed fork, as the version currently in homebrew-core has a bug that, while not serious, is annoying. If the proposed PR does not meet notability criteria, perhaps it'd be convenient to consider removing the formula (I'm happy to move it to a personal tap), but I'm not familiar with the procedure to do so. |
What about I'm worried to deprecate the formula as there are 783 downloads / month. We will not advertise private taps so we can't redirect our users to your tap. So that's also not an ideal solution. |
The The version currently in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I think it would be useful for some folks! 👍
I would like some more thoughts from @Homebrew/core maintainers. Looks good IMHO, let's just wait a few more days if someone wants to block this from being merged. |
@LanikSJ's fork is the most active, and people obviously care enough so let's merge it.. They've also been a Homebrew contributor for some time, so that helps me with my decision 😆. |
Thank you @p-linnane I fully intend to support this project as long as humanly possible. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hold please. Usual/documented fork rules should apply: either upstream needs to bless this fork or we need evidence that multiple other distros have moved to this fork.
I've skimmed for evidence of either and can't see them. That's not saying they aren't the case, just let's be explicit in the PR first, thanks.
On this point it's worth noting we're the only known package manager to ship this at all, largely because it's macOS-only. So there are no other distros to check. |
In that case: I'd rather vote (again, as per-docs) that we disabled it rather than move to a fork that doesn't seem to be sufficiently canonical. We could also consider accepting this as-is as |
I'm willing to take whatever steps necessary for this formula to be included. I haven't proposed it myself as in my experience I didn't think it would get included. In the mean time I maintain my own tap https://github.com/LanikSJ/homebrew-tap however I don't pretend to know how write Taps correctly. 🙂 |
HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingHOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
)? If this is a new formula, does it passbrew audit --new <formula>
?The previous version of this formula was pointing to a repo that is now abandoned, I have updated the URL to the most current fork of this project.