Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Formula for git-flow AVH Edition #16876

Closed
wants to merge 1 commit into from
Closed

Formula for git-flow AVH Edition #16876

wants to merge 1 commit into from

Conversation

petervanderdoes
Copy link
Contributor

git-flow AVH Edition is an updated, rewritten version of the original git-flow.

@adamv
Copy link
Contributor

adamv commented Jan 3, 2013

Ping @nvie @bobthecow - as authors and users of git-flow, what's the story here?

@adamv
Copy link
Contributor

adamv commented Jan 14, 2013

It seems that this is a more recent, maintained fork of the git-flow code.

Though per policy we don't like packaging forks in this way, we like having one definitive source for a package.

Not sure what exactly to suggest here; I sent some questions out on Twitter but couldn't get any sort of definitive answer from the userbase on which version is preferred.

@jacknagel
Copy link
Contributor

It would at least need to be marked as conflicts_with 'git-flow', though I'm not that comfortable with the fork issue either... it would be nice if it had a different name or something.

@nvie
Copy link
Contributor

nvie commented Jan 14, 2013

The fork has diverged so much that merging it isn't going to happen anymore. I know there are users that like it, but most users use the main repo. With the conflict marker in place, I don't think adding the git-flow-avh package hurts. It gives users a choice, like between fish or fishfish.

@bobthecow
Copy link
Contributor

It gives users a choice, like between fish or fishfish.

I was going to make the same analogy :)

I decided it was more like Notational Velocity vs nvALT. While the latter has a number of passionate users, the main project is still Notational Velocity. With fish, the main project is effectively now fishfish.

@petervanderdoes
Copy link
Contributor Author

As suggested I added the conflicts_with line in the formulas git-flow-avh.rb anf git-flow.rb to conflict with each other.

@deepakinseattle
Copy link

+1 for this formula.

@aaronzirbes
Copy link

👍 for this formula. The old one is outdated and dead.

@adamv
Copy link
Contributor

adamv commented Jan 24, 2013

Please squash to a single commit, thanks.

@petervanderdoes
Copy link
Contributor Author

@adamv Squashed the commits.

@petervanderdoes
Copy link
Contributor Author

Fixed the missing prefix for the make command.

@adamv adamv closed this in 6d382cd Jan 26, 2013
@Homebrew Homebrew 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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants