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

libmagic optional python bindings #17994

Closed
wants to merge 2 commits into from
Closed

libmagic optional python bindings #17994

wants to merge 2 commits into from

Conversation

syndrowm
Copy link

update to include building python bindings.

update to include building python bindings.
@@ -9,13 +9,19 @@ class Libmagic < Formula
# Fixed upstream, should be in next release
# See http://bugs.gw.com/view.php?id=230
def patches; DATA; end if MacOS.version < :lion


Copy link
Member

Choose a reason for hiding this comment

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

Watch your whitespace ;)

@@ -10,12 +10,19 @@ class Libmagic < Formula
# See http://bugs.gw.com/view.php?id=230
def patches; DATA; end if MacOS.version < :lion

option "python", "Build Python bindings."
Copy link
Contributor

Choose a reason for hiding this comment

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

Call this option "with-python"

@jacknagel
Copy link
Contributor

This seems to install the python stuff outside of its keg... /cc @samueljohn

@samueljohn
Copy link
Contributor

Please see "How to create a formula with nice Python bindings" section in https://github.com/mxcl/homebrew/wiki/Homebrew-and-Python.

I hope to improve python support in the future, so that we can just use --prefix=... but right now we need few more option to be passed to python.

@samueljohn
Copy link
Contributor

Or we wait just until #18588 is ready, so the --prefix=... works as expected.

@samueljohn
Copy link
Contributor

Included in 6e97ac1. With the new depends_on :python this becomes quite easy. Thanks for triggering this! And your patience.

@samueljohn samueljohn closed this Jun 13, 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants