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

gobject-introspection: add head, add python3 which requires head #45442

Closed
wants to merge 2 commits into from

Conversation

moyogo
Copy link
Contributor

@moyogo moyogo commented Oct 29, 2015

Python3 support was added on 2015-09-30 in gobject-introspection, after the 1.46.0 release.

@@ -17,6 +29,7 @@ class GobjectIntrospection < Formula
depends_on "cairo"
depends_on "libffi"
depends_on "python" if MacOS.version <= :mavericks
depends_on "python3" => :optional
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just put this in the head block, then you can bin the if build.with?("python3") && !build.head? logic and this line. Also :python3, not "python3".

@moyogo
Copy link
Contributor Author

moyogo commented Oct 30, 2015

@DomT4 Is this better?

@DomT4
Copy link
Member

DomT4 commented Oct 30, 2015

You've broken the formula there 😉 -

checking whether python1.46.0 version is >= 2.7... no
configure: error: Python interpreter is too old

#{version} in the context you've used it for stable = formula version, not Python version.

system "make", "install"
if build.head?
Language::Python.each_python(build) do |python, version|
sprintf "HEAD #{python} #{version}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why you need to print here.

@moyogo moyogo force-pushed the gobject-introspection/head branch 2 times, most recently from 0e5d03b to 39c4c99 Compare October 31, 2015 20:23
@bfontaine
Copy link
Contributor

This PR needs to be rebased on top of master.

@bfontaine
Copy link
Contributor

Closing due to no response. Feel free to open another PR addressing the comments above. Thanks!

@bfontaine bfontaine closed this Dec 28, 2015
@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
@moyogo moyogo deleted the gobject-introspection/head branch April 20, 2017 14:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants