Arcanist #14445

Closed
wants to merge 3 commits into
from

4 participants

@Hebo

Arcanist, facebook's OSS code review tool

@adamv

Need to use a tagged download.

@adamv

Need to use a tagged download.

@adamv

That's not actually a tag; can we ask the upstream project to add a 1.0.1 git tag?

@adamv

Per official documentation http://www.phabricator.com/docs/phabricator/article/Arcanist_User_Guide.html#installing-arcanist this is a head-only formula; rejecting for inclusion in core because of this (sorry.)

@adamv adamv closed this Oct 24, 2012
@gnzlbg

@adamv Why are head-only formulas not allowed?

@MikeMcQuaid
Homebrew member

@gnzlbg Because if upstream pushes a bad commit then it breaks the formula and we have no way of fixing it.

@gnzlbg

@mikemcquaid if upstream pushes a bad commit the formula remains correct, only the code/software is broken. That is upstream fault due to the bad commit. People interested in providing a workaround could do something about it (like temporarily checking out a given revision) but I think it is not worth it.

@MikeMcQuaid
Homebrew member

It's a poor user experience, regardless. That's why we don't support them in core.

@gnzlbg

Maybe one could support them in a different repo that one can tap like homebrew-heads or similar.

@gnzlbg

@mikemcquaid my bad :D

@vmrob vmrob referenced this pull request in Homebrew/homebrew-head-only Jan 17, 2014
Closed

Arcanist, a command-line interface to Phabricator #34

@xu-cheng xu-cheng locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.