ztellman / penumbra

not under active development - idiomatic opengl bindings for clojure
354 stars 43 forks source link

targeting clojure.contrib-1.2.0 #16

Closed krumholt closed 14 years ago

krumholt commented 14 years ago

Hi, is there a reason not to target clojure.contrib-1.2.0 ? The only problem seems to be that seq-utils has been renamed to seq. See http://groups.google.com/group/clojure-dev/browse_thread/thread/49068754a8c2efb9?pli=1 This "seems" to be the only problem. If there is no other reason not to target 1.2.0 you can merge from my repository if you like.

ztellman commented 14 years ago

In 0.5.0, I'm targeting 1.1.0 for both clojure and clojure-contrib, since Leiningen doesn't play nicely with anything newer. I do plan, however, to target clojure-contrib-1.2.0 in the master branch, I just hadn't gone through the trouble of making the necessary changes.

lkuty commented 14 years ago

I just did a sed to replace seq-utils by seq and it looks like it is working. After taking the master branch of penumbra, running leiningen, adapting the script to launch it and running sed on the source code, it works !

find src -name "*.clj" | xargs -n 1  sed -i "" "s/seq-utils/seq/"

Script to launch penumbra:

#!/bin/bash

BREAK_CHARS="(){}[],^%$#@\"\";:''|\\"

# Augment the Java CLASSPATH
export CLASSPATH=$CLASSPATH:/Users/ludo/Applications/clojure/clojure-1.2.0-master-SNAPSHOT.jar:/Users/ludo/Applications/clojure/clojure-contrib-1.2.0-SNAPSHOT.jar:/Users/ludo/Applications/clojure/penumbra/src:/Users/ludo/Applications/clojure/penumbra/test:/Users/ludo/Applications/clojure/penumbra/lib/*

# Default command to launch Java is java
JAVA=${CLOJURE_JAVA:-java}

# No default options
OPTS=${CLOJURE_OPTS:-}

OPTS="$OPTS -Djava.library.path=/Users/ludo/Applications/clojure/penumbra/native/macosx/x86_64"

if [ $# -eq 0 ]; then 
   exec rlwrap --remember -c -b $BREAK_CHARS -f $HOME/.clojure/clj_completions $JAVA $OPTS clojure.main -i ~/.clojure/cljrc.clj -r
else
   exec $JAVA $OPTS clojure.main -i ~/.clojure/cljrc.clj "$@"
fi
ztellman commented 14 years ago

This should all be sorted out now.

krumholt commented 14 years ago

Thanks. Much appreciated!