CogComp / lbjava

Learning Based Java (LBJava)
http://cogcomp.cs.illinois.edu/page/software_view/LBJava
Other
13 stars 17 forks source link

Removing the lbj version number from scripts #16

Closed danyaljj closed 8 years ago

danyaljj commented 8 years ago

removing the lbj version number from scripts, so that CI doesn't fail upon version increase

christos-c commented 8 years ago

@danyaljj Can you update the illinois-core-utilities version since you're changing the script? Also, I noticed there are two places where this dependency exist: in the pom.xml files as well as a static jar in the lib/ directory. Is there a way to unite these?

danyaljj commented 8 years ago

Yeah this duplicate core-utilities is really annoying. I think this compileLBJ.sh script is really a temporary solution. I am trying to convince @mayhewsw to use his magical techniques to bypass this step with lbjava-maven-plugin.

danyaljj commented 8 years ago

core-utils updated.

christos-c commented 8 years ago

You forgot the one in the example's pom!

danyaljj commented 8 years ago

done.