Permalink
Browse files

konoha: Update URL, fix build on 10.8

Is anybody using this at all?
Had to disable mecab as it is no longer in OS X 10.8.
  • Loading branch information...
1 parent 08de5da commit 0cddc3583d084f757b2cb27b7ab1adfce53510bb @samueljohn samueljohn committed Aug 27, 2013
Showing with 10 additions and 4 deletions.
  1. +10 −4 Library/Formula/konoha.rb
View
14 Library/Formula/konoha.rb
@@ -2,8 +2,8 @@
class Konoha < Formula
homepage 'http://www.konohascript.org/'
- url 'https://github.com/downloads/konoha-project/minikonoha/minikonoha-0.1.0.tar.gz'
- sha1 '621aa1244c344a9e7be96fb6e6d067bae7f43d64'
+ url 'https://github.com/konoha-project/konoha3/archive/v0.1.tar.gz'
+ sha1 '85ee3527c9a2db2874280e506c09aab8809a6ad5'
option 'tests', 'Verify the build with make test (1 min)'
@@ -12,14 +12,20 @@ class Konoha < Formula
depends_on 'pcre'
depends_on 'json-c'
depends_on 'sqlite'
+ depends_on :python # for python glue code
def install
args = std_cmake_args + ['..']
+ # mecab is no longer in the 10.8 SDK but superenv does not (yet) support
+ # switching to an older SDK. And if we add the include path so that
+ # `mecab.h` is found, we get more errors in mecab_glue.c:
+ # error: no member named 'id' in 'struct mecab_node_t'
+ args << "-DHAVE_LIBMECAB=OFF" if MacOS.version >= :mountain_lion
mkdir 'build' do
system 'cmake', *args
system 'make'
- system 'make test' if build.include? 'tests'
- system 'make install'
+ system 'make', 'test' if build.include? 'tests'
+ system 'make', 'install'
end
end
end

0 comments on commit 0cddc35

Please sign in to comment.