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

rmtrash: remove formula #65438

Merged
merged 1 commit into from
Nov 22, 2020
Merged

rmtrash: remove formula #65438

merged 1 commit into from
Nov 22, 2020

Conversation

waldyrious
Copy link
Contributor

Unclear license, maintainer unresponsive — no response 3 months after attempted contact for clarification.

Quoting discussion from #58225 for context:

@waldyrious:

Any thoughts about what to do with the rmtrash license? I downloaded the source code but the only references to copyright are these two lines in the manpage:

Copyright \(co 2003 Night Productions.
.br
This is free software.  There is NO warranty; not even for merchantability or fitness for a particular purpose.

@SMillerDev:

If it's not a valid license it needs to be removed from homebrew.

@sjackman:

This is free software.

Contact the upstream author to ask what the license is. Their e-mail address is directly above the copyright statement. If we haven't heard from the upstream author in say thirty days, remove the formula.

@waldyrious:

I've just messaged the author, using the email address in the manpage and the one listed in the contact page. I'll keep you posted about any response.

☝️ That was 3 months ago. I've received no response from the author.

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

No clear license, maintainer unresponsive
@BrewTestBot BrewTestBot added marked for removal/rejection PR is probably going to be closed or formula deleted missing license Formula has a missing license which should be added labels Nov 22, 2020
@fxcoudert fxcoudert merged commit a2f05fb into Homebrew:master Nov 22, 2020
@waldyrious waldyrious deleted the patch-1 branch November 22, 2020 23:29
@changkun
Copy link

changkun commented Dec 11, 2020

Hi @waldyrious, I am a user of rmtrash and recently discovered that the formula was removed while setting up my new environment.

I understand that the author is not responsible and the license is not clear at all. Therefore I just wrote a new rmtrash that basically does the same job as the old one and released under the MIT license.

Would you consider adding my version back as a rmtrash formula?

@SMillerDev
Copy link
Member

This was referenced Dec 11, 2020
@balupton
Copy link

there's also https://github.com/PhrozenByte/rmtrash

machupicchubeta added a commit to machupicchubeta/dotfiles that referenced this pull request Jan 4, 2021
It formulae had already been removed.
- Homebrew/homebrew-core#65438
machupicchubeta added a commit to machupicchubeta/dotfiles that referenced this pull request Jan 4, 2021
It formulae had already been removed.
- Homebrew/homebrew-core#65438

It seems no longer maintained.
- http://www.nightproductions.net/cli.htm
@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Jan 17, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Jan 17, 2021
@chenrui333 chenrui333 added the license License PRs label Feb 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
license License PRs marked for removal/rejection PR is probably going to be closed or formula deleted missing license Formula has a missing license which should be added outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants