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

Import formulae from homebrew/games. #9685

Closed
wants to merge 175 commits into from
Closed

Import formulae from homebrew/games. #9685

wants to merge 175 commits into from

Conversation

MikeMcQuaid
Copy link
Member

@ilovezfs
Copy link
Contributor

ilovezfs commented Feb 6, 2017

Looks like something went wrong. Only 16/174.

@MikeMcQuaid
Copy link
Member Author

@ilovezfs Have rerun with --debug, let's see what happens.

@JCount
Copy link
Contributor

JCount commented Feb 6, 2017

@MikeMcQuaid The issue seems to be with the blastem formula itself. It still depends on homebrew/python/pillow instead of homebrew/science/pillow. This seems cause the failure of entire job when the CI cannot find the pillow formula after tapping homebrew/python.

@mistydemeo
Copy link
Contributor

I'll look at fixing anything here that has issues before we make a decision on removing anything.

@MikeMcQuaid
Copy link
Member Author

@mistydemeo That's fine but I'd like to time box it. How does a week sound (i.e. ship this on Wednesday 15th). I would say that the things that don't install at all as-is without fixes are unlikely to be widely used so personally don't think they are good candidates for inclusion.

@MikeMcQuaid MikeMcQuaid deleted the import-homebrew-games branch February 8, 2017 10:13
@mistydemeo
Copy link
Contributor

That's fine but I'd like to time box it. How does a week sound (i.e. ship this on Wednesday 15th).

I've been at a conference, so I haven't had a chance to look before today. Let's timebox it for a week today (18th AEST).

@mistydemeo
Copy link
Contributor

Oops, missed this was already shipped.

@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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants