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

mmark 1.3.4 (new formula) #7792

Closed
wants to merge 1 commit into from
Closed

mmark 1.3.4 (new formula) #7792

wants to merge 1 commit into from

Conversation

dericed
Copy link
Contributor

@dericed dericed commented Dec 12, 2016

No description provided.

@dericed
Copy link
Contributor Author

dericed commented Dec 12, 2016

In brew audit --new-formula mmark.rb I only get this error:

  • GitHub fork (not canonical repository)

I hope the patch can be considered despite this as mmark contains significant development not present in the canonical repo. mmark has 1,134 commits while the canonical blackfriday has 391.

@woodruffw woodruffw changed the title add mmark formula mmark 1.3.4 (new formula) Dec 12, 2016
@woodruffw woodruffw added the new formula PR adds a new formula to Homebrew/homebrew-core label Dec 12, 2016
@woodruffw
Copy link
Member

Pinging @ilovezfs since I'm not sure what our policy is here -- both seem to be under semi-active development, mmark has more commits but blackfriday seems to be more publicized.

@ilovezfs
Copy link
Contributor

@woodruffw there's no hard rule, but typically would be guided by who has the lion's share of users.

@dericed
Copy link
Contributor Author

dericed commented Dec 13, 2016

Cc'ing @miekg (maintainer of mmark), but want to note that mmark and blackfriday have different user groups. As an IETF contributor I regularly use mmark, but haven't had any need to use blackfriday which is very different. mmark is 877 commits ahead and 134 commits behind blackfriday. I know with other forks, such as libav and ffmpeg, homebrew has supported installers for both rather than only the canonical one.

@miekg
Copy link

miekg commented Dec 13, 2016 via email

@ilovezfs
Copy link
Contributor

This no-fork policy is silly

You can save the editorializing. As I said above, there is no hard rule here. The audit check is only run on new formulae so if we decide it's fine, then it won't subsequently cause an audit failure the next time the formula goes through CI as a non-new formula.

@MikeMcQuaid
Copy link
Member

@miekg Also, if you wish you can get someone who works at GitHub (i.e. me) to make it not a fork any more if you'd like that. That said, there's no reason to do that purely for Homebrew's benefit.

@MikeMcQuaid
Copy link
Member

Thanks for your contribution to Homebrew! Without people like you submitting PRs we couldn't run this project. You rock!

For future reference the preferred commit message format is mmark 1.3.4 (new formula). Please complete the issue template in future PRs where this format is detailed in the linked Contributing guidelines. Thanks!

@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new formula PR adds a new formula to Homebrew/homebrew-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants